fix tests

Author: raju-opti

lib/core/decision_service/index.tests.js

@@ -34,6 +34,9 @@ import AudienceEvaluator from '../audience_evaluator';
 import errorHandler from '../../plugins/error_handler';
 import eventDispatcher from '../../plugins/event_dispatcher/index.node';
 import * as jsonSchemaValidator from '../../utils/json_schema_validator';
+import { getMockProjectConfigManager } from '../../tests/mock_project_config_manager';
+import { createProjectConfig } from '../project_config';
+
 import {
   getTestProjectConfig,
   getTestProjectConfigWithFeatures,
@@ -1067,7 +1070,9 @@ describe('lib/core/decision_service', function() {
     beforeEach(function() {
       optlyInstance = new Optimizely({
         clientEngine: 'node-sdk',
-        datafile: cloneDeep(testData),
+        projectConfigManager: getMockProjectConfigManager({
+          initConfig: createProjectConfig(cloneDeep(testData))
+        }),
         jsonSchemaValidator: jsonSchemaValidator,
         isValidInstance: true,
         logger: createdLogger,

lib/optimizely_user_context/index.tests.js

@@ -30,6 +30,8 @@ import eventDispatcher from '../plugins/event_dispatcher/index.node';
 import { CONTROL_ATTRIBUTES, LOG_LEVEL, LOG_MESSAGES } from '../utils/enums';
 import testData from '../tests/test_data';
 import { OptimizelyDecideOption } from '../shared_types';
+import { getMockProjectConfigManager } from '../tests/mock_project_config_manager';
+import { createProjectConfig } from '../core/project_config';
 
 describe('lib/optimizely_user_context', function() {
   describe('APIs', function() {
@@ -356,7 +358,9 @@ describe('lib/optimizely_user_context', function() {
         beforeEach(function() {
           optlyInstance = new Optimizely({
             clientEngine: 'node-sdk',
-            datafile: testData.getTestDecideProjectConfig(),
+            projectConfigManager: getMockProjectConfigManager({
+              initConfig: createProjectConfig(testData.getTestDecideProjectConfig())
+            }),
             errorHandler: errorHandler,
             eventProcessor,
             isValidInstance: true,
@@ -689,7 +693,9 @@ describe('lib/optimizely_user_context', function() {
         beforeEach(function() {
           optlyInstance = new Optimizely({
             clientEngine: 'node-sdk',
-            datafile: testData.getTestDecideProjectConfig(),
+            projectConfigManager: getMockProjectConfigManager({
+              initConfig: createProjectConfig(testData.getTestDecideProjectConfig())
+            }),
             errorHandler: errorHandler,
             eventProcessor,
             isValidInstance: true,
@@ -791,7 +797,9 @@ describe('lib/optimizely_user_context', function() {
         beforeEach(function() {
           optlyInstance = new Optimizely({
             clientEngine: 'node-sdk',
-            datafile: testData.getTestDecideProjectConfig(),
+            projectConfigManager: getMockProjectConfigManager({
+              initConfig: createProjectConfig(testData.getTestDecideProjectConfig())
+            }),
             errorHandler: errorHandler,
             eventProcessor,
             isValidInstance: true,
@@ -833,7 +841,9 @@ describe('lib/optimizely_user_context', function() {
         });
         var optlyInstance = new Optimizely({
           clientEngine: 'node-sdk',
-          datafile: testData.getTestDecideProjectConfig(),
+          projectConfigManager: getMockProjectConfigManager({
+            initConfig: createProjectConfig(testData.getTestDecideProjectConfig())
+          }),
           errorHandler: errorHandler,
           eventProcessor,
           isValidInstance: true,

lib/utils/ticker/ticker.ts

@@ -25,12 +25,12 @@ export class ExponentialBackoff implements BackoffController {
     this.current = base;
   }
 
-  backoff() {
+  backoff(): number {
     this.current = Math.min(this.current * 2, this.max);
     return this.current + this.maxJitter * Math.random();
   }
 
-  reset() {
+  reset(): void {
     this.current = this.base;
   }
 }
@@ -39,7 +39,7 @@ export class IntervalTicker implements Ticker {
   private timeoutId?: NodeJS.Timeout;
   private handler?: AsyncTransformer<boolean, void>;
   private interval: number;
-  private prevSuccess: boolean = true;
+  private prevSuccess = true;
   private backoffController?: BackoffController;
 
   constructor(interval: number, backoffController?: BackoffController) {
@@ -56,7 +56,7 @@ export class IntervalTicker implements Ticker {
         this.setTimer(this.interval);
       }, () => {
         this.prevSuccess = false;
-        let time = this.backoffController?.backoff() ?? this.interval;
+        const time = this.backoffController?.backoff() ?? this.interval;
         this.setTimer(time);
       });
     }, timeout);
@@ -70,7 +70,7 @@ export class IntervalTicker implements Ticker {
     clearInterval(this.timeoutId);
   }
 
-  onTick(handler: AsyncTransformer<boolean, void>) {
+  onTick(handler: AsyncTransformer<boolean, void>): void {
     this.handler = handler;
   }
 }

package.json

@@ -74,7 +74,7 @@
     "clean:win": "(if exist dist rd /s/q dist)",
     "lint": "tsc --noEmit && eslint 'lib/**/*.js' 'lib/**/*.ts'",
     "test-vitest": "tsc --noEmit --p tsconfig.spec.json && vitest run",
-    "test-mocha": "TS_NODE_COMPILER_OPTIONS='{\"module\": \"commonjs\" }' mocha -r ts-node/register -r lib/tests/exit_on_unhandled_rejection.js 'lib/index.browser.tests.js'",
+    "test-mocha": "TS_NODE_COMPILER_OPTIONS='{\"module\": \"commonjs\" }' mocha -r ts-node/register -r lib/tests/exit_on_unhandled_rejection.js '**/*.tests.js'",
     "test": "npm run test-mocha && npm run test-vitest",
     "posttest": "npm run lint",
     "test-ci": "npm run test-xbrowser && npm run test-umdbrowser",

vitest.config.mts

@@ -4,7 +4,7 @@ export default defineConfig({
   test: {
     onConsoleLog: () => true,
     environment: 'happy-dom',
-    include: ["**/static_project_config_manager.spec.ts"],
+    include: ["**/*.spec.ts"],
     typecheck: {
       tsconfig: 'tsconfig.spec.json',
       exclude: ['**/index.react_native.spec.ts'],