un async bootstrap :)

Author: typotter

src/client/eppo-client-with-bandits.spec.ts

@@ -151,7 +151,7 @@ describe('EppoClient Bandits E2E test', () => {
         client.setIsGracefulFailureMode(false);
 
         // Bootstrap using the bandit flag config.
-        await client.bootstrap(banditFlagsConfig);
+        client.bootstrap(banditFlagsConfig);
       });
 
       it.each(Object.keys(testCases))('Shared bandit test case - %s', async (fileName: string) => {

src/client/eppo-client.spec.ts

@@ -365,14 +365,14 @@ describe('EppoClient E2E test', () => {
 
       describe('Not obfuscated', () => {
         let client: EppoClient;
-        beforeAll(async () => {
+        beforeAll(() => {
           client = new EppoClient({
             flagConfigurationStore: new MemoryOnlyConfigurationStore(),
           });
           client.setIsGracefulFailureMode(false);
 
           // Bootstrap using the flags config.
-          await client.bootstrap(bootstrapFlagsConfig);
+          client.bootstrap(bootstrapFlagsConfig);
         });
 
         it('contains some key flags', () => {
@@ -389,14 +389,14 @@ describe('EppoClient E2E test', () => {
 
       describe('Obfuscated', () => {
         let client: EppoClient;
-        beforeAll(async () => {
+        beforeAll(() => {
           client = new EppoClient({
             flagConfigurationStore: new MemoryOnlyConfigurationStore(),
           });
           client.setIsGracefulFailureMode(false);
 
           // Bootstrap using the obfuscated flags config.
-          await client.bootstrap(bootstrapFlagsObfuscatedConfig);
+          client.bootstrap(bootstrapFlagsObfuscatedConfig);
         });
 
         it('contains some key flags', () => {

src/client/eppo-client.ts

@@ -326,7 +326,7 @@ export default class EppoClient {
    * accommodate writing to a persistent store. For fastest initialization, (at the cost of persisting configuration),
    * use `bootstrap` in conjunction with `MemoryOnlyConfigurationStore` instances which won't do an async write.
    */
-  async bootstrap(configuration: IConfigurationWire): Promise<void> {
+  bootstrap(configuration: IConfigurationWire): void {
     if (!configuration.config) {
       throw new Error('Flag configuration not provided');
     }
@@ -340,7 +340,8 @@ export default class EppoClient {
     // This method runs async because the configuration stores potentially have an async write at the end of updating
     // the configuration. Most instances of offlineInit will use `MemoryOnlyConfigurationStore` instances which actually
     // accomplish the config write synchronously.
-    await this.configurationManager.hydrateConfigurationStoresFromUfc(
+    // `void` keyword here suppresses warnings about leaving the promise hanging.
+    void this.configurationManager.hydrateConfigurationStoresFromUfc(
       flagConfigResponse,
       banditParamResponse,
     );