refactor: remove additional mock implentation in tests

packages/backend/src/graphql/queries/get-automatisch-info.test.ts

@@ -1,7 +1,7 @@
-import { setMockConfig } from '../../../test/setup/set-mock-config';
 import request from 'supertest';
 import app from '../../app';
 import * as license from '../../helpers/license.ee';
+import appConfig from '../../config/app';
 
 describe('graphQL getAutomatischInfo query', () => {
   const query = `
@@ -21,6 +21,8 @@ describe('graphQL getAutomatischInfo query', () => {
   describe('and without valid license', () => {
     beforeEach(async () => {
       jest.spyOn(license, 'getLicense').mockResolvedValue(false);
+
+      jest.replaceProperty(appConfig, 'isCloud', false)
     });
 
     it('should return empty license data', async () => {
@@ -61,7 +63,7 @@ describe('graphQL getAutomatischInfo query', () => {
 
     describe('and with cloud flag enabled', () => {
       beforeEach(async () => {
-        setMockConfig({ isCloud: true });
+        jest.replaceProperty(appConfig, 'isCloud', true)
       });
 
       it('should return all license data', async () => {
@@ -90,7 +92,7 @@ describe('graphQL getAutomatischInfo query', () => {
 
     describe('and with cloud flag disabled', () => {
       beforeEach(async () => {
-        setMockConfig({ isCloud: false });
+        jest.replaceProperty(appConfig, 'isCloud', false)
       });
 
       it('should return all license data', async () => {

packages/backend/src/graphql/queries/get-trial-status.ee.test.ts

@@ -1,4 +1,3 @@
-import { setMockConfig } from '../../../test/setup/set-mock-config';
 import request from 'supertest';
 import app from '../../app';
 import { IUser } from '@automatisch/types';
@@ -6,6 +5,7 @@ import User from '../../models/user';
 import createUser from '../../../test/fixtures/user';
 import createAuthTokenByUserId from '../../helpers/create-auth-token-by-user-id';
 import { DateTime } from 'luxon';
+import appConfig from '../../config/app';
 
 describe('graphQL getTrialStatus query', () => {
   const query = `
@@ -43,7 +43,7 @@ describe('graphQL getTrialStatus query', () => {
 
     describe('and with cloud flag disabled', () => {
       beforeEach(async () => {
-        setMockConfig({ isCloud: false });
+        jest.replaceProperty(appConfig, 'isCloud', false);
       });
 
       it('should return null', async () => {
@@ -63,7 +63,7 @@ describe('graphQL getTrialStatus query', () => {
 
     describe('and with cloud flag enabled', () => {
       beforeEach(async () => {
-        setMockConfig({ isCloud: true });
+        jest.replaceProperty(appConfig, 'isCloud', true);
       });
 
       describe('and not in trial and has active subscription', () => {

packages/backend/test/setup/global-hooks.ts

@@ -1,16 +1,6 @@
 import { Model } from 'objection';
 import { client as knex } from '../../src/config/database';
 import logger from '../../src/helpers/logger';
-import { mockConfigState, resetMockConfig } from './set-mock-config';
-
-jest.mock('../../src/config/app', () => ({
-  ...jest.requireActual('../../src/config/app').default,
-  get isCloud() {
-    return mockConfigState.isCloud !== undefined
-      ? mockConfigState.isCloud
-      : jest.requireActual('../../src/config/app').default.isCloud;
-  },
-}));
 
 global.beforeAll(async () => {
   global.knex = null;
@@ -33,7 +23,6 @@ global.afterEach(async () => {
   Model.knex(knex);
 
   jest.clearAllMocks();
-  resetMockConfig();
 });
 
 global.afterAll(async () => {

packages/backend/test/setup/set-mock-config.ts

@@ -1,13 +0,0 @@
-export const mockConfigState = {
-  isCloud: false,
-};
-
-export const resetMockConfig = () => {
-  for (const key in mockConfigState) {
-    delete (mockConfigState as any)[key];
-  }
-};
-
-export const setMockConfig = (config: Partial<typeof mockConfigState>) => {
-  Object.assign(mockConfigState, config);
-};