change loadedGlobalConfigs to getLoadedGlobalConfigs (#803)

Author: Assem-Uber

src/utils/config/__tests__/get-config-value.node.ts

@@ -1,13 +1,13 @@
 import { z } from 'zod';
 
 import getConfigValue from '../get-config-value';
-import { loadedGlobalConfigs } from '../global-configs-ref';
+import { getLoadedGlobalConfigs } from '../global-configs-ref';
 
 jest.mock('../global-configs-ref', () => ({
-  loadedGlobalConfigs: {
+  getLoadedGlobalConfigs: jest.fn().mockReturnValue({
     COMPUTED: jest.fn(),
     CADENCE_WEB_PORT: 'someValue',
-  },
+  }),
 }));
 
 jest.mock('@/config/dynamic/resolvers/schemas/resolver-schemas', () => ({
@@ -30,8 +30,8 @@ describe('getConfigValue', () => {
 
   it('calls the function with the provided argument and returns the result', async () => {
     // TODO: Fix the type of LoadedConfigs and make it more unit testable
-    // @ts-expect-error COMPUTED is a function
-    const mockFn = loadedGlobalConfigs.COMPUTED as jest.Mock;
+    // @ts-expect-error COMPUTED doesn't exist in the original loaded configs but it exists in the mocks
+    const mockFn = getLoadedGlobalConfigs().COMPUTED as jest.Mock;
     mockFn.mockResolvedValue('resolvedValue');
     // @ts-expect-error COMPUTED is not a loaded config
     const result = await getConfigValue('COMPUTED');

src/utils/config/get-config-value.ts

@@ -8,7 +8,7 @@ import type {
   ConfigKeysWithoutArgs,
   ResolverSchemas,
 } from './config.types';
-import { loadedGlobalConfigs } from './global-configs-ref';
+import { getLoadedGlobalConfigs } from './global-configs-ref';
 
 // Overload for keys requiring arguments
 export default async function getConfigValue<K extends ConfigKeysWithArgs>(
@@ -35,6 +35,7 @@ export default async function getConfigValue<K extends keyof LoadedConfigs>(
     throw new Error('getConfigValue cannot be invoked on browser');
   }
 
+  const loadedGlobalConfigs = getLoadedGlobalConfigs();
   const value = loadedGlobalConfigs[key as K];
 
   if (typeof value === 'function') {

src/utils/config/global-configs-ref.ts

@@ -7,5 +7,8 @@ const setLoadedGlobalConfigs = (c: LoadedConfigs): void => {
   globalConfigRef.value = c;
 };
 
-const loadedGlobalConfigs: LoadedConfigs = globalConfigRef.value;
-export { loadedGlobalConfigs, setLoadedGlobalConfigs };
+const getLoadedGlobalConfigs = (): LoadedConfigs => {
+  return globalConfigRef.value;
+};
+
+export { getLoadedGlobalConfigs, setLoadedGlobalConfigs };