test: cleanup users after tests are run

CONTRIBUTING.md

@@ -21,7 +21,7 @@ These will automatically watch the filesystem for changes, and restart when need
 
 There are also two development settings you have to change to develop the frontend.
 
-Before commiting your changes, you should run `yarn lint` to fix any linting errors.
+Before commiting your changes, you should run `yarn lint` and `yarn lint --fix` to fix any linting errors.
 
 You should also use `yarn test` to ensure there are no regressions.
 

server/test/integration/auth.js

@@ -17,7 +17,7 @@ beforeAll(async () => {
   await app.ready()
 })
 
-test('cleanup test user', async () => {
+afterAll('cleanup test user', async () => {
   config.email = oldEmail
   await removeUserByEmail({
     email: testUser.email

server/test/integration/challenges.js

@@ -17,7 +17,7 @@ beforeAll(async () => {
   uuid = testUserData.user.id
 })
 
-test('cleanup test user', async () => {
+afterAll('cleanup test user', async () => {
   await testUserData.cleanup()
 })
 

server/test/integration/profile.js

@@ -37,7 +37,7 @@ test('succeeds with goodUserData', async () => {
   expect(resp.body.data.name).toBe(testUser.name)
 })
 
-test('cleanup test user', async () => {
+afterAll('cleanup test user', async () => {
   await removeUserByEmail({
     email: testUser.email
   })

server/test/integration/submit.js

@@ -20,7 +20,7 @@ beforeAll(async () => {
   uuid = testUserData.user.id
 })
 
-test('cleanup test user', async () => {
+afterAll('cleanup test user', async () => {
   await db.solves.removeSolvesByUserId({ userid: uuid })
   await testUserData.cleanup()
 })

server/test/integration/submitTiming.js

@@ -19,7 +19,7 @@ beforeAll(async () => {
   uuid = testUserData.user.id
 })
 
-test('cleanup test user', async () => {
+afterAll('cleanup test user', async () => {
   await testUserData.cleanup()
 })