optimizely · junaed-optimizely · May 7, 2025 · May 7, 2025
diff --git a/lib/odp/odp_manager.ts b/lib/odp/odp_manager.ts
@@ -179,11 +179,7 @@ export class DefaultOdpManager extends BaseService implements OdpManager {
     }
 
     this.odpIntegrationConfig = odpIntegrationConfig;
-
-    if (this.isStarting()) {
-      this.configPromise.resolve();
-    }
-
+    this.configPromise.resolve();
     this.segmentManager.updateConfig(odpIntegrationConfig)
     this.eventManager.updateConfig(odpIntegrationConfig);
 

diff --git a/lib/core/audience_evaluator/index.spec.ts b/lib/core/audience_evaluator/index.spec.ts
  isQualifiedFor: segment => segments ? segments.indexOf(segment) > -1 : false,
  qualifiedSegments: segments || [],
  getUserId: () => 'mockUserId',
  setAttribute: (key: string, value: any) => {},

  decide: (key: string, options?: OptimizelyDecideOption[]): OptimizelyDecision => ({
    variationKey: 'mockVariationKey',
    enabled: true,
    variables: { mockVariable: 'mockValue' },
          });

          it('calls customAttributeConditionEvaluator.evaluate in the leaf evaluator for audience conditions', () => {
            vi.spyOn(conditionTreeEvaluator, 'evaluate').mockImplementation((conditions: any, leafEvaluator) => {
              return leafEvaluator(conditions[1]);
            });
          
          });

          it('logs correctly when conditionTreeEvaluator.evaluate returns null', () => {
            vi.spyOn(conditionTreeEvaluator, 'evaluate').mockImplementationOnce((conditions: any, leafEvaluator) => {
              return leafEvaluator(conditions[1]);
            });

          });

          it('logs correctly when conditionTreeEvaluator.evaluate returns true', () => {
            vi.spyOn(conditionTreeEvaluator, 'evaluate').mockImplementationOnce((conditions: any, leafEvaluator) => {
              return leafEvaluator(conditions[1]);
            });

          });

          it('logs correctly when conditionTreeEvaluator.evaluate returns false', () => {
            vi.spyOn(conditionTreeEvaluator, 'evaluate').mockImplementationOnce((conditions: any, leafEvaluator) => {
              return leafEvaluator(conditions[1]);
            });

          };
          audienceEvaluator = createAudienceEvaluator({
            special_condition_type: {
              evaluate: (condition: any, user: any) => {
                // eslint-disable-next-line @typescript-eslint/ban-ts-comment
                // @ts-ignore
                const result = mockEnvironment[condition.value] && user.getAttributes()[condition.match] > 0;
	isQualifiedFor: segment => segments ? segments.indexOf(segment) > -1 : false,
	qualifiedSegments: segments \|\| [],
	getUserId: () => 'mockUserId',
	setAttribute: (key: string, value: any) => {},
Check warning on line 33 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / lint `'key' is defined but never used` Check warning on line 33 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / lint `'value' is defined but never used` Check warning on line 33 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / lint `Unexpected any. Specify a different type` Check warning on line 33 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (18) `'key' is defined but never used` Check warning on line 33 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (18) `'value' is defined but never used` Check warning on line 33 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (18) `Unexpected any. Specify a different type` Check warning on line 33 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (20) `'key' is defined but never used` Check warning on line 33 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (20) `'value' is defined but never used` Check warning on line 33 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (20) `Unexpected any. Specify a different type` Check warning on line 33 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (22) `'key' is defined but never used` Check warning on line 33 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (22) `'value' is defined but never used` Check warning on line 33 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (22) `Unexpected any. Specify a different type`

	decide: (key: string, options?: OptimizelyDecideOption[]): OptimizelyDecision => ({
Check warning on line 35 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / lint `'key' is defined but never used` Check warning on line 35 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / lint `'options' is defined but never used` Check warning on line 35 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (18) `'key' is defined but never used` Check warning on line 35 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (18) `'options' is defined but never used` Check warning on line 35 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (20) `'key' is defined but never used` Check warning on line 35 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (20) `'options' is defined but never used` Check warning on line 35 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (22) `'key' is defined but never used` Check warning on line 35 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (22) `'options' is defined but never used`
	variationKey: 'mockVariationKey',
	enabled: true,
	variables: { mockVariable: 'mockValue' },
	});

	it('calls customAttributeConditionEvaluator.evaluate in the leaf evaluator for audience conditions', () => {
	vi.spyOn(conditionTreeEvaluator, 'evaluate').mockImplementation((conditions: any, leafEvaluator) => {
Check warning on line 253 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / lint `Unexpected any. Specify a different type` Check warning on line 253 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (18) `Unexpected any. Specify a different type` Check warning on line 253 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (20) `Unexpected any. Specify a different type` Check warning on line 253 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (22) `Unexpected any. Specify a different type`
	return leafEvaluator(conditions[1]);
	});

	});

	it('logs correctly when conditionTreeEvaluator.evaluate returns null', () => {
	vi.spyOn(conditionTreeEvaluator, 'evaluate').mockImplementationOnce((conditions: any, leafEvaluator) => {
Check warning on line 295 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / lint `Unexpected any. Specify a different type` Check warning on line 295 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (18) `Unexpected any. Specify a different type` Check warning on line 295 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (20) `Unexpected any. Specify a different type` Check warning on line 295 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (22) `Unexpected any. Specify a different type`
	return leafEvaluator(conditions[1]);
	});

	});

	it('logs correctly when conditionTreeEvaluator.evaluate returns true', () => {
	vi.spyOn(conditionTreeEvaluator, 'evaluate').mockImplementationOnce((conditions: any, leafEvaluator) => {
Check warning on line 322 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / lint `Unexpected any. Specify a different type` Check warning on line 322 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (18) `Unexpected any. Specify a different type` Check warning on line 322 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (20) `Unexpected any. Specify a different type` Check warning on line 322 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (22) `Unexpected any. Specify a different type`
	return leafEvaluator(conditions[1]);
	});

	});

	it('logs correctly when conditionTreeEvaluator.evaluate returns false', () => {
	vi.spyOn(conditionTreeEvaluator, 'evaluate').mockImplementationOnce((conditions: any, leafEvaluator) => {
Check warning on line 348 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / lint `Unexpected any. Specify a different type` Check warning on line 348 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (18) `Unexpected any. Specify a different type` Check warning on line 348 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (20) `Unexpected any. Specify a different type` Check warning on line 348 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (22) `Unexpected any. Specify a different type`
	return leafEvaluator(conditions[1]);
	});

	};
	audienceEvaluator = createAudienceEvaluator({
	special_condition_type: {
	evaluate: (condition: any, user: any) => {
Check warning on line 384 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / lint `Unexpected any. Specify a different type` Check warning on line 384 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (18) `Unexpected any. Specify a different type` Check warning on line 384 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (20) `Unexpected any. Specify a different type` Check warning on line 384 in lib/core/audience_evaluator/index.spec.ts GitHub Actions / unit_tests (22) `Unexpected any. Specify a different type`
	// eslint-disable-next-line @typescript-eslint/ban-ts-comment
	// @ts-ignore
	const result = mockEnvironment[condition.value] && user.getAttributes()[condition.match] > 0;