save

Author: raju-opti

lib/index.browser.tests.js

@@ -0,0 +1,48 @@
+import { test, vi, expect } from 'vitest';
+
+import { StaticProjectConfigManager } from './static_project_config_manager';
+import { ServiceState } from '../service';
+import testData from '../tests/test_data';
+import { advanceTimersByTime } from '../../tests/testUtils';
+
+test('state is New on instatiation', () => {
+  const configManagaer = new StaticProjectConfigManager({});
+  expect(configManagaer.getState()).toEqual(ServiceState.New);
+});
+
+test('promise returned from onRunning is is not fulfilled until start() is called', async() => {
+  const configManagaer = new StaticProjectConfigManager({});
+  const startSpy = vi.spyOn(configManagaer, 'start');
+  const promiseSpy = vi.fn();
+
+  configManagaer.onRunning().then(promiseSpy, promiseSpy);
+
+  vi.useFakeTimers();
+  await advanceTimersByTime(10000);
+
+  expect(promiseSpy).not.toHaveBeenCalled();
+
+  configManagaer.start();
+  await Promise.resolve();
+
+  expect(promiseSpy).toHaveBeenCalledOnce();
+  vi.useRealTimers();
+});
+
+test('promise returned from onRunning is rejected in case of invalid datafile', () => {
+  const configManagaer = new StaticProjectConfigManager({});
+  configManagaer.start();
+  expect(configManagaer.onRunning()).rejects.toBeInstanceOf(Error);
+});
+
+test('promise returned from onRunning is rejected in case of invalid datafile', () => {
+  const configManagaer = new StaticProjectConfigManager({});
+  configManagaer.start();
+  expect(configManagaer.onRunning()).rejects.toBeInstanceOf(Error);
+});
+
+test('promise returned from onRunning is resolved in case of valid datafile', () => {
+  const configManagaer = new StaticProjectConfigManager(testData.getTestProjectConfig());
+  configManagaer.start();
+  expect(configManagaer.onRunning()).resolves.toBeUndefined();
+});

lib/project_config/static_project_config_manager.spec.ts

@@ -28,8 +28,8 @@ export class StaticProjectConfigManager implements ProjectConfigManager {
     this.stopPrommise = resolvablePromise();
   }
 
-  getOptimizelyConfig(): OptimizelyConfig | null {
-    return this.optimizelyConfig || null;
+  getOptimizelyConfig(): OptimizelyConfig | undefined {
+    return this.optimizelyConfig;
   }
 
   getState(): ServiceState {

lib/project_config/static_project_config_manager.ts

@@ -0,0 +1,28 @@
+import sinon from 'sinon';
+
+export const getMockProjectConfigManager = (opt = {}) => {
+  return {
+    config: opt.initConfig,
+    start: sinon.spy(),
+    onRunning: () => opt.onRunning || Promise.resolve(),
+    stop: sinon.spy(),
+    onTerminated: () => opt.onTerminated || Promise.resolve(),
+    getConfig: function() {
+      return this.config;
+    },
+    setConfig: function(config) {
+      this.config = config;
+    },
+    onUpdate: function(listener) {
+      if (this.listeners === undefined) {
+        this.listeners = [];
+      }
+      this.listeners.push(listener);
+      return () => {};
+    },
+    pushUpdate: function(config) {
+      this.config = config;
+      this.listeners.forEach((listener) => listener(config));
+    }
+  }
+};

lib/tests/mock_project_config_manager.js

@@ -1,8 +1,6 @@
 import { it, vi, expect } from 'vitest';
 
 import { EventEmitter } from './event_emitter';
-import exp from 'constants';
-import { emit, removeAllListeners } from 'process';
 
 it('should call all registered listeners correctly on emit event', () => {
   const emitter = new EventEmitter<{ foo: number, bar: string, baz: boolean}>();

lib/utils/event_emitter/event_emitter.spec.ts

@@ -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/optimizely/index.tests.js'",
+    "test-mocha": "TS_NODE_COMPILER_OPTIONS='{\"module\": \"commonjs\" }' mocha -r ts-node/register -r lib/tests/exit_on_unhandled_rejection.js 'lib/index.node.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",

package.json

@@ -2,10 +2,10 @@
   "extends": "./tsconfig.json",
   "compilerOptions": {
     "types": [
-      "vitest/jsdom"
+      "vitest/jsdom",
     ],
     "typeRoots": [
-      "./node_modules/@types"
+      "./node_modules/@types", "."
     ]
   },
   "include": [

tsconfig.spec.json

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