feat: ImmuatableConfiguration

src/configuration-requestor.spec.ts

@@ -14,7 +14,7 @@
   IHttpClient,
   IUniversalFlagConfigResponse,
 } from './http-client';
-import { StoreBackedConfiguration } from './i-configuration';
+import { ImmuatableConfiguration } from './i-configuration';
 import { BanditParameters, BanditVariation, Flag } from './interfaces';
 
 describe('ConfigurationRequestor', () => {
@@ -506,7 +506,7 @@
         );
 
         const config = requestor.getConfiguration();
-        expect(config).toBeInstanceOf(StoreBackedConfiguration);
+        expect(config).toBeInstanceOf(ImmuatableConfiguration);
         expect(config.getFlagKeys()).toEqual([]);
       });
 
@@ -521,7 +521,7 @@
         await requestor.fetchAndStoreConfigurations();
 
         const config = requestor.getConfiguration();
-        expect(config).toBeInstanceOf(StoreBackedConfiguration);
+        expect(config).toBeInstanceOf(ImmuatableConfiguration);
         expect(config.getFlagKeys()).toEqual(['test_flag']);
       });
     });
@@ -534,10 +534,11 @@
           banditVariationStore,
           banditModelStore,
         );
-        const config = requestor.getConfiguration();
 
         await requestor.fetchAndStoreConfigurations();
 
+        const config = requestor.getConfiguration();
+
         expect(config.getFlagKeys()).toEqual(['test_flag']);
         expect(config.getFlagConfigDetails()).toEqual({
           configEnvironment: { name: 'Test' },
@@ -554,10 +555,11 @@
           banditVariationStore,
           banditModelStore,
         );
-        const config = requestor.getConfiguration();
 
         await requestor.fetchAndStoreConfigurations();
 
+        const config = requestor.getConfiguration();
+
         // Verify flag configuration
         expect(config.getFlagKeys()).toEqual(['test_flag']);
 
@@ -610,7 +612,7 @@
           banditModelStore,
         );
 
         const ufcResponse = {
           flags: { test_flag: { key: 'test_flag', value: true } },
           banditReferences: {
             bandit: {

src/configuration-requestor.ts

@@ -11,6 +11,7 @@ import { BanditVariation, BanditParameters, Flag, BanditReference } from './inte
 export default class ConfigurationRequestor {
   private banditModelVersions: string[] = [];
   private readonly configuration: StoreBackedConfiguration;
+  private configurationCopy: IConfiguration;
 
   constructor(
     private readonly httpClient: IHttpClient,
@@ -25,14 +26,15 @@ export default class ConfigurationRequestor {
       this.banditVariationConfigurationStore,
       this.banditModelConfigurationStore,
     );
+    this.configurationCopy = this.configuration.copy();
   }
 
   public isFlagConfigExpired(): Promise<boolean> {
     return this.flagConfigurationStore.isExpired();
   }
 
   public getConfiguration(): IConfiguration {
-    return this.configuration;
+    return this.configurationCopy;
   }
 
   async fetchAndStoreConfigurations(): Promise<void> {
@@ -92,6 +94,7 @@ export default class ConfigurationRequestor {
         banditModelPacket,
       )
     ) {
+      this.configurationCopy = this.configuration.copy();
       // TODO: Notify that config updated.
     }
   }