logger now mocked propperly

2025-12-13 07:45:54 +00:00 · 2022-10-28 11:22:58 +02:00 · 2022-10-28 11:22:58 +02:00 · 9846e095bf
commit 9846e095bf
parent 87e0f82a9f
4 changed files with 50 additions and 0 deletions
--- a/backend/src/graphql/resolver/AdminResolver.test.ts
+++ b/backend/src/graphql/resolver/AdminResolver.test.ts
@ -139,6 +139,7 @@ describe('AdminResolver', () => {

        describe('user to get a new role does not exist', () => {
          it('throws an error', async () => {
+            jest.clearAllMocks()
            await expect(
              mutate({ mutation: setUserRole, variables: { userId: admin.id + 1, isAdmin: true } }),
            ).resolves.toEqual(
@ -195,6 +196,7 @@ describe('AdminResolver', () => {
        describe('change role with error', () => {
          describe('is own role', () => {
            it('throws an error', async () => {
+              jest.clearAllMocks()
              await expect(
                mutate({ mutation: setUserRole, variables: { userId: admin.id, isAdmin: false } }),
              ).resolves.toEqual(
@ -211,6 +213,7 @@ describe('AdminResolver', () => {
          describe('user has already role to be set', () => {
            describe('to admin', () => {
              it('throws an error', async () => {
+                jest.clearAllMocks()
                await mutate({
                  mutation: setUserRole,
                  variables: { userId: user.id, isAdmin: true },
@ -231,6 +234,7 @@ describe('AdminResolver', () => {

            describe('to usual user', () => {
              it('throws an error', async () => {
+                jest.clearAllMocks()
                await mutate({
                  mutation: setUserRole,
                  variables: { userId: user.id, isAdmin: false },
@ -307,6 +311,7 @@ describe('AdminResolver', () => {

        describe('user to be deleted does not exist', () => {
          it('throws an error', async () => {
+            jest.clearAllMocks()
            await expect(
              mutate({ mutation: deleteUser, variables: { userId: admin.id + 1 } }),
            ).resolves.toEqual(
@ -323,6 +328,7 @@ describe('AdminResolver', () => {

        describe('delete self', () => {
          it('throws an error', async () => {
+            jest.clearAllMocks()
            await expect(
              mutate({ mutation: deleteUser, variables: { userId: admin.id } }),
            ).resolves.toEqual(
@ -356,6 +362,7 @@ describe('AdminResolver', () => {

          describe('delete deleted user', () => {
            it('throws an error', async () => {
+              jest.clearAllMocks()
              await expect(
                mutate({ mutation: deleteUser, variables: { userId: user.id } }),
              ).resolves.toEqual(
@ -427,6 +434,7 @@ describe('AdminResolver', () => {

        describe('user to be undelete does not exist', () => {
          it('throws an error', async () => {
+            jest.clearAllMocks()
            await expect(
              mutate({ mutation: unDeleteUser, variables: { userId: admin.id + 1 } }),
            ).resolves.toEqual(
@ -447,6 +455,7 @@ describe('AdminResolver', () => {
          })

          it('throws an error', async () => {
+            jest.clearAllMocks()
            await expect(
              mutate({ mutation: unDeleteUser, variables: { userId: user.id } }),
            ).resolves.toEqual(
@ -939,6 +948,7 @@ describe('AdminResolver', () => {

          describe('user to create for does not exist', () => {
            it('throws an error', async () => {
+              jest.clearAllMocks()
              await expect(
                mutate({ mutation: adminCreateContribution, variables }),
              ).resolves.toEqual(
@ -962,6 +972,7 @@ describe('AdminResolver', () => {
            })

            it('throws an error', async () => {
+              jest.clearAllMocks()
              await expect(
                mutate({ mutation: adminCreateContribution, variables }),
              ).resolves.toEqual(
@ -987,6 +998,7 @@ describe('AdminResolver', () => {
            })

            it('throws an error', async () => {
+              jest.clearAllMocks()
              await expect(
                mutate({ mutation: adminCreateContribution, variables }),
              ).resolves.toEqual(
@ -1013,6 +1025,7 @@ describe('AdminResolver', () => {

            describe('date of creation is not a date string', () => {
              it('throws an error', async () => {
+                jest.clearAllMocks()
                await expect(
                  mutate({ mutation: adminCreateContribution, variables }),
                ).resolves.toEqual(
@ -1034,6 +1047,7 @@ describe('AdminResolver', () => {

            describe('date of creation is four months ago', () => {
              it('throws an error', async () => {
+                jest.clearAllMocks()
                const now = new Date()
                variables.creationDate = new Date(
                  now.getFullYear(),
@ -1061,6 +1075,7 @@ describe('AdminResolver', () => {

            describe('date of creation is in the future', () => {
              it('throws an error', async () => {
+                jest.clearAllMocks()
                const now = new Date()
                variables.creationDate = new Date(
                  now.getFullYear(),
@ -1088,6 +1103,7 @@ describe('AdminResolver', () => {

            describe('amount of creation is too high', () => {
              it('throws an error', async () => {
+                jest.clearAllMocks()
                variables.creationDate = new Date().toString()
                await expect(
                  mutate({ mutation: adminCreateContribution, variables }),
@ -1213,6 +1229,7 @@ describe('AdminResolver', () => {

          describe('user for creation to update does not exist', () => {
            it('throws an error', async () => {
+              jest.clearAllMocks()
              await expect(
                mutate({
                  mutation: adminUpdateContribution,
@ -1242,6 +1259,7 @@ describe('AdminResolver', () => {

          describe('user for creation to update is deleted', () => {
            it('throws an error', async () => {
+              jest.clearAllMocks()
              await expect(
                mutate({
                  mutation: adminUpdateContribution,
@ -1267,6 +1285,7 @@ describe('AdminResolver', () => {

          describe('creation does not exist', () => {
            it('throws an error', async () => {
+              jest.clearAllMocks()
              await expect(
                mutate({
                  mutation: adminUpdateContribution,
@ -1292,6 +1311,7 @@ describe('AdminResolver', () => {

          describe('user email does not match creation user', () => {
            it('throws an error', async () => {
+              jest.clearAllMocks()
              await expect(
                mutate({
                  mutation: adminUpdateContribution,
@ -1326,6 +1346,7 @@ describe('AdminResolver', () => {
          describe('creation update is not valid', () => {
            // as this test has not clearly defined that date, it is a false positive
            it('throws an error', async () => {
+              jest.clearAllMocks()
              await expect(
                mutate({
                  mutation: adminUpdateContribution,
@ -1502,6 +1523,7 @@ describe('AdminResolver', () => {
        describe('adminDeleteContribution', () => {
          describe('creation id does not exist', () => {
            it('throws an error', async () => {
+              jest.clearAllMocks()
              await expect(
                mutate({
                  mutation: adminDeleteContribution,
@ -1538,6 +1560,7 @@ describe('AdminResolver', () => {
            })

            it('throws an error', async () => {
+              jest.clearAllMocks()
              await expect(
                mutate({
                  mutation: adminDeleteContribution,
@ -1583,6 +1606,7 @@ describe('AdminResolver', () => {
        describe('confirmContribution', () => {
          describe('creation does not exits', () => {
            it('throws an error', async () => {
+              jest.clearAllMocks()
              await expect(
                mutate({
                  mutation: confirmContribution,
--- a/backend/src/graphql/resolver/ContributionResolver.test.ts
+++ b/backend/src/graphql/resolver/ContributionResolver.test.ts
@ -375,6 +375,7 @@ describe('ContributionResolver', () => {

      describe('wrong contribution id', () => {
        it('throws an error', async () => {
+          jest.clearAllMocks()
          await expect(
            mutate({
              mutation: updateContribution,
@ -456,6 +457,7 @@ describe('ContributionResolver', () => {
        })

        it('throws an error', async () => {
+          jest.clearAllMocks()
          await expect(
            mutate({
              mutation: updateContribution,
@ -486,6 +488,7 @@ describe('ContributionResolver', () => {

      describe('admin tries to update a user contribution', () => {
        it('throws an error', async () => {
+          jest.clearAllMocks()
          await expect(
            mutate({
              mutation: adminUpdateContribution,
@ -516,6 +519,7 @@ describe('ContributionResolver', () => {
        })

        it('throws an error', async () => {
+          jest.clearAllMocks()
          await expect(
            mutate({
              mutation: updateContribution,
@ -546,6 +550,7 @@ describe('ContributionResolver', () => {

      describe('update creation to a date that is older than 3 months', () => {
        it('throws an error', async () => {
+          jest.clearAllMocks()
          const date = new Date()
          await expect(
            mutate({
@ -830,6 +835,7 @@ describe('ContributionResolver', () => {

      describe('User deletes already confirmed contribution', () => {
        it('throws an error', async () => {
+          jest.clearAllMocks()
          await mutate({
            mutation: login,
            variables: { email: 'peter@lustig.de', password: 'Aa12345_' },
--- a/backend/src/graphql/resolver/TransactionResolver.test.ts
+++ b/backend/src/graphql/resolver/TransactionResolver.test.ts
@ -67,6 +67,7 @@ describe('send coins', () => {

  describe('unknown recipient', () => {
    it('throws an error', async () => {
+      jest.clearAllMocks()
      await mutate({
        mutation: login,
        variables: bobData,
@ -93,6 +94,7 @@ describe('send coins', () => {

    describe('deleted recipient', () => {
      it('throws an error', async () => {
+        jest.clearAllMocks()
        await mutate({
          mutation: login,
          variables: peterData,
@ -125,6 +127,7 @@ describe('send coins', () => {

    describe('recipient account not activated', () => {
      it('throws an error', async () => {
+        jest.clearAllMocks()
        await mutate({
          mutation: login,
          variables: peterData,
@ -166,6 +169,7 @@ describe('send coins', () => {

    describe('sender and recipient are the same', () => {
      it('throws an error', async () => {
+        jest.clearAllMocks()
        expect(
          await mutate({
            mutation: sendCoins,
@ -189,6 +193,7 @@ describe('send coins', () => {

    describe('memo text is too long', () => {
      it('throws an error', async () => {
+        jest.clearAllMocks()
        expect(
          await mutate({
            mutation: sendCoins,
@ -212,6 +217,7 @@ describe('send coins', () => {

    describe('memo text is too short', () => {
      it('throws an error', async () => {
+        jest.clearAllMocks()
        expect(
          await mutate({
            mutation: sendCoins,
@ -235,6 +241,7 @@ describe('send coins', () => {

    describe('user has not enough GDD', () => {
      it('throws an error', async () => {
+        jest.clearAllMocks()
        expect(
          await mutate({
            mutation: sendCoins,
@ -260,6 +267,7 @@ describe('send coins', () => {

    describe('sending negative amount', () => {
      it('throws an error', async () => {
+        jest.clearAllMocks()
        expect(
          await mutate({
            mutation: sendCoins,
--- a/backend/src/graphql/resolver/UserResolver.test.ts
+++ b/backend/src/graphql/resolver/UserResolver.test.ts
@ -525,6 +525,7 @@ describe('UserResolver', () => {
      })

      it('throws an error', () => {
+        jest.clearAllMocks()
        expect(result).toEqual(
          expect.objectContaining({
            errors: [
@ -555,6 +556,7 @@ describe('UserResolver', () => {
      })

      it('throws an error', () => {
+        jest.clearAllMocks()
        expect(result).toEqual(
          expect.objectContaining({
            errors: [new GraphQLError('Could not login with emailVerificationCode')],
@ -588,6 +590,7 @@ describe('UserResolver', () => {
      })

      it('throws an error', () => {
+        jest.clearAllMocks()
        expect(result).toEqual(
          expect.objectContaining({
            errors: [new GraphQLError('No user with this credentials')],
@ -666,6 +669,7 @@ describe('UserResolver', () => {
  describe('logout', () => {
    describe('unauthenticated', () => {
      it('throws an error', async () => {
+        jest.clearAllMocks()
        resetToken()
        await expect(mutate({ mutation: logout })).resolves.toEqual(
          expect.objectContaining({
@ -704,6 +708,7 @@ describe('UserResolver', () => {
  describe('verifyLogin', () => {
    describe('unauthenticated', () => {
      it('throws an error', async () => {
+        jest.clearAllMocks()
        resetToken()
        await expect(query({ query: verifyLogin })).resolves.toEqual(
          expect.objectContaining({
@ -723,6 +728,7 @@ describe('UserResolver', () => {
      })

      it('throws an error', async () => {
+        jest.clearAllMocks()
        resetToken()
        await expect(query({ query: verifyLogin })).resolves.toEqual(
          expect.objectContaining({
@ -883,6 +889,7 @@ describe('UserResolver', () => {

    describe('wrong optin code', () => {
      it('throws an error', async () => {
+        jest.clearAllMocks()
        await expect(
          query({ query: queryOptIn, variables: { optIn: 'not-valid' } }),
        ).resolves.toEqual(
@ -919,6 +926,7 @@ describe('UserResolver', () => {
  describe('updateUserInfos', () => {
    describe('unauthenticated', () => {
      it('throws an error', async () => {
+        jest.clearAllMocks()
        resetToken()
        await expect(mutate({ mutation: updateUserInfos })).resolves.toEqual(
          expect.objectContaining({
@ -976,6 +984,7 @@ describe('UserResolver', () => {

      describe('language is not valid', () => {
        it('throws an error', async () => {
+          jest.clearAllMocks()
          await expect(
            mutate({
              mutation: updateUserInfos,
@ -998,6 +1007,7 @@ describe('UserResolver', () => {
      describe('password', () => {
        describe('wrong old password', () => {
          it('throws an error', async () => {
+            jest.clearAllMocks()
            await expect(
              mutate({
                mutation: updateUserInfos,
@ -1020,6 +1030,7 @@ describe('UserResolver', () => {

        describe('invalid new password', () => {
          it('throws an error', async () => {
+            jest.clearAllMocks()
            await expect(
              mutate({
                mutation: updateUserInfos,
@ -1108,6 +1119,7 @@ describe('UserResolver', () => {
  describe('searchAdminUsers', () => {
    describe('unauthenticated', () => {
      it('throws an error', async () => {
+        jest.clearAllMocks()
        resetToken()
        await expect(mutate({ mutation: searchAdminUsers })).resolves.toEqual(
          expect.objectContaining({