Call onError if API request fails

Flagsmith · Oct 10, 2024 · 464c3ed · 464c3ed
1 parent db9e757
commit 464c3ed
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 4 deletions.
diff --git a/flagsmith-core.ts b/flagsmith-core.ts
@@ -479,9 +479,10 @@ const Flagsmith = class {
                                     }
                                     if (shouldFetchFlags) {
                                         // We want to resolve init since we have cached flags
-                                        this.getFlags().catch((e) => {
-                                            this.log("Exception fetching cached logs", e);
-                                        });
+
+                                        this.getFlags().catch((error) => {
+                                            this.onError?.(error)
+                                        })
                                     }
                                 } else {
                                     if (!preventFetch) {

diff --git a/test/init.test.ts b/test/init.test.ts
@@ -1,4 +1,5 @@
 // Sample test
+import { waitFor } from '@testing-library/react';
 import { defaultState, environmentID, getFlagsmith, getStateToCheck, identityState } from './test-constants';
 import { promises as fs } from 'fs'
 
@@ -149,17 +150,24 @@ describe('Flagsmith.init', () => {
               })}),
         )
     });
-    test('should not reject when the API cannot be reached but the cache is populated', async () => {
+    test('should not reject but call onError, when the API cannot be reached with the cache populated', async () => {
+        const onError = jest.fn()
         const { flagsmith, initConfig, AsyncStorage } = getFlagsmith({
             cacheFlags: true,
             fetch: async () => {
                 return Promise.resolve({ text: () => Promise.resolve('Mocked fetch error'), ok: false, status: 401 });
             },
+            onError,
         });
         await AsyncStorage.setItem('BULLET_TRAIN_DB', JSON.stringify(defaultState));
         await flagsmith.init(initConfig);
 
         expect(getStateToCheck(flagsmith.getState())).toEqual(defaultState);
+
+        await waitFor(() => {
+            expect(onError).toHaveBeenCalledTimes(1)
+            expect(onError).toHaveBeenCalledWith(new Error('Mocked fetch error'))
+        })
     })
     test('should not reject when the API cannot be reached but default flags are set', async () => {
         const { flagsmith, initConfig } = getFlagsmith({