fix linter

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

Author: thomaspoignant

libs/providers/go-feature-flag/package.json

@@ -17,4 +17,4 @@
     "@ljharb/eslint-config": "^21.2.0",
     "eslint-plugin-tree-shaking": "^1.12.2"
   }
-}
+}

libs/providers/go-feature-flag/project.json

@@ -29,15 +29,11 @@
     },
     "lint": {
       "executor": "@nx/eslint:lint",
-      "outputs": [
-        "{options.outputFile}"
-      ]
+      "outputs": ["{options.outputFile}"]
     },
     "test": {
       "executor": "@nx/jest:jest",
-      "outputs": [
-        "{workspaceRoot}/coverage/libs/providers/go-feature-flag"
-      ],
+      "outputs": ["{workspaceRoot}/coverage/libs/providers/go-feature-flag"],
       "options": {
         "jestConfig": "libs/providers/go-feature-flag/jest.config.ts"
       },
@@ -49,9 +45,7 @@
     },
     "package": {
       "executor": "@nx/rollup:rollup",
-      "outputs": [
-        "{options.outputPath}"
-      ],
+      "outputs": ["{options.outputPath}"],
       "dependsOn": [
         {
           "target": "copy-wasm"
@@ -66,10 +60,7 @@
         "generateExportsField": true,
         "umdName": "go-feature-flag",
         "external": "all",
-        "format": [
-          "cjs",
-          "esm"
-        ],
+        "format": ["cjs", "esm"],
         "assets": [
           {
             "glob": "package.json",
@@ -91,4 +82,4 @@
     }
   },
   "tags": []
-}
+}

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

@@ -9,96 +9,7 @@ import {
   ImpossibleToSendDataToTheCollectorException,
   InvalidOptionsException,
 } from '../exception';
-
-// Mock Response class
-class MockResponse {
-  public status: number;
-  public headers: Headers;
-  public body: string;
-  public ok: boolean;
-
-  constructor(status: number, body = '', headers: Record<string, string> = {}) {
-    this.status = status;
-    this.body = body;
-    this.headers = new Headers(headers);
-    this.ok = status >= 200 && status < 300;
-  }
-
-  async text(): Promise<string> {
-    return this.body;
-  }
-
-  async json(): Promise<any> {
-    return JSON.parse(this.body);
-  }
-}
-
-// Mock fetch implementation
-class MockFetch {
-  private responses: Map<string, MockResponse> = new Map();
-  private lastRequest?: {
-    url: string;
-    options: RequestInit;
-  };
-
-  setResponse(url: string, response: MockResponse): void {
-    this.responses.set(url, response);
-  }
-
-  setResponseByStatus(status: string, response: MockResponse): void {
-    this.responses.set(status, response);
-  }
-
-  getLastRequest() {
-    return this.lastRequest;
-  }
-
-  reset(): void {
-    this.responses.clear();
-    this.lastRequest = undefined;
-  }
-
-  async fetch(url: string, options: RequestInit = {}): Promise<Response> {
-    this.lastRequest = { url, options };
-
-    // Handle AbortSignal for timeout tests
-    if (options.signal) {
-      const signal = options.signal as AbortSignal;
-      if (signal.aborted) {
-        throw new Error('Request aborted');
-      }
-
-      // For timeout tests, we'll simulate a delay and then check if aborted
-      if (url.includes('timeout')) {
-        await new Promise((resolve) => setTimeout(resolve, 100));
-        if (signal.aborted) {
-          throw new Error('Request aborted');
-        }
-      }
-    }
-
-    // Check if we have a specific response for this URL
-    if (this.responses.has(url)) {
-      return this.responses.get(url)! 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;
-    }
-
-    // Check if we have a response by status code in the responses map
-    for (const [key, response] of this.responses.entries()) {
-      if (key.match(/^\d{3}$/) && response.status === parseInt(key)) {
-        return response as unknown as Response;
-      }
-    }
-
-    // Default response
-    return new MockResponse(200, '{}') as unknown as Response;
-  }
-}
+import { MockFetch, MockResponse } from '../testutil/mock-fetch';
 
 describe('GoFeatureFlagApi', () => {
   let mockFetch: MockFetch;

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

@@ -0,0 +1,89 @@
+// Mock Response class
+export class MockResponse {
+  public status: number;
+  public headers: Headers;
+  public body: string;
+  public ok: boolean;
+
+  constructor(status: number, body = '', headers: Record<string, string> = {}) {
+    this.status = status;
+    this.body = body;
+    this.headers = new Headers(headers);
+    this.ok = status >= 200 && status < 300;
+  }
+
+  async text(): Promise<string> {
+    return this.body;
+  }
+
+  async json(): Promise<any> {
+    return JSON.parse(this.body);
+  }
+}
+
+// Mock fetch implementation
+export class MockFetch {
+  private responses: Map<string, MockResponse> = new Map();
+  private lastRequest?: {
+    url: string;
+    options: RequestInit;
+  };
+
+  setResponse(url: string, response: MockResponse): void {
+    this.responses.set(url, response);
+  }
+
+  setResponseByStatus(status: string, response: MockResponse): void {
+    this.responses.set(status, response);
+  }
+
+  getLastRequest() {
+    return this.lastRequest;
+  }
+
+  reset(): void {
+    this.responses.clear();
+    this.lastRequest = undefined;
+  }
+
+  async fetch(url: string, options: RequestInit = {}): Promise<Response> {
+    this.lastRequest = { url, options };
+
+    // Handle AbortSignal for timeout tests
+    if (options.signal) {
+      const signal = options.signal as AbortSignal;
+      if (signal.aborted) {
+        throw new Error('Request aborted');
+      }
+
+      // For timeout tests, we'll simulate a delay and then check if aborted
+      if (url.includes('timeout')) {
+        await new Promise((resolve) => setTimeout(resolve, 100));
+        if (signal.aborted) {
+          throw new Error('Request aborted');
+        }
+      }
+    }
+
+    // Check if we have a specific response for this URL
+    if (this.responses.has(url)) {
+      return this.responses.get(url)! 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;
+    }
+
+    // Check if we have a response by status code in the responses map
+    for (const [key, response] of this.responses.entries()) {
+      if (key.match(/^\d{3}$/) && response.status === parseInt(key)) {
+        return response as unknown as Response;
+      }
+    }
+
+    // Default response
+    return new MockResponse(200, '{}') as unknown as Response;
+  }
+}

package.json

@@ -100,3 +100,4 @@
     "verdaccio": "^5.0.4"
   }
 }
+