fix warning

Signed-off-by: Thomas Poignant <[email protected]>

Author: thomaspoignant

libs/providers/go-feature-flag/src/lib/service/api.test.ts

@@ -48,9 +48,9 @@ describe('GoFeatureFlagApi', () => {
       await api.retrieveFlagConfiguration();
 
       const request = mockFetch.getLastRequest();
-      expect(request!.url).toBe('http://localhost:8080/v1/flag/configuration');
-      expect(request!.options.method).toBe('POST');
-      expect(request!.options.body).toBe(JSON.stringify({ flags: [] }));
+      expect(request?.url).toBe('http://localhost:8080/v1/flag/configuration');
+      expect(request?.options.method).toBe('POST');
+      expect(request?.options.body).toEqual(JSON.stringify({ flags: [] }));
     });
 
     it('should include API key in authorization header when provided', async () => {
@@ -64,7 +64,7 @@ describe('GoFeatureFlagApi', () => {
       await api.retrieveFlagConfiguration();
 
       const request = mockFetch.getLastRequest();
-      expect(request!.options.headers).toHaveProperty('Authorization', 'Bearer my-api-key');
+      expect(request?.options.headers).toHaveProperty('Authorization', 'Bearer my-api-key');
     });
 
     it('should not include authorization header when API key is not provided', async () => {

libs/providers/go-feature-flag/src/lib/testutil/mock-fetch.ts

@@ -67,13 +67,23 @@ export class MockFetch {
 
     // Check if we have a specific response for this URL
     if (this.responses.has(url)) {
-      return this.responses.get(url)! as unknown as Response;
+      const response = this.responses.get(url);
+      if (typeof response === 'function') {
+        // Allow for dynamic response functions (for advanced mocking)
+        return (response as any)(url, options) as Response;
+      }
+      return response as unknown as Response;
     }
 
     // Check if we have a response by status code
     const statusMatch = url.match(/(\d{3})/);
     if (statusMatch && this.responses.has(statusMatch[1])) {
-      return this.responses.get(statusMatch[1])! as unknown as Response;
+      const response = this.responses.get(statusMatch[1]);
+      if (typeof response === 'function') {
+        // Allow for dynamic response functions (for advanced mocking)
+        return (response as any)(url, options) as Response;
+      }
+      return response as unknown as Response;
     }
 
     // Check if we have a response by status code in the responses map