fix unit tests

Author: raju-opti

lib/core/audience_evaluator/index.spec.ts

@@ -354,7 +354,6 @@ describe('lib/core/audience_evaluator', () => {
 
           it('logs correctly when conditionTreeEvaluator.evaluate returns false', () => {
             evaluateSpy.mockImplementationOnce((conditions: any, leafEvaluator) => {
-              console.log('evaluateSpy leafEvaluator called with conditions:', conditions);
               return leafEvaluator(conditions[1]);
             });
 

lib/core/audience_evaluator/index.ts

@@ -39,8 +39,6 @@ export class AudienceEvaluator {
    * @constructor
    */
   constructor(UNSTABLE_conditionEvaluators: unknown, logger?: LoggerFacade) {
-
-    // console.log('AudienceEvaluator constructor called', customAttributeConditionEvaluator.getEvaluator());
     this.logger = logger;
     this.typeToEvaluatorMap = {
       ...UNSTABLE_conditionEvaluators as any,

lib/core/bucketer/index.spec.ts

@@ -94,7 +94,7 @@ describe('excluding groups', () => {
   });
 
   afterEach(() => {
-    vi.restoreAllMocks();
+    vi.resetAllMocks();
   });
 
   it('should return decision response with correct variation ID when provided bucket value', async () => {
@@ -142,7 +142,7 @@ describe('including groups: random', () => {
   });
 
   afterEach(() => {
-    vi.restoreAllMocks();
+    vi.resetAllMocks();
   });
 
   it('should return decision response with the proper variation for a user in a grouped experiment', () => {
@@ -248,7 +248,7 @@ describe('including groups: overlapping', () => {
   });
 
   afterEach(() => {
-    vi.restoreAllMocks();
+    vi.resetAllMocks();
   });
 
   it('should return decision response with variation when a user falls into an experiment within an overlapping group', () => {
@@ -301,7 +301,7 @@ describe('bucket value falls into empty traffic allocation ranges', () => {
   });
 
   afterEach(() => {
-    vi.restoreAllMocks();
+    vi.resetAllMocks();
   });
 
   it('should return decision response with variation null', () => {
@@ -351,7 +351,7 @@ describe('traffic allocation has invalid variation ids', () => {
   });
 
   afterEach(() => {
-    vi.restoreAllMocks();
+    vi.resetAllMocks();
   });
 
   it('should return decision response with variation null', () => {

vitest.browser.config.mts

@@ -312,7 +312,7 @@ export default defineConfig({
     testTimeout: 60000, // 60 seconds per test - if longer, something is wrong
     hookTimeout: 30000, // 30 seconds for hooks
     include: [
-      'lib/**/**/*.spec.ts',
+      'lib/**/*.spec.ts',
     ],
     exclude: [
       'lib/**/*.react_native.spec.ts',

vitest.config.mts

@@ -26,7 +26,7 @@ export default defineConfig({
   test: {
     onConsoleLog: () => true,
     environment: 'happy-dom',
-    include: ['**/audience_evaluator/*.spec.ts'],
+    include: ['lib/**/*.spec.ts'],
     typecheck: {
       enabled: true,
       tsconfig: 'tsconfig.spec.json',