[FSSDK-10935] review update

junaed-optimizely · junaed-optimizely · commit c6840873a167 · 2025-01-03T01:03:06.000+06:00
diff --git a/lib/core/audience_evaluator/index.ts b/lib/core/audience_evaluator/index.ts
@@ -23,8 +23,8 @@ import * as conditionTreeEvaluator from '../condition_tree_evaluator';
 import * as customAttributeConditionEvaluator from '../custom_attribute_condition_evaluator';
 import * as odpSegmentsConditionEvaluator from './odp_segment_condition_evaluator';
 import { Audience, Condition, OptimizelyUserContext } from '../../shared_types';
-import { CONDITION_EVALUATOR_ERROR } from '../../error_messages';
-import { AUDIENCE_EVALUATION_RESULT, EVALUATING_AUDIENCE, UNKNOWN_CONDITION_TYPE } from '../../log_messages';
+import { CONDITION_EVALUATOR_ERROR, UNKNOWN_CONDITION_TYPE } from '../../error_messages';
+import { AUDIENCE_EVALUATION_RESULT, EVALUATING_AUDIENCE} from '../../log_messages';
 
 const logger = getLogger();
 const MODULE_NAME = 'AUDIENCE_EVALUATOR';
diff --git a/lib/core/audience_evaluator/odp_segment_condition_evaluator/index.tests.js b/lib/core/audience_evaluator/odp_segment_condition_evaluator/index.tests.js
@@ -20,7 +20,7 @@ import { sprintf } from '../../../utils/fns';
 import { LOG_LEVEL } from '../../../utils/enums';
 import * as logging from '../../../modules/logging';
 import * as odpSegmentEvalutor from './';
-import { UNKNOWN_MATCH_TYPE } from '../../../log_messages';
+import { UNKNOWN_MATCH_TYPE } from '../../../error_messages';
 
 var odpSegment1Condition = {
   "value": "odp-segment-1",
diff --git a/lib/core/audience_evaluator/odp_segment_condition_evaluator/index.ts b/lib/core/audience_evaluator/odp_segment_condition_evaluator/index.ts
@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and      *
  * limitations under the License.                                           *
  ***************************************************************************/
-import { UNKNOWN_MATCH_TYPE } from '../../../log_messages';
+import { UNKNOWN_MATCH_TYPE } from '../../../error_messages';
 import { getLogger } from '../../../modules/logging';
 import { Condition, OptimizelyUserContext } from '../../../shared_types';
 
diff --git a/lib/core/bucketer/index.tests.js b/lib/core/bucketer/index.tests.js
@@ -25,11 +25,11 @@ import projectConfig from '../../project_config/project_config';
 import { getTestProjectConfig } from '../../tests/test_data';
 import { INVALID_BUCKETING_ID, INVALID_GROUP_ID } from '../../error_messages';
 import {
-  USER_ASSIGNED_TO_EXPERIMENT_BUCKET,
   USER_BUCKETED_INTO_EXPERIMENT_IN_GROUP,
   USER_NOT_BUCKETED_INTO_EXPERIMENT_IN_GROUP,
   USER_NOT_IN_ANY_EXPERIMENT,
-} from '../../log_messages';
+  USER_ASSIGNED_TO_EXPERIMENT_BUCKET,
+} from '.';
 
 var buildLogMessageFromArgs = args => sprintf(args[1], ...args.splice(2));
 var testData = getTestProjectConfig();
