From 2e5b44c4248c4e0267fc671ac8c6eb85cefddce1 Mon Sep 17 00:00:00 2001 From: Faruk AYDIN Date: Fri, 25 Oct 2024 12:33:39 +0200 Subject: [PATCH] refactor: Use toStrictEqual instead of toEqual for tests --- .../api/v1/access-tokens/create-access-token.test.js | 2 +- .../api/v1/admin/apps/create-auth-client.ee.test.js | 2 +- .../api/v1/admin/apps/create-config.ee.test.js | 2 +- .../api/v1/admin/apps/get-auth-client.ee.test.js | 2 +- .../api/v1/admin/apps/get-auth-clients.ee.test.js | 2 +- .../api/v1/admin/apps/update-config.ee.test.js | 2 +- .../controllers/api/v1/admin/config/update.ee.test.js | 10 +++++----- .../permissions/get-permissions-catalog.ee.test.js | 2 +- .../api/v1/admin/roles/create-role.ee.test.js | 2 +- .../controllers/api/v1/admin/roles/get-role.ee.test.js | 2 +- .../api/v1/admin/roles/get-roles.ee.test.js | 2 +- .../saml-auth-providers/get-role-mappings.ee.test.js | 2 +- .../get-saml-auth-provider.ee.test.js | 2 +- .../get-saml-auth-providers.ee.test.js | 2 +- .../controllers/api/v1/admin/users/get-user.ee.test.js | 2 +- .../api/v1/admin/users/get-users.ee.test.js | 2 +- .../api/v1/admin/users/update-user.ee.test.js | 3 ++- .../api/v1/apps/get-action-substeps.test.js | 4 ++-- .../src/controllers/api/v1/apps/get-actions.test.js | 2 +- .../src/controllers/api/v1/apps/get-app.test.js | 2 +- .../src/controllers/api/v1/apps/get-apps.test.js | 8 ++++---- .../controllers/api/v1/apps/get-auth-client.ee.test.js | 2 +- .../api/v1/apps/get-auth-clients.ee.test.js | 2 +- .../src/controllers/api/v1/apps/get-auth.test.js | 2 +- .../src/controllers/api/v1/apps/get-config.ee.test.js | 2 +- .../controllers/api/v1/apps/get-connections.test.js | 4 ++-- .../src/controllers/api/v1/apps/get-flows.test.js | 4 ++-- .../api/v1/apps/get-trigger-substeps.test.js | 4 ++-- .../src/controllers/api/v1/apps/get-triggers.test.js | 2 +- .../src/controllers/api/v1/automatisch/info.test.js | 2 +- .../src/controllers/api/v1/automatisch/license.test.js | 2 +- .../src/controllers/api/v1/automatisch/version.test.js | 2 +- .../controllers/api/v1/connections/get-flows.test.js | 4 ++-- .../api/v1/connections/test-connection.test.js | 4 ++-- .../api/v1/connections/verify-connection.test.js | 2 +- .../api/v1/executions/get-execution-steps.test.js | 4 ++-- .../api/v1/executions/get-execution.test.js | 4 ++-- .../api/v1/executions/get-executions.test.js | 4 ++-- .../src/controllers/api/v1/flows/get-flow.test.js | 4 ++-- .../src/controllers/api/v1/flows/get-flows.test.js | 4 ++-- .../api/v1/installation/users/create-user.test.js | 2 +- .../api/v1/payment/get-paddle-info.ee.test.js | 2 +- .../controllers/api/v1/payment/get-plans.ee.test.js | 2 +- .../get-saml-auth-providers.ee.test.js | 2 +- .../api/v1/steps/create-dynamic-data.test.js | 6 +++--- .../api/v1/steps/create-dynamic-fields.test.js | 4 ++-- .../controllers/api/v1/steps/get-connection.test.js | 4 ++-- .../api/v1/steps/get-previous-steps.test.js | 4 ++-- .../src/controllers/api/v1/users/get-apps.test.js | 8 ++++---- .../controllers/api/v1/users/get-current-user.test.js | 2 +- .../controllers/api/v1/users/get-invoices.ee.test.js | 2 +- .../api/v1/users/get-plan-and-usage.ee.test.js | 4 ++-- .../api/v1/users/get-subscription.ee.test.js | 2 +- .../controllers/api/v1/users/get-user-trial.ee.test.js | 2 +- .../api/v1/users/update-current-user-password.test.js | 2 +- .../api/v1/users/update-current-user.test.js | 3 ++- packages/backend/src/models/app-auth-client.test.js | 8 ++++++-- packages/backend/src/models/connection.test.js | 4 ++-- packages/backend/src/serializers/action.test.js | 2 +- .../serializers/admin-saml-auth-provider.ee.test.js | 2 +- packages/backend/src/serializers/admin/user.test.js | 2 +- .../backend/src/serializers/app-auth-client.test.js | 4 +++- packages/backend/src/serializers/app.test.js | 2 +- packages/backend/src/serializers/auth.test.js | 2 +- packages/backend/src/serializers/config.test.js | 2 +- packages/backend/src/serializers/connection.test.js | 2 +- .../backend/src/serializers/execution-step.test.js | 4 +++- packages/backend/src/serializers/execution.test.js | 4 ++-- packages/backend/src/serializers/flow.test.js | 2 +- packages/backend/src/serializers/permission.test.js | 2 +- packages/backend/src/serializers/role.test.js | 2 +- .../src/serializers/saml-auth-provider.ee.test.js | 2 +- packages/backend/src/serializers/step.test.js | 2 +- .../backend/src/serializers/subscription.ee.test.js | 2 +- packages/backend/src/serializers/trigger.test.js | 2 +- packages/backend/src/serializers/user.test.js | 2 +- .../mocks/rest/api/v1/admin/apps/get-auth-client.js | 1 - .../test/mocks/rest/api/v1/admin/users/get-user.js | 1 - .../test/mocks/rest/api/v1/admin/users/get-users.js | 1 - .../test/mocks/rest/api/v1/apps/get-auth-client.js | 1 - .../backend/test/mocks/rest/api/v1/apps/get-config.js | 1 - .../test/mocks/rest/api/v1/users/get-current-user.js | 1 - 82 files changed, 116 insertions(+), 112 deletions(-) diff --git a/packages/backend/src/controllers/api/v1/access-tokens/create-access-token.test.js b/packages/backend/src/controllers/api/v1/access-tokens/create-access-token.test.js index cc54b094..2232cb5e 100644 --- a/packages/backend/src/controllers/api/v1/access-tokens/create-access-token.test.js +++ b/packages/backend/src/controllers/api/v1/access-tokens/create-access-token.test.js @@ -32,7 +32,7 @@ describe('POST /api/v1/access-tokens', () => { }) .expect(422); - expect(response.body.errors.general).toEqual([ + expect(response.body.errors.general).toStrictEqual([ 'Incorrect email or password.', ]); }); diff --git a/packages/backend/src/controllers/api/v1/admin/apps/create-auth-client.ee.test.js b/packages/backend/src/controllers/api/v1/admin/apps/create-auth-client.ee.test.js index 57afc704..ea658f88 100644 --- a/packages/backend/src/controllers/api/v1/admin/apps/create-auth-client.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/apps/create-auth-client.ee.test.js @@ -83,7 +83,7 @@ describe('POST /api/v1/admin/apps/:appKey/auth-clients', () => { .send(appAuthClient) .expect(422); - expect(response.body.meta.type).toEqual('ModelValidation'); + expect(response.body.meta.type).toStrictEqual('ModelValidation'); expect(response.body.errors).toMatchObject({ name: ["must have required property 'name'"], formattedAuthDefaults: [ diff --git a/packages/backend/src/controllers/api/v1/admin/apps/create-config.ee.test.js b/packages/backend/src/controllers/api/v1/admin/apps/create-config.ee.test.js index f92df586..9d59a699 100644 --- a/packages/backend/src/controllers/api/v1/admin/apps/create-config.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/apps/create-config.ee.test.js @@ -59,7 +59,7 @@ describe('POST /api/v1/admin/apps/:appKey/config', () => { }) .expect(422); - expect(response.body.meta.type).toEqual('UniqueViolationError'); + expect(response.body.meta.type).toStrictEqual('UniqueViolationError'); expect(response.body.errors).toMatchObject({ key: ["'key' must be unique."], }); diff --git a/packages/backend/src/controllers/api/v1/admin/apps/get-auth-client.ee.test.js b/packages/backend/src/controllers/api/v1/admin/apps/get-auth-client.ee.test.js index a3b1c6c4..2edb0ffe 100644 --- a/packages/backend/src/controllers/api/v1/admin/apps/get-auth-client.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/apps/get-auth-client.ee.test.js @@ -32,7 +32,7 @@ describe('GET /api/v1/admin/apps/:appKey/auth-clients/:appAuthClientId', () => { .expect(200); const expectedPayload = getAppAuthClientMock(currentAppAuthClient); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for not existing app auth client ID', async () => { diff --git a/packages/backend/src/controllers/api/v1/admin/apps/get-auth-clients.ee.test.js b/packages/backend/src/controllers/api/v1/admin/apps/get-auth-clients.ee.test.js index 58c25bff..7fbba6e0 100644 --- a/packages/backend/src/controllers/api/v1/admin/apps/get-auth-clients.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/apps/get-auth-clients.ee.test.js @@ -39,6 +39,6 @@ describe('GET /api/v1/admin/apps/:appKey/auth-clients', () => { appAuthClientOne, ]); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/admin/apps/update-config.ee.test.js b/packages/backend/src/controllers/api/v1/admin/apps/update-config.ee.test.js index c5436ad5..3b1fb8ab 100644 --- a/packages/backend/src/controllers/api/v1/admin/apps/update-config.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/apps/update-config.ee.test.js @@ -83,7 +83,7 @@ describe('PATCH /api/v1/admin/apps/:appKey/config', () => { }) .expect(422); - expect(response.body.meta.type).toEqual('ModelValidation'); + expect(response.body.meta.type).toStrictEqual('ModelValidation'); expect(response.body.errors).toMatchObject({ disabled: ['must be boolean'], }); diff --git a/packages/backend/src/controllers/api/v1/admin/config/update.ee.test.js b/packages/backend/src/controllers/api/v1/admin/config/update.ee.test.js index bf4b103c..5984cdbb 100644 --- a/packages/backend/src/controllers/api/v1/admin/config/update.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/config/update.ee.test.js @@ -50,8 +50,8 @@ describe('PATCH /api/v1/admin/config', () => { .send(newConfigValues) .expect(200); - expect(response.body.data.title).toEqual(newTitle); - expect(response.body.meta.type).toEqual('Config'); + expect(response.body.data.title).toStrictEqual(newTitle); + expect(response.body.meta.type).toStrictEqual('Config'); }); it('should return created config for unexisting config', async () => { @@ -67,8 +67,8 @@ describe('PATCH /api/v1/admin/config', () => { .send(newConfigValues) .expect(200); - expect(response.body.data.title).toEqual(newTitle); - expect(response.body.meta.type).toEqual('Config'); + expect(response.body.data.title).toStrictEqual(newTitle); + expect(response.body.meta.type).toStrictEqual('Config'); }); it('should return null for deleted config entry', async () => { @@ -83,6 +83,6 @@ describe('PATCH /api/v1/admin/config', () => { .expect(200); expect(response.body.data.title).toBeNull(); - expect(response.body.meta.type).toEqual('Config'); + expect(response.body.meta.type).toStrictEqual('Config'); }); }); diff --git a/packages/backend/src/controllers/api/v1/admin/permissions/get-permissions-catalog.ee.test.js b/packages/backend/src/controllers/api/v1/admin/permissions/get-permissions-catalog.ee.test.js index cac504fc..f491ba74 100644 --- a/packages/backend/src/controllers/api/v1/admin/permissions/get-permissions-catalog.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/permissions/get-permissions-catalog.ee.test.js @@ -27,6 +27,6 @@ describe('GET /api/v1/admin/permissions/catalog', () => { const expectedPayload = await getPermissionsCatalogMock(); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/admin/roles/create-role.ee.test.js b/packages/backend/src/controllers/api/v1/admin/roles/create-role.ee.test.js index 6dcd5bcf..2d10fc82 100644 --- a/packages/backend/src/controllers/api/v1/admin/roles/create-role.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/roles/create-role.ee.test.js @@ -58,7 +58,7 @@ describe('POST /api/v1/admin/roles', () => { ] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return unprocessable entity response for invalid role data', async () => { diff --git a/packages/backend/src/controllers/api/v1/admin/roles/get-role.ee.test.js b/packages/backend/src/controllers/api/v1/admin/roles/get-role.ee.test.js index 15c688f7..1a51fb3c 100644 --- a/packages/backend/src/controllers/api/v1/admin/roles/get-role.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/roles/get-role.ee.test.js @@ -34,7 +34,7 @@ describe('GET /api/v1/admin/roles/:roleId', () => { permissionTwo, ]); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for not existing role UUID', async () => { diff --git a/packages/backend/src/controllers/api/v1/admin/roles/get-roles.ee.test.js b/packages/backend/src/controllers/api/v1/admin/roles/get-roles.ee.test.js index e07a22ea..facee6d8 100644 --- a/packages/backend/src/controllers/api/v1/admin/roles/get-roles.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/roles/get-roles.ee.test.js @@ -28,6 +28,6 @@ describe('GET /api/v1/admin/roles', () => { const expectedPayload = await getRolesMock([roleOne, roleTwo]); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/admin/saml-auth-providers/get-role-mappings.ee.test.js b/packages/backend/src/controllers/api/v1/admin/saml-auth-providers/get-role-mappings.ee.test.js index e9c54912..fdac3488 100644 --- a/packages/backend/src/controllers/api/v1/admin/saml-auth-providers/get-role-mappings.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/saml-auth-providers/get-role-mappings.ee.test.js @@ -46,6 +46,6 @@ describe('GET /api/v1/admin/saml-auth-providers/:samlAuthProviderId/role-mapping roleMappingTwo, ]); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/admin/saml-auth-providers/get-saml-auth-provider.ee.test.js b/packages/backend/src/controllers/api/v1/admin/saml-auth-providers/get-saml-auth-provider.ee.test.js index de12c036..474dc0b1 100644 --- a/packages/backend/src/controllers/api/v1/admin/saml-auth-providers/get-saml-auth-provider.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/saml-auth-providers/get-saml-auth-provider.ee.test.js @@ -30,7 +30,7 @@ describe('GET /api/v1/admin/saml-auth-provider/:samlAuthProviderId', () => { const expectedPayload = await getSamlAuthProviderMock(samlAuthProvider); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for not existing saml auth provider UUID', async () => { diff --git a/packages/backend/src/controllers/api/v1/admin/saml-auth-providers/get-saml-auth-providers.ee.test.js b/packages/backend/src/controllers/api/v1/admin/saml-auth-providers/get-saml-auth-providers.ee.test.js index 0818f4da..7965d4ac 100644 --- a/packages/backend/src/controllers/api/v1/admin/saml-auth-providers/get-saml-auth-providers.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/saml-auth-providers/get-saml-auth-providers.ee.test.js @@ -34,6 +34,6 @@ describe('GET /api/v1/admin/saml-auth-providers', () => { samlAuthProviderOne, ]); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/admin/users/get-user.ee.test.js b/packages/backend/src/controllers/api/v1/admin/users/get-user.ee.test.js index 2ed6e9e8..2724ea9a 100644 --- a/packages/backend/src/controllers/api/v1/admin/users/get-user.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/users/get-user.ee.test.js @@ -30,7 +30,7 @@ describe('GET /api/v1/admin/users/:userId', () => { .expect(200); const expectedPayload = getUserMock(anotherUser, anotherUserRole); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for not existing user UUID', async () => { diff --git a/packages/backend/src/controllers/api/v1/admin/users/get-users.ee.test.js b/packages/backend/src/controllers/api/v1/admin/users/get-users.ee.test.js index f3f0bcb7..7b3ea362 100644 --- a/packages/backend/src/controllers/api/v1/admin/users/get-users.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/users/get-users.ee.test.js @@ -40,6 +40,6 @@ describe('GET /api/v1/admin/users', () => { [anotherUserRole, currentUserRole] ); - expect(response.body).toEqual(expectedResponsePayload); + expect(response.body).toStrictEqual(expectedResponsePayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/admin/users/update-user.ee.test.js b/packages/backend/src/controllers/api/v1/admin/users/update-user.ee.test.js index a5f4d26b..8700dd13 100644 --- a/packages/backend/src/controllers/api/v1/admin/users/update-user.ee.test.js +++ b/packages/backend/src/controllers/api/v1/admin/users/update-user.ee.test.js @@ -61,7 +61,8 @@ describe('PATCH /api/v1/admin/users/:userId', () => { .send(anotherUserUpdatedData) .expect(422); - expect(response.body.meta.type).toEqual('ModelValidation'); + expect(response.body.meta.type).toStrictEqual('ModelValidation'); + expect(response.body.errors).toMatchObject({ email: ['must be string'], fullName: ['must be string'], diff --git a/packages/backend/src/controllers/api/v1/apps/get-action-substeps.test.js b/packages/backend/src/controllers/api/v1/apps/get-action-substeps.test.js index cc4c01a3..bc28ae33 100644 --- a/packages/backend/src/controllers/api/v1/apps/get-action-substeps.test.js +++ b/packages/backend/src/controllers/api/v1/apps/get-action-substeps.test.js @@ -29,7 +29,7 @@ describe('GET /api/v1/apps/:appKey/actions/:actionKey/substeps', () => { .expect(200); const expectedPayload = getActionSubstepsMock(exampleAction.substeps); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for invalid app key', async () => { @@ -47,6 +47,6 @@ describe('GET /api/v1/apps/:appKey/actions/:actionKey/substeps', () => { .set('Authorization', token) .expect(200); - expect(response.body.data).toEqual([]); + expect(response.body.data).toStrictEqual([]); }); }); diff --git a/packages/backend/src/controllers/api/v1/apps/get-actions.test.js b/packages/backend/src/controllers/api/v1/apps/get-actions.test.js index 52219147..b2bf638f 100644 --- a/packages/backend/src/controllers/api/v1/apps/get-actions.test.js +++ b/packages/backend/src/controllers/api/v1/apps/get-actions.test.js @@ -23,7 +23,7 @@ describe('GET /api/v1/apps/:appKey/actions', () => { .expect(200); const expectedPayload = getActionsMock(exampleApp.actions); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for invalid app key', async () => { diff --git a/packages/backend/src/controllers/api/v1/apps/get-app.test.js b/packages/backend/src/controllers/api/v1/apps/get-app.test.js index 2cc0d839..321064d5 100644 --- a/packages/backend/src/controllers/api/v1/apps/get-app.test.js +++ b/packages/backend/src/controllers/api/v1/apps/get-app.test.js @@ -23,7 +23,7 @@ describe('GET /api/v1/apps/:appKey', () => { .expect(200); const expectedPayload = getAppMock(exampleApp); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for invalid app key', async () => { diff --git a/packages/backend/src/controllers/api/v1/apps/get-apps.test.js b/packages/backend/src/controllers/api/v1/apps/get-apps.test.js index 87f1702a..5524410f 100644 --- a/packages/backend/src/controllers/api/v1/apps/get-apps.test.js +++ b/packages/backend/src/controllers/api/v1/apps/get-apps.test.js @@ -22,7 +22,7 @@ describe('GET /api/v1/apps', () => { .expect(200); const expectedPayload = getAppsMock(apps); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return all apps filtered by name', async () => { @@ -34,7 +34,7 @@ describe('GET /api/v1/apps', () => { .expect(200); const expectedPayload = getAppsMock(appsWithNameGit); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return only the apps with triggers', async () => { @@ -46,7 +46,7 @@ describe('GET /api/v1/apps', () => { .expect(200); const expectedPayload = getAppsMock(appsWithTriggers); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return only the apps with actions', async () => { @@ -58,6 +58,6 @@ describe('GET /api/v1/apps', () => { .expect(200); const expectedPayload = getAppsMock(appsWithActions); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/apps/get-auth-client.ee.test.js b/packages/backend/src/controllers/api/v1/apps/get-auth-client.ee.test.js index 050432ff..d5bea452 100644 --- a/packages/backend/src/controllers/api/v1/apps/get-auth-client.ee.test.js +++ b/packages/backend/src/controllers/api/v1/apps/get-auth-client.ee.test.js @@ -29,7 +29,7 @@ describe('GET /api/v1/apps/:appKey/auth-clients/:appAuthClientId', () => { .expect(200); const expectedPayload = getAppAuthClientMock(currentAppAuthClient); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for not existing app auth client ID', async () => { diff --git a/packages/backend/src/controllers/api/v1/apps/get-auth-clients.ee.test.js b/packages/backend/src/controllers/api/v1/apps/get-auth-clients.ee.test.js index 8bc479d9..d84bf167 100644 --- a/packages/backend/src/controllers/api/v1/apps/get-auth-clients.ee.test.js +++ b/packages/backend/src/controllers/api/v1/apps/get-auth-clients.ee.test.js @@ -37,6 +37,6 @@ describe('GET /api/v1/apps/:appKey/auth-clients', () => { appAuthClientOne, ]); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/apps/get-auth.test.js b/packages/backend/src/controllers/api/v1/apps/get-auth.test.js index a6405f70..6a9a8088 100644 --- a/packages/backend/src/controllers/api/v1/apps/get-auth.test.js +++ b/packages/backend/src/controllers/api/v1/apps/get-auth.test.js @@ -23,7 +23,7 @@ describe('GET /api/v1/apps/:appKey/auth', () => { .expect(200); const expectedPayload = getAuthMock(exampleApp.auth); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for invalid app key', async () => { diff --git a/packages/backend/src/controllers/api/v1/apps/get-config.ee.test.js b/packages/backend/src/controllers/api/v1/apps/get-config.ee.test.js index a39b315b..75c70b25 100644 --- a/packages/backend/src/controllers/api/v1/apps/get-config.ee.test.js +++ b/packages/backend/src/controllers/api/v1/apps/get-config.ee.test.js @@ -32,7 +32,7 @@ describe('GET /api/v1/apps/:appKey/config', () => { .expect(200); const expectedPayload = getAppConfigMock(appConfig); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for not existing app key', async () => { diff --git a/packages/backend/src/controllers/api/v1/apps/get-connections.test.js b/packages/backend/src/controllers/api/v1/apps/get-connections.test.js index c95cc467..8afb7eef 100644 --- a/packages/backend/src/controllers/api/v1/apps/get-connections.test.js +++ b/packages/backend/src/controllers/api/v1/apps/get-connections.test.js @@ -47,7 +47,7 @@ describe('GET /api/v1/apps/:appKey/connections', () => { currentUserConnectionOne, ]); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return the connections data of specified app for another user', async () => { @@ -82,7 +82,7 @@ describe('GET /api/v1/apps/:appKey/connections', () => { anotherUserConnectionOne, ]); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for invalid connection UUID', async () => { diff --git a/packages/backend/src/controllers/api/v1/apps/get-flows.test.js b/packages/backend/src/controllers/api/v1/apps/get-flows.test.js index b53d70ca..7f4ce775 100644 --- a/packages/backend/src/controllers/api/v1/apps/get-flows.test.js +++ b/packages/backend/src/controllers/api/v1/apps/get-flows.test.js @@ -62,7 +62,7 @@ describe('GET /api/v1/apps/:appKey/flows', () => { [triggerStepFlowOne, actionStepFlowOne] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return the flows data of specified app for another user', async () => { @@ -110,7 +110,7 @@ describe('GET /api/v1/apps/:appKey/flows', () => { [triggerStepFlowOne, actionStepFlowOne] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for invalid app key', async () => { diff --git a/packages/backend/src/controllers/api/v1/apps/get-trigger-substeps.test.js b/packages/backend/src/controllers/api/v1/apps/get-trigger-substeps.test.js index 840cb85e..0748ee5a 100644 --- a/packages/backend/src/controllers/api/v1/apps/get-trigger-substeps.test.js +++ b/packages/backend/src/controllers/api/v1/apps/get-trigger-substeps.test.js @@ -29,7 +29,7 @@ describe('GET /api/v1/apps/:appKey/triggers/:triggerKey/substeps', () => { .expect(200); const expectedPayload = getTriggerSubstepsMock(exampleTrigger.substeps); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for invalid app key', async () => { @@ -47,6 +47,6 @@ describe('GET /api/v1/apps/:appKey/triggers/:triggerKey/substeps', () => { .set('Authorization', token) .expect(200); - expect(response.body.data).toEqual([]); + expect(response.body.data).toStrictEqual([]); }); }); diff --git a/packages/backend/src/controllers/api/v1/apps/get-triggers.test.js b/packages/backend/src/controllers/api/v1/apps/get-triggers.test.js index ff825a71..768a4d52 100644 --- a/packages/backend/src/controllers/api/v1/apps/get-triggers.test.js +++ b/packages/backend/src/controllers/api/v1/apps/get-triggers.test.js @@ -23,7 +23,7 @@ describe('GET /api/v1/apps/:appKey/triggers', () => { .expect(200); const expectedPayload = getTriggersMock(exampleApp.triggers); - expect(response.body).toEqual(expectedPayload); + expect(expectedPayload).toMatchObject(response.body); }); it('should return not found response for invalid app key', async () => { diff --git a/packages/backend/src/controllers/api/v1/automatisch/info.test.js b/packages/backend/src/controllers/api/v1/automatisch/info.test.js index b48a7283..431a3d91 100644 --- a/packages/backend/src/controllers/api/v1/automatisch/info.test.js +++ b/packages/backend/src/controllers/api/v1/automatisch/info.test.js @@ -20,6 +20,6 @@ describe('GET /api/v1/automatisch/info', () => { const expectedPayload = infoMock(); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/automatisch/license.test.js b/packages/backend/src/controllers/api/v1/automatisch/license.test.js index 211ef630..a35567c1 100644 --- a/packages/backend/src/controllers/api/v1/automatisch/license.test.js +++ b/packages/backend/src/controllers/api/v1/automatisch/license.test.js @@ -18,6 +18,6 @@ describe('GET /api/v1/automatisch/license', () => { const expectedPayload = licenseMock(); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/automatisch/version.test.js b/packages/backend/src/controllers/api/v1/automatisch/version.test.js index 8f08e32a..ae99e507 100644 --- a/packages/backend/src/controllers/api/v1/automatisch/version.test.js +++ b/packages/backend/src/controllers/api/v1/automatisch/version.test.js @@ -21,6 +21,6 @@ describe('GET /api/v1/automatisch/version', () => { }, }; - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/connections/get-flows.test.js b/packages/backend/src/controllers/api/v1/connections/get-flows.test.js index bc9088d6..1f596720 100644 --- a/packages/backend/src/controllers/api/v1/connections/get-flows.test.js +++ b/packages/backend/src/controllers/api/v1/connections/get-flows.test.js @@ -69,7 +69,7 @@ describe('GET /api/v1/connections/:connectionId/flows', () => { [triggerStepFlowOne, actionStepFlowOne] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return the flows data of specified connection for another user', async () => { @@ -123,6 +123,6 @@ describe('GET /api/v1/connections/:connectionId/flows', () => { [triggerStepFlowOne, actionStepFlowOne] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/connections/test-connection.test.js b/packages/backend/src/controllers/api/v1/connections/test-connection.test.js index 152b404a..8d11f907 100644 --- a/packages/backend/src/controllers/api/v1/connections/test-connection.test.js +++ b/packages/backend/src/controllers/api/v1/connections/test-connection.test.js @@ -43,7 +43,7 @@ describe('POST /api/v1/connections/:connectionId/test', () => { .set('Authorization', token) .expect(200); - expect(response.body.data.verified).toEqual(false); + expect(response.body.data.verified).toStrictEqual(false); }); it('should update the connection as not verified for another user', async () => { @@ -74,7 +74,7 @@ describe('POST /api/v1/connections/:connectionId/test', () => { .set('Authorization', token) .expect(200); - expect(response.body.data.verified).toEqual(false); + expect(response.body.data.verified).toStrictEqual(false); }); it('should return not found response for not existing connection UUID', async () => { diff --git a/packages/backend/src/controllers/api/v1/connections/verify-connection.test.js b/packages/backend/src/controllers/api/v1/connections/verify-connection.test.js index 6187d2db..4fd6f974 100644 --- a/packages/backend/src/controllers/api/v1/connections/verify-connection.test.js +++ b/packages/backend/src/controllers/api/v1/connections/verify-connection.test.js @@ -47,7 +47,7 @@ describe('POST /api/v1/connections/:connectionId/verify', () => { .set('Authorization', token) .expect(200); - expect(response.body.data.verified).toEqual(true); + expect(response.body.data.verified).toStrictEqual(true); }); it('should return not found response for not existing connection UUID', async () => { diff --git a/packages/backend/src/controllers/api/v1/executions/get-execution-steps.test.js b/packages/backend/src/controllers/api/v1/executions/get-execution-steps.test.js index d93fbb11..58a326ae 100644 --- a/packages/backend/src/controllers/api/v1/executions/get-execution-steps.test.js +++ b/packages/backend/src/controllers/api/v1/executions/get-execution-steps.test.js @@ -69,7 +69,7 @@ describe('GET /api/v1/executions/:executionId/execution-steps', () => { [stepOne, stepTwo] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return the execution steps of another user execution', async () => { @@ -118,7 +118,7 @@ describe('GET /api/v1/executions/:executionId/execution-steps', () => { [stepOne, stepTwo] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for not existing execution step UUID', async () => { diff --git a/packages/backend/src/controllers/api/v1/executions/get-execution.test.js b/packages/backend/src/controllers/api/v1/executions/get-execution.test.js index a6a86ce9..b447882a 100644 --- a/packages/backend/src/controllers/api/v1/executions/get-execution.test.js +++ b/packages/backend/src/controllers/api/v1/executions/get-execution.test.js @@ -57,7 +57,7 @@ describe('GET /api/v1/executions/:executionId', () => { [stepOne, stepTwo] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return the execution data of another user', async () => { @@ -99,7 +99,7 @@ describe('GET /api/v1/executions/:executionId', () => { [stepOne, stepTwo] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for not existing execution UUID', async () => { diff --git a/packages/backend/src/controllers/api/v1/executions/get-executions.test.js b/packages/backend/src/controllers/api/v1/executions/get-executions.test.js index 49f29c9f..ebf9b0c6 100644 --- a/packages/backend/src/controllers/api/v1/executions/get-executions.test.js +++ b/packages/backend/src/controllers/api/v1/executions/get-executions.test.js @@ -66,7 +66,7 @@ describe('GET /api/v1/executions', () => { [stepOne, stepTwo] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return the executions of another user', async () => { @@ -114,6 +114,6 @@ describe('GET /api/v1/executions', () => { [stepOne, stepTwo] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/flows/get-flow.test.js b/packages/backend/src/controllers/api/v1/flows/get-flow.test.js index cd150f5a..19357ce7 100644 --- a/packages/backend/src/controllers/api/v1/flows/get-flow.test.js +++ b/packages/backend/src/controllers/api/v1/flows/get-flow.test.js @@ -41,7 +41,7 @@ describe('GET /api/v1/flows/:flowId', () => { actionStep, ]); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return the flow data of another user', async () => { @@ -67,7 +67,7 @@ describe('GET /api/v1/flows/:flowId', () => { actionStep, ]); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for not existing flow UUID', async () => { diff --git a/packages/backend/src/controllers/api/v1/flows/get-flows.test.js b/packages/backend/src/controllers/api/v1/flows/get-flows.test.js index 3df8a2bb..3af8537d 100644 --- a/packages/backend/src/controllers/api/v1/flows/get-flows.test.js +++ b/packages/backend/src/controllers/api/v1/flows/get-flows.test.js @@ -63,7 +63,7 @@ describe('GET /api/v1/flows', () => { ] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return the flows data of another user', async () => { @@ -113,6 +113,6 @@ describe('GET /api/v1/flows', () => { ] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/installation/users/create-user.test.js b/packages/backend/src/controllers/api/v1/installation/users/create-user.test.js index a94f4c61..428437d7 100644 --- a/packages/backend/src/controllers/api/v1/installation/users/create-user.test.js +++ b/packages/backend/src/controllers/api/v1/installation/users/create-user.test.js @@ -53,7 +53,7 @@ describe('POST /api/v1/installation/users', () => { const usersCountAfter = await User.query().resultSize(); - expect(usersCountBefore).toEqual(usersCountAfter); + expect(usersCountBefore).toStrictEqual(usersCountAfter); }); }); diff --git a/packages/backend/src/controllers/api/v1/payment/get-paddle-info.ee.test.js b/packages/backend/src/controllers/api/v1/payment/get-paddle-info.ee.test.js index 7fccd60f..4e81f4d4 100644 --- a/packages/backend/src/controllers/api/v1/payment/get-paddle-info.ee.test.js +++ b/packages/backend/src/controllers/api/v1/payment/get-paddle-info.ee.test.js @@ -28,6 +28,6 @@ describe('GET /api/v1/payment/paddle-info', () => { const expectedResponsePayload = await getPaddleInfoMock(); - expect(response.body).toEqual(expectedResponsePayload); + expect(response.body).toStrictEqual(expectedResponsePayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/payment/get-plans.ee.test.js b/packages/backend/src/controllers/api/v1/payment/get-plans.ee.test.js index 4f7665aa..7d953cd4 100644 --- a/packages/backend/src/controllers/api/v1/payment/get-plans.ee.test.js +++ b/packages/backend/src/controllers/api/v1/payment/get-plans.ee.test.js @@ -24,6 +24,6 @@ describe('GET /api/v1/payment/plans', () => { const expectedResponsePayload = await getPaymentPlansMock(); - expect(response.body).toEqual(expectedResponsePayload); + expect(response.body).toStrictEqual(expectedResponsePayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/saml-auth-providers/get-saml-auth-providers.ee.test.js b/packages/backend/src/controllers/api/v1/saml-auth-providers/get-saml-auth-providers.ee.test.js index f2f21b7d..faed4f0b 100644 --- a/packages/backend/src/controllers/api/v1/saml-auth-providers/get-saml-auth-providers.ee.test.js +++ b/packages/backend/src/controllers/api/v1/saml-auth-providers/get-saml-auth-providers.ee.test.js @@ -25,6 +25,6 @@ describe('GET /api/v1/saml-auth-providers', () => { samlAuthProviderOne, ]); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/steps/create-dynamic-data.test.js b/packages/backend/src/controllers/api/v1/steps/create-dynamic-data.test.js index b8028ccb..64b1c254 100644 --- a/packages/backend/src/controllers/api/v1/steps/create-dynamic-data.test.js +++ b/packages/backend/src/controllers/api/v1/steps/create-dynamic-data.test.js @@ -78,7 +78,7 @@ describe('POST /api/v1/steps/:stepId/dynamic-data', () => { }) .expect(200); - expect(response.body.data).toEqual(repositories); + expect(response.body.data).toStrictEqual(repositories); }); it('of the another users step', async () => { @@ -117,7 +117,7 @@ describe('POST /api/v1/steps/:stepId/dynamic-data', () => { }) .expect(200); - expect(response.body.data).toEqual(repositories); + expect(response.body.data).toStrictEqual(repositories); }); }); @@ -171,7 +171,7 @@ describe('POST /api/v1/steps/:stepId/dynamic-data', () => { }) .expect(422); - expect(response.body.errors).toEqual(errors); + expect(response.body.errors).toStrictEqual(errors); }); }); diff --git a/packages/backend/src/controllers/api/v1/steps/create-dynamic-fields.test.js b/packages/backend/src/controllers/api/v1/steps/create-dynamic-fields.test.js index 7ae163b5..3081351d 100644 --- a/packages/backend/src/controllers/api/v1/steps/create-dynamic-fields.test.js +++ b/packages/backend/src/controllers/api/v1/steps/create-dynamic-fields.test.js @@ -56,7 +56,7 @@ describe('POST /api/v1/steps/:stepId/dynamic-fields', () => { const expectedPayload = await createDynamicFieldsMock(); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return dynamically created fields of the another users step', async () => { @@ -97,7 +97,7 @@ describe('POST /api/v1/steps/:stepId/dynamic-fields', () => { const expectedPayload = await createDynamicFieldsMock(); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for not existing step UUID', async () => { diff --git a/packages/backend/src/controllers/api/v1/steps/get-connection.test.js b/packages/backend/src/controllers/api/v1/steps/get-connection.test.js index 27e6151e..71a49ce7 100644 --- a/packages/backend/src/controllers/api/v1/steps/get-connection.test.js +++ b/packages/backend/src/controllers/api/v1/steps/get-connection.test.js @@ -43,7 +43,7 @@ describe('GET /api/v1/steps/:stepId/connection', () => { const expectedPayload = await getConnectionMock(currentUserConnection); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return the current user connection data of specified step', async () => { @@ -70,7 +70,7 @@ describe('GET /api/v1/steps/:stepId/connection', () => { const expectedPayload = await getConnectionMock(anotherUserConnection); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for not existing step without connection', async () => { diff --git a/packages/backend/src/controllers/api/v1/steps/get-previous-steps.test.js b/packages/backend/src/controllers/api/v1/steps/get-previous-steps.test.js index f9205295..b40446e3 100644 --- a/packages/backend/src/controllers/api/v1/steps/get-previous-steps.test.js +++ b/packages/backend/src/controllers/api/v1/steps/get-previous-steps.test.js @@ -70,7 +70,7 @@ describe('GET /api/v1/steps/:stepId/previous-steps', () => { [executionStepOne, executionStepTwo] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return the previous steps of the specified step of another user', async () => { @@ -124,7 +124,7 @@ describe('GET /api/v1/steps/:stepId/previous-steps', () => { [executionStepOne, executionStepTwo] ); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response for not existing step UUID', async () => { diff --git a/packages/backend/src/controllers/api/v1/users/get-apps.test.js b/packages/backend/src/controllers/api/v1/users/get-apps.test.js index eb24b66a..c3c7ca7d 100644 --- a/packages/backend/src/controllers/api/v1/users/get-apps.test.js +++ b/packages/backend/src/controllers/api/v1/users/get-apps.test.js @@ -79,7 +79,7 @@ describe('GET /api/v1/users/:userId/apps', () => { .expect(200); const expectedPayload = getAppsMock(); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return all apps of the another user', async () => { @@ -143,7 +143,7 @@ describe('GET /api/v1/users/:userId/apps', () => { .expect(200); const expectedPayload = getAppsMock(); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return specified app of the current user', async () => { @@ -204,7 +204,7 @@ describe('GET /api/v1/users/:userId/apps', () => { .set('Authorization', token) .expect(200); - expect(response.body.data.length).toEqual(1); - expect(response.body.data[0].key).toEqual('deepl'); + expect(response.body.data.length).toStrictEqual(1); + expect(response.body.data[0].key).toStrictEqual('deepl'); }); }); diff --git a/packages/backend/src/controllers/api/v1/users/get-current-user.test.js b/packages/backend/src/controllers/api/v1/users/get-current-user.test.js index 362c9815..e8f29f2b 100644 --- a/packages/backend/src/controllers/api/v1/users/get-current-user.test.js +++ b/packages/backend/src/controllers/api/v1/users/get-current-user.test.js @@ -39,6 +39,6 @@ describe('GET /api/v1/users/me', () => { permissionTwo, ]); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/users/get-invoices.ee.test.js b/packages/backend/src/controllers/api/v1/users/get-invoices.ee.test.js index 1b0e8732..de354530 100644 --- a/packages/backend/src/controllers/api/v1/users/get-invoices.ee.test.js +++ b/packages/backend/src/controllers/api/v1/users/get-invoices.ee.test.js @@ -29,6 +29,6 @@ describe('GET /api/v1/user/invoices', () => { const expectedPayload = await getInvoicesMock(invoices); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/controllers/api/v1/users/get-plan-and-usage.ee.test.js b/packages/backend/src/controllers/api/v1/users/get-plan-and-usage.ee.test.js index 4986ad2b..a08f3e00 100644 --- a/packages/backend/src/controllers/api/v1/users/get-plan-and-usage.ee.test.js +++ b/packages/backend/src/controllers/api/v1/users/get-plan-and-usage.ee.test.js @@ -36,7 +36,7 @@ describe('GET /api/v1/users/:userId/plan-and-usage', () => { }, }; - expect(response.body.data).toEqual(expectedResponseData); + expect(response.body.data).toStrictEqual(expectedResponseData); }); it('should return current plan and usage data', async () => { @@ -63,6 +63,6 @@ describe('GET /api/v1/users/:userId/plan-and-usage', () => { }, }; - expect(response.body.data).toEqual(expectedResponseData); + expect(response.body.data).toStrictEqual(expectedResponseData); }); }); diff --git a/packages/backend/src/controllers/api/v1/users/get-subscription.ee.test.js b/packages/backend/src/controllers/api/v1/users/get-subscription.ee.test.js index 24c6db47..ac9ace6a 100644 --- a/packages/backend/src/controllers/api/v1/users/get-subscription.ee.test.js +++ b/packages/backend/src/controllers/api/v1/users/get-subscription.ee.test.js @@ -33,7 +33,7 @@ describe('GET /api/v1/users/:userId/subscription', () => { const expectedPayload = getSubscriptionMock(subscription); - expect(response.body).toEqual(expectedPayload); + expect(response.body).toStrictEqual(expectedPayload); }); it('should return not found response if there is no current subscription', async () => { diff --git a/packages/backend/src/controllers/api/v1/users/get-user-trial.ee.test.js b/packages/backend/src/controllers/api/v1/users/get-user-trial.ee.test.js index f560554b..7f6f7e96 100644 --- a/packages/backend/src/controllers/api/v1/users/get-user-trial.ee.test.js +++ b/packages/backend/src/controllers/api/v1/users/get-user-trial.ee.test.js @@ -32,7 +32,7 @@ describe('GET /api/v1/users/:userId/trial', () => { .expect(200); const expectedResponsePayload = await getUserTrialMock(user); - expect(response.body).toEqual(expectedResponsePayload); + expect(response.body).toStrictEqual(expectedResponsePayload); }); }); }); diff --git a/packages/backend/src/controllers/api/v1/users/update-current-user-password.test.js b/packages/backend/src/controllers/api/v1/users/update-current-user-password.test.js index 55c5c70a..ddb4a0da 100644 --- a/packages/backend/src/controllers/api/v1/users/update-current-user-password.test.js +++ b/packages/backend/src/controllers/api/v1/users/update-current-user-password.test.js @@ -43,7 +43,7 @@ describe('PATCH /api/v1/users/:userId/password', () => { .send(userData) .expect(422); - expect(response.body.meta.type).toEqual('ValidationError'); + expect(response.body.meta.type).toStrictEqual('ValidationError'); expect(response.body.errors).toMatchObject({ currentPassword: ['is incorrect.'], }); diff --git a/packages/backend/src/controllers/api/v1/users/update-current-user.test.js b/packages/backend/src/controllers/api/v1/users/update-current-user.test.js index cf45e89b..38a56a8a 100644 --- a/packages/backend/src/controllers/api/v1/users/update-current-user.test.js +++ b/packages/backend/src/controllers/api/v1/users/update-current-user.test.js @@ -47,7 +47,8 @@ describe('PATCH /api/v1/users/:userId', () => { .send(userData) .expect(422); - expect(response.body.meta.type).toEqual('ModelValidation'); + expect(response.body.meta.type).toStrictEqual('ModelValidation'); + expect(response.body.errors).toMatchObject({ email: ['must be string'], fullName: ['must be string'], diff --git a/packages/backend/src/models/app-auth-client.test.js b/packages/backend/src/models/app-auth-client.test.js index 94b5c9f8..af1fefc2 100644 --- a/packages/backend/src/models/app-auth-client.test.js +++ b/packages/backend/src/models/app-auth-client.test.js @@ -69,7 +69,9 @@ describe('AppAuthClient model', () => { ); expect(formattedAuthDefaults).toStrictEqual(expectedDecryptedValue); - expect(appAuthClient.authDefaults).not.toEqual(formattedAuthDefaults); + expect(appAuthClient.authDefaults).not.toStrictEqual( + formattedAuthDefaults + ); }); it('should encrypt formattedAuthDefaults and remove formattedAuthDefaults', async () => { @@ -124,7 +126,9 @@ describe('AppAuthClient model', () => { expect(appAuthClient.formattedAuthDefaults).toStrictEqual( formattedAuthDefaults ); - expect(appAuthClient.authDefaults).not.toEqual(formattedAuthDefaults); + expect(appAuthClient.authDefaults).not.toStrictEqual( + formattedAuthDefaults + ); }); }); diff --git a/packages/backend/src/models/connection.test.js b/packages/backend/src/models/connection.test.js index 9818a3ec..7c5057bb 100644 --- a/packages/backend/src/models/connection.test.js +++ b/packages/backend/src/models/connection.test.js @@ -193,7 +193,7 @@ describe('Connection model', () => { ); expect(formattedData).toStrictEqual(expectedDecryptedValue); - expect(connection.data).not.toEqual(formattedData); + expect(connection.data).not.toStrictEqual(formattedData); }); it('should encrypt formattedData and remove formattedData', async () => { @@ -243,7 +243,7 @@ describe('Connection model', () => { connection.decryptData(); expect(connection.formattedData).toStrictEqual(formattedData); - expect(connection.data).not.toEqual(formattedData); + expect(connection.data).not.toStrictEqual(formattedData); }); }); diff --git a/packages/backend/src/serializers/action.test.js b/packages/backend/src/serializers/action.test.js index 32b6b81b..ffd792be 100644 --- a/packages/backend/src/serializers/action.test.js +++ b/packages/backend/src/serializers/action.test.js @@ -16,6 +16,6 @@ describe('actionSerializer', () => { type: action.type, }; - expect(actionSerializer(action)).toEqual(expectedPayload); + expect(expectedPayload).toMatchObject(actionSerializer(action)); }); }); diff --git a/packages/backend/src/serializers/admin-saml-auth-provider.ee.test.js b/packages/backend/src/serializers/admin-saml-auth-provider.ee.test.js index 0ad55c81..21fd0e04 100644 --- a/packages/backend/src/serializers/admin-saml-auth-provider.ee.test.js +++ b/packages/backend/src/serializers/admin-saml-auth-provider.ee.test.js @@ -25,7 +25,7 @@ describe('adminSamlAuthProviderSerializer', () => { defaultRoleId: samlAuthProvider.defaultRoleId, }; - expect(adminSamlAuthProviderSerializer(samlAuthProvider)).toEqual( + expect(adminSamlAuthProviderSerializer(samlAuthProvider)).toStrictEqual( expectedPayload ); }); diff --git a/packages/backend/src/serializers/admin/user.test.js b/packages/backend/src/serializers/admin/user.test.js index 4f7528e1..3ef76f6e 100644 --- a/packages/backend/src/serializers/admin/user.test.js +++ b/packages/backend/src/serializers/admin/user.test.js @@ -12,7 +12,7 @@ describe('adminUserSerializer', () => { it('should return user data with accept invitation url', async () => { const serializedUser = adminUserSerializer(user); - expect(serializedUser.acceptInvitationUrl).toEqual( + expect(serializedUser.acceptInvitationUrl).toStrictEqual( user.acceptInvitationUrl ); }); diff --git a/packages/backend/src/serializers/app-auth-client.test.js b/packages/backend/src/serializers/app-auth-client.test.js index 7352f70d..d4ed178e 100644 --- a/packages/backend/src/serializers/app-auth-client.test.js +++ b/packages/backend/src/serializers/app-auth-client.test.js @@ -17,6 +17,8 @@ describe('appAuthClient serializer', () => { active: appAuthClient.active, }; - expect(appAuthClientSerializer(appAuthClient)).toEqual(expectedPayload); + expect(appAuthClientSerializer(appAuthClient)).toStrictEqual( + expectedPayload + ); }); }); diff --git a/packages/backend/src/serializers/app.test.js b/packages/backend/src/serializers/app.test.js index c2714dc2..ec5716a9 100644 --- a/packages/backend/src/serializers/app.test.js +++ b/packages/backend/src/serializers/app.test.js @@ -15,6 +15,6 @@ describe('appSerializer', () => { primaryColor: app.primaryColor, }; - expect(appSerializer(app)).toEqual(expectedPayload); + expect(appSerializer(app)).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/serializers/auth.test.js b/packages/backend/src/serializers/auth.test.js index 6746c559..e9adb259 100644 --- a/packages/backend/src/serializers/auth.test.js +++ b/packages/backend/src/serializers/auth.test.js @@ -12,6 +12,6 @@ describe('authSerializer', () => { reconnectionSteps: auth.reconnectionSteps, }; - expect(authSerializer(auth)).toEqual(expectedPayload); + expect(authSerializer(auth)).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/serializers/config.test.js b/packages/backend/src/serializers/config.test.js index d4984725..d3339bf2 100644 --- a/packages/backend/src/serializers/config.test.js +++ b/packages/backend/src/serializers/config.test.js @@ -27,6 +27,6 @@ describe('configSerializer', () => { updatedAt: config.updatedAt.getTime(), }; - expect(configSerializer(config)).toEqual(expectedPayload); + expect(configSerializer(config)).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/serializers/connection.test.js b/packages/backend/src/serializers/connection.test.js index 2a4df3c3..c322af6b 100644 --- a/packages/backend/src/serializers/connection.test.js +++ b/packages/backend/src/serializers/connection.test.js @@ -23,6 +23,6 @@ describe('connectionSerializer', () => { updatedAt: connection.updatedAt.getTime(), }; - expect(connectionSerializer(connection)).toEqual(expectedPayload); + expect(connectionSerializer(connection)).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/serializers/execution-step.test.js b/packages/backend/src/serializers/execution-step.test.js index 037ccbcd..6c22131c 100644 --- a/packages/backend/src/serializers/execution-step.test.js +++ b/packages/backend/src/serializers/execution-step.test.js @@ -26,7 +26,9 @@ describe('executionStepSerializer', () => { updatedAt: executionStep.updatedAt.getTime(), }; - expect(executionStepSerializer(executionStep)).toEqual(expectedPayload); + expect(executionStepSerializer(executionStep)).toStrictEqual( + expectedPayload + ); }); it('should return the execution step data with the step', async () => { diff --git a/packages/backend/src/serializers/execution.test.js b/packages/backend/src/serializers/execution.test.js index 5711b0ea..0e28e775 100644 --- a/packages/backend/src/serializers/execution.test.js +++ b/packages/backend/src/serializers/execution.test.js @@ -23,7 +23,7 @@ describe('executionSerializer', () => { updatedAt: execution.updatedAt.getTime(), }; - expect(executionSerializer(execution)).toEqual(expectedPayload); + expect(executionSerializer(execution)).toStrictEqual(expectedPayload); }); it('should return the execution data with status', async () => { @@ -37,7 +37,7 @@ describe('executionSerializer', () => { status: 'success', }; - expect(executionSerializer(execution)).toEqual(expectedPayload); + expect(executionSerializer(execution)).toStrictEqual(expectedPayload); }); it('should return the execution data with the flow', async () => { diff --git a/packages/backend/src/serializers/flow.test.js b/packages/backend/src/serializers/flow.test.js index 65dc9399..6c185dd7 100644 --- a/packages/backend/src/serializers/flow.test.js +++ b/packages/backend/src/serializers/flow.test.js @@ -31,7 +31,7 @@ describe('flowSerializer', () => { updatedAt: flow.updatedAt.getTime(), }; - expect(flowSerializer(flow)).toEqual(expectedPayload); + expect(flowSerializer(flow)).toStrictEqual(expectedPayload); }); it('should return flow data with the steps', async () => { diff --git a/packages/backend/src/serializers/permission.test.js b/packages/backend/src/serializers/permission.test.js index 9fff7f61..73c00555 100644 --- a/packages/backend/src/serializers/permission.test.js +++ b/packages/backend/src/serializers/permission.test.js @@ -20,6 +20,6 @@ describe('permissionSerializer', () => { updatedAt: permission.updatedAt.getTime(), }; - expect(permissionSerializer(permission)).toEqual(expectedPayload); + expect(permissionSerializer(permission)).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/serializers/role.test.js b/packages/backend/src/serializers/role.test.js index b2929019..f415c826 100644 --- a/packages/backend/src/serializers/role.test.js +++ b/packages/backend/src/serializers/role.test.js @@ -34,7 +34,7 @@ describe('roleSerializer', () => { isAdmin: role.isAdmin, }; - expect(roleSerializer(role)).toEqual(expectedPayload); + expect(roleSerializer(role)).toStrictEqual(expectedPayload); }); it('should return role data with the permissions', async () => { diff --git a/packages/backend/src/serializers/saml-auth-provider.ee.test.js b/packages/backend/src/serializers/saml-auth-provider.ee.test.js index 9a05c3db..ca1da1a7 100644 --- a/packages/backend/src/serializers/saml-auth-provider.ee.test.js +++ b/packages/backend/src/serializers/saml-auth-provider.ee.test.js @@ -17,7 +17,7 @@ describe('samlAuthProviderSerializer', () => { issuer: samlAuthProvider.issuer, }; - expect(samlAuthProviderSerializer(samlAuthProvider)).toEqual( + expect(samlAuthProviderSerializer(samlAuthProvider)).toStrictEqual( expectedPayload ); }); diff --git a/packages/backend/src/serializers/step.test.js b/packages/backend/src/serializers/step.test.js index 8245196b..dfeffbeb 100644 --- a/packages/backend/src/serializers/step.test.js +++ b/packages/backend/src/serializers/step.test.js @@ -24,7 +24,7 @@ describe('stepSerializer', () => { parameters: step.parameters, }; - expect(stepSerializer(step)).toEqual(expectedPayload); + expect(stepSerializer(step)).toStrictEqual(expectedPayload); }); it('should return step data with the last execution step', async () => { diff --git a/packages/backend/src/serializers/subscription.ee.test.js b/packages/backend/src/serializers/subscription.ee.test.js index d8195932..3441051e 100644 --- a/packages/backend/src/serializers/subscription.ee.test.js +++ b/packages/backend/src/serializers/subscription.ee.test.js @@ -30,6 +30,6 @@ describe('subscriptionSerializer', () => { cancellationEffectiveDate: subscription.cancellationEffectiveDate, }; - expect(subscriptionSerializer(subscription)).toEqual(expectedPayload); + expect(subscriptionSerializer(subscription)).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/serializers/trigger.test.js b/packages/backend/src/serializers/trigger.test.js index 126a6637..531e7977 100644 --- a/packages/backend/src/serializers/trigger.test.js +++ b/packages/backend/src/serializers/trigger.test.js @@ -16,6 +16,6 @@ describe('triggerSerializer', () => { type: trigger.type, }; - expect(triggerSerializer(trigger)).toEqual(expectedPayload); + expect(triggerSerializer(trigger)).toStrictEqual(expectedPayload); }); }); diff --git a/packages/backend/src/serializers/user.test.js b/packages/backend/src/serializers/user.test.js index f084836a..6908abd4 100644 --- a/packages/backend/src/serializers/user.test.js +++ b/packages/backend/src/serializers/user.test.js @@ -39,7 +39,7 @@ describe('userSerializer', () => { updatedAt: user.updatedAt.getTime(), }; - expect(userSerializer(user)).toEqual(expectedPayload); + expect(userSerializer(user)).toStrictEqual(expectedPayload); }); it('should return user data with the role', async () => { diff --git a/packages/backend/test/mocks/rest/api/v1/admin/apps/get-auth-client.js b/packages/backend/test/mocks/rest/api/v1/admin/apps/get-auth-client.js index 3789ffe7..4d437eca 100644 --- a/packages/backend/test/mocks/rest/api/v1/admin/apps/get-auth-client.js +++ b/packages/backend/test/mocks/rest/api/v1/admin/apps/get-auth-client.js @@ -3,7 +3,6 @@ const getAppAuthClientMock = (appAuthClient) => { data: { name: appAuthClient.name, id: appAuthClient.id, - appConfigId: appAuthClient.appConfigId, active: appAuthClient.active, }, meta: { diff --git a/packages/backend/test/mocks/rest/api/v1/admin/users/get-user.js b/packages/backend/test/mocks/rest/api/v1/admin/users/get-user.js index 0ce3dd26..fcf40ece 100644 --- a/packages/backend/test/mocks/rest/api/v1/admin/users/get-user.js +++ b/packages/backend/test/mocks/rest/api/v1/admin/users/get-user.js @@ -10,7 +10,6 @@ const getUserMock = (currentUser, role) => { description: null, id: role.id, isAdmin: role.isAdmin, - key: role.key, name: role.name, updatedAt: role.updatedAt.getTime(), }, diff --git a/packages/backend/test/mocks/rest/api/v1/admin/users/get-users.js b/packages/backend/test/mocks/rest/api/v1/admin/users/get-users.js index e135217b..7daff409 100644 --- a/packages/backend/test/mocks/rest/api/v1/admin/users/get-users.js +++ b/packages/backend/test/mocks/rest/api/v1/admin/users/get-users.js @@ -13,7 +13,6 @@ const getUsersMock = async (users, roles) => { description: role.description, id: role.id, isAdmin: role.isAdmin, - key: role.key, name: role.name, updatedAt: role.updatedAt.getTime(), } diff --git a/packages/backend/test/mocks/rest/api/v1/apps/get-auth-client.js b/packages/backend/test/mocks/rest/api/v1/apps/get-auth-client.js index 3789ffe7..4d437eca 100644 --- a/packages/backend/test/mocks/rest/api/v1/apps/get-auth-client.js +++ b/packages/backend/test/mocks/rest/api/v1/apps/get-auth-client.js @@ -3,7 +3,6 @@ const getAppAuthClientMock = (appAuthClient) => { data: { name: appAuthClient.name, id: appAuthClient.id, - appConfigId: appAuthClient.appConfigId, active: appAuthClient.active, }, meta: { diff --git a/packages/backend/test/mocks/rest/api/v1/apps/get-config.js b/packages/backend/test/mocks/rest/api/v1/apps/get-config.js index 60772644..3cb4ab11 100644 --- a/packages/backend/test/mocks/rest/api/v1/apps/get-config.js +++ b/packages/backend/test/mocks/rest/api/v1/apps/get-config.js @@ -1,7 +1,6 @@ const getAppConfigMock = (appConfig) => { return { data: { - id: appConfig.id, key: appConfig.key, customConnectionAllowed: appConfig.customConnectionAllowed, shared: appConfig.shared, diff --git a/packages/backend/test/mocks/rest/api/v1/users/get-current-user.js b/packages/backend/test/mocks/rest/api/v1/users/get-current-user.js index 2802290d..5fc7ab6a 100644 --- a/packages/backend/test/mocks/rest/api/v1/users/get-current-user.js +++ b/packages/backend/test/mocks/rest/api/v1/users/get-current-user.js @@ -19,7 +19,6 @@ const getCurrentUserMock = (currentUser, role, permissions) => { description: null, id: role.id, isAdmin: role.isAdmin, - key: role.key, name: role.name, updatedAt: role.updatedAt.getTime(), },