[FSSDK-9897] update return type of getFeatureVariable method

optimizely · Dec 28, 2023 · e2e6025 · e2e6025
1 parent 2b40193
commit e2e6025
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 6 deletions.
diff --git a/lib/core/project_config/index.ts b/lib/core/project_config/index.ts
@@ -32,6 +32,7 @@ import {
   VariableType,
   VariationVariable,
   Integration,
+  OptimizelyVariableValue,
 } from '../../shared_types';
 
 interface TryCreatingProjectConfigConfig {
@@ -692,8 +693,8 @@ export const getTypeCastValue = function(
   variableValue: string,
   variableType: VariableType,
   logger: LogHandler
-): unknown {
-  let castValue;
+): OptimizelyVariableValue {
+  let castValue : OptimizelyVariableValue;
 
   switch (variableType) {
     case FEATURE_VARIABLE_TYPES.BOOLEAN:

diff --git a/lib/export_types.ts b/lib/export_types.ts
@@ -22,6 +22,7 @@ export {
   UserAttributeValue,
   UserAttributes,
   OptimizelyConfig,
+  OptimizelyVariableValue,
   OptimizelyVariable,
   OptimizelyVariation,
   OptimizelyExperiment,

diff --git a/lib/optimizely/index.ts b/lib/optimizely/index.ts
@@ -35,6 +35,7 @@ import {
   FeatureVariable,
   OptimizelyOptions,
   OptimizelyDecideOption,
+  OptimizelyVariableValue,
   OptimizelyDecision,
   Client,
 } from '../shared_types';
@@ -777,7 +778,7 @@ export default class Optimizely implements Client {
    *                                                type, or null if the feature key is invalid or
    *                                                the variable key is invalid
    */
-  getFeatureVariable(featureKey: string, variableKey: string, userId: string, attributes?: UserAttributes): unknown {
+  getFeatureVariable(featureKey: string, variableKey: string, userId: string, attributes?: UserAttributes): OptimizelyVariableValue {
     try {
       if (!this.isValidInstance()) {
         this.logger.log(LOG_LEVEL.ERROR, LOG_MESSAGES.INVALID_OBJECT, MODULE_NAME, 'getFeatureVariable');
@@ -819,7 +820,7 @@ export default class Optimizely implements Client {
     variableType: string | null,
     userId: string,
     attributes?: UserAttributes
-  ): unknown {
+  ): OptimizelyVariableValue {
     if (!this.validateInputs({ feature_key: featureKey, variable_key: variableKey, user_id: userId }, attributes)) {
       return null;
     }
@@ -911,7 +912,7 @@ export default class Optimizely implements Client {
     variation: Variation | null,
     variable: FeatureVariable,
     userId: string
-  ): unknown {
+  ): OptimizelyVariableValue {
     const configObj = this.projectConfigManager.getConfig();
     if (!configObj) {
       return null;

diff --git a/lib/shared_types.ts b/lib/shared_types.ts
@@ -311,6 +311,8 @@ export interface OptimizelyExperiment {
   };
 }
 
+export type OptimizelyVariableValue = number | string | boolean | object | null;
+
 export interface OptimizelyVariable {
   id: string;
   key: string;
@@ -330,7 +332,7 @@ export interface Client {
   getForcedVariation(experimentKey: string, userId: string): string | null;
   isFeatureEnabled(featureKey: string, userId: string, attributes?: UserAttributes): boolean;
   getEnabledFeatures(userId: string, attributes?: UserAttributes): string[];
-  getFeatureVariable(featureKey: string, variableKey: string, userId: string, attributes?: UserAttributes): unknown;
+  getFeatureVariable(featureKey: string, variableKey: string, userId: string, attributes?: UserAttributes): OptimizelyVariableValue;
   getFeatureVariableBoolean(
     featureKey: string,
     variableKey: string,