diff --git a/lib/core/bucketer/index.ts b/lib/core/bucketer/index.ts
@@ -29,13 +29,12 @@ import {
 
 import { LOG_LEVEL } from '../../utils/enums';
 import { INVALID_BUCKETING_ID, INVALID_GROUP_ID } from '../../error_messages';
-import {
-  INVALID_VARIATION_ID,
-  USER_ASSIGNED_TO_EXPERIMENT_BUCKET,
-  USER_BUCKETED_INTO_EXPERIMENT_IN_GROUP,
-  USER_NOT_BUCKETED_INTO_EXPERIMENT_IN_GROUP,
-  USER_NOT_IN_ANY_EXPERIMENT,
-} from '../../log_messages';
+
+export const USER_NOT_IN_ANY_EXPERIMENT = '%s: User %s is not in any experiment of group %s.';
+export const USER_NOT_BUCKETED_INTO_EXPERIMENT_IN_GROUP = '%s: User %s is not in experiment %s of group %s.';
+export const USER_BUCKETED_INTO_EXPERIMENT_IN_GROUP = '%s: User %s is in experiment %s of group %s.';
+export const USER_ASSIGNED_TO_EXPERIMENT_BUCKET = '%s: Assigned bucket %s to user with bucketing ID %s.';
+export const INVALID_VARIATION_ID = '%s: Bucketed into an invalid variation ID. Returning null.';
 
 const HASH_SEED = 1;
 const MAX_HASH_VALUE = Math.pow(2, 32);
diff --git a/lib/core/custom_attribute_condition_evaluator/index.tests.js b/lib/core/custom_attribute_condition_evaluator/index.tests.js
@@ -28,8 +28,8 @@ import {
   UNEXPECTED_CONDITION_VALUE,
   UNEXPECTED_TYPE,
   UNEXPECTED_TYPE_NULL,
-  UNKNOWN_MATCH_TYPE,
 } from '../../log_messages';
+import { UNKNOWN_MATCH_TYPE } from '../../error_messages';
 
 var browserConditionSafari = {
   name: 'browser_type',
diff --git a/lib/core/custom_attribute_condition_evaluator/index.ts b/lib/core/custom_attribute_condition_evaluator/index.ts
@@ -24,8 +24,8 @@ import {
   UNEXPECTED_CONDITION_VALUE,
   UNEXPECTED_TYPE,
   UNEXPECTED_TYPE_NULL,
-  UNKNOWN_MATCH_TYPE,
 } from '../../log_messages';
+import { UNKNOWN_MATCH_TYPE } from '../../error_messages';
 
 const MODULE_NAME = 'CUSTOM_ATTRIBUTE_CONDITION_EVALUATOR';
 
diff --git a/lib/error_messages.ts b/lib/error_messages.ts
@@ -13,7 +13,6 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-
 export const BROWSER_ODP_MANAGER_INITIALIZATION_FAILED = '%s: Error initializing Browser ODP Manager.';
 export const CONDITION_EVALUATOR_ERROR = '%s: Error evaluating audience condition of type %s: %s';
 export const DATAFILE_AND_SDK_KEY_MISSING =
@@ -98,11 +97,11 @@ export const INVALID_VARIATION_KEY = '%s: Provided variation key is in an invali
 export const UNABLE_TO_GET_VUID = 'Unable to get VUID - ODP Manager is not instantiated yet.';
 export const ERROR_FETCHING_DATAFILE = 'Error fetching datafile: %s';
 export const DATAFILE_FETCH_REQUEST_FAILED = 'Datafile fetch request failed with status: %s';
-export const ONRUNNING_ERROR = 'onRunning error';
 export const EVENT_DATA_FOUND_TO_BE_INVALID = 'Event data found to be invalid.';
 export const EVENT_ACTION_INVALID = 'Event action invalid.';
 export const FAILED_TO_SEND_ODP_EVENTS = 'failed to send odp events';
-export const UNABLE_TO_OVERWRITE_GLOBAL_WINDOW = 'Unable to overwrite global.window.';
 export const UNABLE_TO_GET_VUID_VUID_MANAGER_NOT_AVAILABLE = 'Unable to get VUID - VuidManager is not available'
-    
-    
+export const UNKNOWN_CONDITION_TYPE =
+  '%s: Audience condition %s has an unknown condition type. You may need to upgrade to a newer release of the Optimizely SDK.';
+export const UNKNOWN_MATCH_TYPE =
+  '%s: Audience condition %s uses an unknown match type. You may need to upgrade to a newer release of the Optimizely SDK.'; 
diff --git a/lib/event_processor/event_processor_factory.react_native.spec.ts b/lib/event_processor/event_processor_factory.react_native.spec.ts
@@ -60,11 +60,11 @@ async function mockRequireNetInfo() {
   M._load = (uri: string, parent: string) => {
     if (uri === '@react-native-community/netinfo') {
       if (isNetInfoAvailable) return {};
-      throw new Error(MODULE_NOT_FOUND_REACT_NATIVE_NETINFO);
+      throw new Error("Module not found: @react-native-community/netinfo");
     }
     if (uri === '@react-native-async-storage/async-storage') {
       if (isAsyncStorageAvailable) return {};
-      throw new Error(MODULE_NOT_FOUND_REACT_NATIVE_ASYNC_STORAGE);
+      throw new Error("Module not found: @react-native-async-storage/async-storage");
     }
 
     return M._load_original(uri, parent);
@@ -80,10 +80,7 @@ import { AsyncCache, AsyncPrefixCache, SyncCache, SyncPrefixCache } from '../uti
 import { AsyncStorageCache } from '../utils/cache/async_storage_cache.react_native';
 import { ReactNativeNetInfoEventProcessor } from './batch_event_processor.react_native';
 import { BatchEventProcessor } from './batch_event_processor';
-import {
-  MODULE_NOT_FOUND_REACT_NATIVE_ASYNC_STORAGE,
-  MODULE_NOT_FOUND_REACT_NATIVE_NETINFO,
-} from '../exception_messages';
+import { MODULE_NOT_FOUND_REACT_NATIVE_ASYNC_STORAGE } from '../utils/import.react_native/@react-native-async-storage/async-storage';
 
 describe('createForwardingEventProcessor', () => {
   const mockGetForwardingEventProcessor = vi.mocked(getForwardingEventProcessor);
diff --git a/lib/exception_messages.ts b/lib/exception_messages.ts
@@ -30,18 +30,14 @@ export const INSTANCE_CLOSED = 'Instance closed';
 export const DATAFILE_MANAGER_STOPPED = 'Datafile manager stopped before it could be started';
 export const DATAFILE_MANAGER_FAILED_TO_START = 'Datafile manager failed to start';
 export const FAILED_TO_FETCH_DATAFILE = 'Failed to fetch datafile';
-export const FAILED_TO_START = 'Failed to start';
 export const FAILED_TO_STOP = 'Failed to stop';
 export const YOU_MUST_PROVIDE_DATAFILE_IN_SSR = 'You must provide datafile in SSR';
 export const YOU_MUST_PROVIDE_AT_LEAST_ONE_OF_SDKKEY_OR_DATAFILE = 'You must provide at least one of sdkKey or datafile';
 export const RETRY_CANCELLED = 'Retry cancelled';
+export const REQUEST_TIMEOUT = 'Request timeout';
 export const REQUEST_ERROR = 'Request error';
 export const REQUEST_FAILED = 'Request failed';
 export const UNSUPPORTED_PROTOCOL = 'Unsupported protocol: %s';
-export const REQUEST_TIMEOUT = 'Request timed out';
 export const NO_STATUS_CODE_IN_RESPONSE = 'No status code in response';
-export const MODULE_NOT_FOUND_REACT_NATIVE_ASYNC_STORAGE = 'Module not found: @react-native-async-storage/async-storage';
-export const MODULE_NOT_FOUND_REACT_NATIVE_NETINFO = 'Module not found: @react-native-community/netinfo';
-export const INVALID_CONFIG_OR_SOMETHING = 'Invalid config or something';
 export const PROMISE_SHOULD_NOT_HAVE_RESOLVED = 'Promise should not have resolved';
 export const VUID_IS_NOT_SUPPORTED_IN_NODEJS= 'VUID is not supported in Node.js environment';
diff --git a/lib/index.browser.tests.js b/lib/index.browser.tests.js
@@ -24,7 +24,6 @@ import optimizelyFactory from './index.browser';
 import configValidator from './utils/config_validator';
 import { getMockProjectConfigManager } from './tests/mock/mock_project_config_manager';
 import { createProjectConfig } from './project_config/project_config';
-import { UNABLE_TO_OVERWRITE_GLOBAL_WINDOW } from './error_messages';
 import { INVALID_CONFIG_OR_SOMETHING } from './exception_messages';
 
 
@@ -56,7 +55,7 @@ if (!global.window) {
       localStorage: new MockLocalStorage(),
     };
   } catch (e) {
-    console.error(UNABLE_TO_OVERWRITE_GLOBAL_WINDOW);
+    console.error("Unable to overwrite global.window");
   }
 }
 
diff --git a/lib/log_messages.ts b/lib/log_messages.ts
@@ -30,7 +30,6 @@ export const INVALID_OBJECT = '%s: Optimizely object is not valid. Failing %s.';
 export const INVALID_CLIENT_ENGINE = '%s: Invalid client engine passed: %s. Defaulting to node-sdk.';
 export const INVALID_DEFAULT_DECIDE_OPTIONS = '%s: Provided default decide options is not an array.';
 export const INVALID_DECIDE_OPTIONS = '%s: Provided decide options is not an array. Using default decide options.';
-export const INVALID_VARIATION_ID = '%s: Bucketed into an invalid variation ID. Returning null.';
 export const NOTIFICATION_LISTENER_EXCEPTION = '%s: Notification listener for (%s) threw exception: %s';
 export const NO_ROLLOUT_EXISTS = '%s: There is no rollout of feature %s.';
 export const NOT_ACTIVATING_USER = '%s: Not activating user %s for experiment %s.';
@@ -54,14 +53,11 @@ export const SHOULD_NOT_DISPATCH_ACTIVATE = '%s: Experiment %s is not in "Runnin
 export const SKIPPING_JSON_VALIDATION = '%s: Skipping JSON schema validation.';
 export const TRACK_EVENT = '%s: Tracking event %s for user %s.';
 export const UNRECOGNIZED_DECIDE_OPTION = '%s: Unrecognized decide option %s provided.';
-export const USER_ASSIGNED_TO_EXPERIMENT_BUCKET = '%s: Assigned bucket %s to user with bucketing ID %s.';
-export const USER_BUCKETED_INTO_EXPERIMENT_IN_GROUP = '%s: User %s is in experiment %s of group %s.';
 export const USER_BUCKETED_INTO_TARGETING_RULE = '%s: User %s bucketed into targeting rule %s.';
 export const USER_IN_FEATURE_EXPERIMENT = '%s: User %s is in variation %s of experiment %s on the feature %s.';
 export const USER_IN_ROLLOUT = '%s: User %s is in rollout of feature %s.';
 export const USER_NOT_BUCKETED_INTO_EVERYONE_TARGETING_RULE =
   '%s: User %s not bucketed into everyone targeting rule due to traffic allocation.';
-export const USER_NOT_BUCKETED_INTO_EXPERIMENT_IN_GROUP = '%s: User %s is not in experiment %s of group %s.';
 export const USER_NOT_BUCKETED_INTO_ANY_EXPERIMENT_IN_GROUP = '%s: User %s is not in any experiment of group %s.';
 export const USER_NOT_BUCKETED_INTO_TARGETING_RULE =
   '%s User %s not bucketed into targeting rule %s due to traffic allocation. Trying everyone rule.';
@@ -86,7 +82,6 @@ export const USER_HAS_NO_VARIATION = '%s: User %s is in no variation of experime
 export const USER_HAS_NO_FORCED_VARIATION = '%s: User %s is not in the forced variation map.';
 export const USER_HAS_NO_FORCED_VARIATION_FOR_EXPERIMENT =
   '%s: No experiment %s mapped to user %s in the forced variation map.';
-export const USER_NOT_IN_ANY_EXPERIMENT = '%s: User %s is not in any experiment of group %s.';
 export const USER_NOT_IN_EXPERIMENT = '%s: User %s does not meet conditions to be in experiment %s.';
 export const USER_NOT_IN_ROLLOUT =  '%s: User %s is not in rollout of feature %s.';
 export const USER_RECEIVED_DEFAULT_VARIABLE_VALUE =
@@ -115,10 +110,6 @@ export const UNEXPECTED_TYPE =
   '%s: Audience condition %s evaluated to UNKNOWN because a value of type "%s" was passed for user attribute "%s".';
 export const UNEXPECTED_TYPE_NULL =
   '%s: Audience condition %s evaluated to UNKNOWN because a null value was passed for user attribute "%s".';
-export const UNKNOWN_CONDITION_TYPE =
-  '%s: Audience condition %s has an unknown condition type. You may need to upgrade to a newer release of the Optimizely SDK.';
-export const UNKNOWN_MATCH_TYPE =
-  '%s: Audience condition %s uses an unknown match type. You may need to upgrade to a newer release of the Optimizely SDK.';
 export const UPDATED_OPTIMIZELY_CONFIG = '%s: Updated Optimizely config to revision %s (project id %s)';
 export const OUT_OF_BOUNDS =
   '%s: Audience condition %s evaluated to UNKNOWN because the number value for user attribute "%s" is not in the range [-2^53, +2^53].';
diff --git a/lib/odp/odp_manager.spec.ts b/lib/odp/odp_manager.spec.ts
@@ -25,7 +25,7 @@ import { ODP_USER_KEY } from './constant';
 import { OptimizelySegmentOption } from './segment_manager/optimizely_segment_option';
 import { OdpEventManager } from './event_manager/odp_event_manager';
 import { CLIENT_VERSION, JAVASCRIPT_CLIENT_ENGINE } from '../utils/enums';
-import { FAILED_TO_START, FAILED_TO_STOP } from '../exception_messages';
+import { FAILED_TO_STOP } from '../exception_messages';
 
 const keyA = 'key-a';
 const hostA = 'host-a';
@@ -167,7 +167,7 @@ describe('DefaultOdpManager', () => {
     await exhaustMicrotasks();
 
     expect(odpManager.getState()).toEqual(ServiceState.Starting);
-    eventManagerPromise.reject(new Error(FAILED_TO_START));
+    eventManagerPromise.reject(new Error("Failed to start"));
 
     await expect(odpManager.onRunning()).rejects.toThrow();
     await expect(odpManager.onTerminated()).rejects.toThrow();
@@ -187,7 +187,7 @@ describe('DefaultOdpManager', () => {
     odpManager.start();
     expect(odpManager.getState()).toEqual(ServiceState.Starting);
 
-    eventManagerPromise.reject(new Error(FAILED_TO_START));
+    eventManagerPromise.reject(new Error("Failed to start"));
 
     await expect(odpManager.onRunning()).rejects.toThrow();
     await expect(odpManager.onTerminated()).rejects.toThrow();
diff --git a/lib/odp/segment_manager/odp_segment_api_manager.spec.ts b/lib/odp/segment_manager/odp_segment_api_manager.spec.ts
@@ -20,7 +20,6 @@ import { ODP_USER_KEY } from '../constant';
 import { getMockRequestHandler } from '../../tests/mock/mock_request_handler';
 import { getMockLogger } from '../../tests/mock/mock_logger';
 import { DefaultOdpSegmentApiManager } from './odp_segment_api_manager';
-import { REQUEST_TIMEOUT } from '../../exception_messages';
 
 const API_KEY = 'not-real-api-key';
 const GRAPHQL_ENDPOINT = 'https://some.example.com/graphql/endpoint';
@@ -47,7 +46,7 @@ describe('DefaultOdpSegmentApiManager', () => {
     const requestHandler = getMockRequestHandler();
     requestHandler.makeRequest.mockReturnValue({
       abort: () => {},
-      responsePromise: Promise.reject(new Error(REQUEST_TIMEOUT)),
+      responsePromise: Promise.reject(new Error("Request timed out")),
     });
     const logger = getMockLogger();
     const manager = new DefaultOdpSegmentApiManager(requestHandler, logger);
diff --git a/lib/optimizely/index.tests.js b/lib/optimizely/index.tests.js
@@ -49,7 +49,6 @@ import {
   NOT_ACTIVATING_USER,
   NOT_TRACKING_USER,
   RETURNING_STORED_VARIATION,
-  USER_BUCKETED_INTO_EXPERIMENT_IN_GROUP,
   USER_DOESNT_MEET_CONDITIONS_FOR_TARGETING_RULE,
   USER_FORCED_IN_VARIATION,
   USER_HAS_FORCED_VARIATION,
@@ -73,6 +72,7 @@ import {
   USER_NOT_IN_FORCED_VARIATION,
 } from '../error_messages';
 import { FAILED_TO_STOP, ONREADY_TIMEOUT_EXPIRED, PROMISE_SHOULD_NOT_HAVE_RESOLVED } from '../exception_messages';
+import { USER_BUCKETED_INTO_EXPERIMENT_IN_GROUP } from '../core/bucketer';
 
 var LOG_LEVEL = enums.LOG_LEVEL;
 var DECISION_SOURCES = enums.DECISION_SOURCES;
diff --git a/lib/project_config/config_manager_factory.react_native.spec.ts b/lib/project_config/config_manager_factory.react_native.spec.ts
@@ -30,7 +30,7 @@ async function mockRequireAsyncStorage() {
   M._load = (uri: string, parent: string) => {
     if (uri === '@react-native-async-storage/async-storage') {
       if (isAsyncStorageAvailable) return { default: {} };
-      throw new Error(MODULE_NOT_FOUND_REACT_NATIVE_ASYNC_STORAGE);
+      throw new Error("Module not found: @react-native-async-storage/async-storage");
     }
     return M._load_original(uri, parent);
   };
@@ -61,7 +61,6 @@ import { createPollingProjectConfigManager } from './config_manager_factory.reac
 import { BrowserRequestHandler } from '../utils/http_request_handler/browser_request_handler';
 import { AsyncStorageCache } from '../utils/cache/async_storage_cache.react_native';
 import { getMockSyncCache } from '../tests/mock/mock_cache';
-import { MODULE_NOT_FOUND_REACT_NATIVE_ASYNC_STORAGE } from '../exception_messages';
 
 describe('createPollingConfigManager', () => {
   const mockGetPollingConfigManager = vi.mocked(getPollingConfigManager);
@@ -158,7 +157,7 @@ describe('createPollingConfigManager', () => {
     };
 
     expect(() => createPollingProjectConfigManager(config)).toThrowError(
-      MODULE_NOT_FOUND_REACT_NATIVE_ASYNC_STORAGE
+      "Module not found: @react-native-async-storage/async-storage"
     );
     isAsyncStorageAvailable = true;
   });
diff --git a/lib/project_config/project_config_manager.spec.ts b/lib/project_config/project_config_manager.spec.ts
@@ -22,7 +22,6 @@ import { createProjectConfig } from './project_config';
 import { resolvablePromise } from '../utils/promise/resolvablePromise';
 import { getMockDatafileManager } from '../tests/mock/mock_datafile_manager';
 import { wait } from '../../tests/testUtils';
-import { ONRUNNING_ERROR } from '../error_messages';
 
 const cloneDeep = (x: any) => JSON.parse(JSON.stringify(x));
 
@@ -128,7 +127,7 @@ describe('ProjectConfigManagerImpl', () => {
         });
 
         it('should resolve onRunning() even if datafileManger.onRunning() rejects', async () => {
-          const onRunning = Promise.reject(new Error(ONRUNNING_ERROR));
+          const onRunning = Promise.reject(new Error("onRunning error"));
           const datafileManager = getMockDatafileManager({
             onRunning,
           });
diff --git a/lib/utils/import.react_native/@react-native-async-storage/async-storage.ts b/lib/utils/import.react_native/@react-native-async-storage/async-storage.ts
@@ -15,7 +15,8 @@
  */
 
 import type { AsyncStorageStatic } from '@react-native-async-storage/async-storage'
-import { MODULE_NOT_FOUND_REACT_NATIVE_ASYNC_STORAGE } from '../../../exception_messages';
+
+export const MODULE_NOT_FOUND_REACT_NATIVE_ASYNC_STORAGE = 'Module not found: @react-native-async-storage/async-storage';
 
 export const getDefaultAsyncStorage = (): AsyncStorageStatic => {
   try {
diff --git a/lib/utils/semantic_version/index.ts b/lib/utils/semantic_version/index.ts
@@ -13,7 +13,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import { UNKNOWN_MATCH_TYPE } from '../../log_messages';
+import { UNKNOWN_MATCH_TYPE } from '../../error_messages';
 import { getLogger } from '../../modules/logging';
 import { VERSION_TYPE } from '../enums';
 
diff --git a/tests/index.react_native.spec.ts b/tests/index.react_native.spec.ts
@@ -24,7 +24,6 @@ import optimizelyFactory from '../lib/index.react_native';
 import configValidator from '../lib/utils/config_validator';
 import { getMockProjectConfigManager } from '../lib/tests/mock/mock_project_config_manager';
 import { createProjectConfig } from '../lib/project_config/project_config';
-import { INVALID_CONFIG_OR_SOMETHING } from '../lib/exception_messages';
 
 vi.mock('@react-native-community/netinfo');
 vi.mock('react-native-get-random-values')
@@ -63,7 +62,7 @@ describe('javascript-sdk/react-native', () => {
         silentLogger = optimizelyFactory.logging.createLogger();
         vi.spyOn(console, 'error');
         vi.spyOn(configValidator, 'validate').mockImplementation(() => {
-          throw new Error(INVALID_CONFIG_OR_SOMETHING);
+          throw new Error('Invalid config or something');
         });
       });
 
