test: increase timeouts and delays for CI stability with low resources

Author: jkyberneees

.github/workflows/tests.yaml

@@ -18,4 +18,4 @@ jobs:
         run: bun run format
 
       - name: Run tests
-        run: bun run test
+        run: bun test --coverage tests/*.test.ts --timeout=120000

tests/dos-prevention.test.ts

@@ -123,7 +123,7 @@ describe("DoS and Resource Exhaustion Security Tests", () => {
     test("should demonstrate timeout importance for DoS prevention", () => {
       // Long timeouts can be exploited for resource exhaustion
       const longTimeout = 300000 // 5 minutes - too long
-      const reasonableTimeout = 30000 // 30 seconds - reasonable
+      const reasonableTimeout = 60000 // 60 seconds - reasonable for CI with low resources
 
       expect(longTimeout).toBeGreaterThan(reasonableTimeout)
 

tests/enhanced-hooks.test.ts

@@ -30,7 +30,7 @@ describe("Enhanced Hook Naming Conventions", () => {
   beforeEach(() => {
     proxy = new FetchProxy({
       base: "https://api.example.com",
-      timeout: 5000,
+      timeout: 15000, // Increased timeout for CI with low resources
     })
 
     mockResponse = new Response(JSON.stringify({ success: true }), {
@@ -62,7 +62,7 @@ describe("Enhanced Hook Naming Conventions", () => {
     it("should handle async beforeRequest hooks", async () => {
       let hookExecuted = false
       const beforeRequestHook = async (req: Request) => {
-        await new Promise((resolve) => setTimeout(resolve, 10))
+        await new Promise((resolve) => setTimeout(resolve, 25)) // Increased delay for CI
         hookExecuted = true
       }
 
@@ -168,7 +168,9 @@ describe("Enhanced Hook Naming Conventions", () => {
       // Add some delay to the fetch
       mockFetch.mockImplementationOnce(
         () =>
-          new Promise((resolve) => setTimeout(() => resolve(mockResponse), 50)),
+          new Promise((resolve) =>
+            setTimeout(() => resolve(mockResponse), 100),
+          ), // Increased delay for CI
       )
 
       const options: ProxyRequestOptions = {

tests/http-method-validation.test.ts

@@ -43,7 +43,7 @@ describe("HTTP Method Validation", () => {
         // Server not ready yet
       }
       retries++
-      await new Promise((resolve) => setTimeout(resolve, 150)) // Increased delay
+      await new Promise((resolve) => setTimeout(resolve, 250)) // Increased delay for CI with low resources
     }
 
     if (!serverReady) {

tests/index.test.ts

@@ -75,7 +75,7 @@ describe("fetch-gate", () => {
         // Server not ready yet
       }
       retries++
-      await new Promise((resolve) => setTimeout(resolve, 150)) // Increased delay
+      await new Promise((resolve) => setTimeout(resolve, 250)) // Increased delay for CI with low resources
     }
 
     if (!serverReady) {
@@ -100,10 +100,10 @@ describe("fetch-gate", () => {
     it("should create proxy instance with custom options", () => {
       const { proxy, getCircuitBreakerState } = createFetchGate({
         base: "https://api.example.com",
-        timeout: 5000,
+        timeout: 15000, // Increased timeout for CI with low resources
         circuitBreaker: {
           failureThreshold: 3,
-          resetTimeout: 30000,
+          resetTimeout: 60000, // Increased reset timeout for CI
         },
       })
 
@@ -182,7 +182,7 @@ describe("fetch-gate", () => {
     it("should handle timeouts", async () => {
       const proxyInstance = new FetchProxy({
         base: baseUrl,
-        timeout: 80, // Slightly longer timeout for CI stability
+        timeout: 80, // Keep original timeout for timeout test functionality
       })
 
       const req = new Request("http://example.com/test")
@@ -215,7 +215,7 @@ describe("fetch-gate", () => {
         base: baseUrl,
         circuitBreaker: {
           failureThreshold: 2,
-          resetTimeout: 1000,
+          resetTimeout: 2000, // Increased for CI with low resources
           enabled: true,
         },
       })
@@ -347,7 +347,7 @@ describe("fetch-gate", () => {
 
       const circuitBreaker = new CircuitBreaker({
         failureThreshold: 1,
-        resetTimeout: 100,
+        resetTimeout: 200, // Increased for CI with low resources
       })
 
       // Trigger failure to open the circuit
@@ -369,7 +369,7 @@ describe("fetch-gate", () => {
     it("should reset failures after successful execution in HALF_OPEN state", async () => {
       const circuitBreaker = new CircuitBreaker({
         failureThreshold: 1,
-        resetTimeout: 150, // Slightly longer for CI stability
+        resetTimeout: 300, // Increased for CI with low resources
       })
 
       // Trigger failure to open the circuit
@@ -380,7 +380,7 @@ describe("fetch-gate", () => {
       expect(circuitBreaker.getState()).toBe(CircuitState.OPEN)
 
       // Wait for reset timeout with a bit of buffer
-      await new Promise((resolve) => setTimeout(resolve, 200))
+      await new Promise((resolve) => setTimeout(resolve, 350)) // Increased wait time for CI
 
       // Execute a successful request
       await expect(

tests/logging.test.ts

@@ -148,14 +148,14 @@ describe("Logging Integration", () => {
     })
 
     it("should log request start events", () => {
-      const context = { requestId: "test-123", timeout: 5000 }
+      const context = { requestId: "test-123", timeout: 15000 } // Increased timeout for CI
 
       proxyLogger.logRequestStart(request, context)
 
       expect(mockLogger.info).toHaveBeenCalledWith(
         expect.objectContaining({
           requestId: "test-123",
-          timeout: 5000,
+          timeout: 15000, // Increased timeout for CI
           event: "request_start",
         }),
         expect.stringContaining("Starting GET request"),
@@ -272,14 +272,14 @@ describe("Logging Integration", () => {
     })
 
     it("should log timeout events", () => {
-      proxyLogger.logTimeout(request, 5000)
+      proxyLogger.logTimeout(request, 15000) // Increased timeout for CI
 
       expect(mockLogger.warn).toHaveBeenCalledWith(
         expect.objectContaining({
-          timeout: 5000,
+          timeout: 15000, // Increased timeout for CI
           event: "request_timeout",
         }),
-        expect.stringContaining("Request timed out after 5000ms"),
+        expect.stringContaining("Request timed out after 15000ms"),
       )
     })
 

tests/proxy-fallback.test.ts

@@ -27,7 +27,7 @@ describe("Proxy Fallback Response", () => {
   beforeEach(() => {
     proxy = new FetchProxy({
       base: "https://api.example.com",
-      timeout: 5000,
+      timeout: 15000, // Increased timeout for CI with low resources
     })
     mockFetch.mockClear()
   })
@@ -72,7 +72,7 @@ describe("Proxy Fallback Response", () => {
 
       const onErrorHook = jest.fn().mockImplementation(async (req, error) => {
         // Simulate async fallback logic
-        await new Promise((resolve) => setTimeout(resolve, 10))
+        await new Promise((resolve) => setTimeout(resolve, 25)) // Increased delay for CI
 
         return new Response(
           JSON.stringify({
@@ -173,7 +173,7 @@ describe("Proxy Fallback Response", () => {
         base: "https://api.example.com",
         circuitBreaker: {
           failureThreshold: 1,
-          resetTimeout: 1000,
+          resetTimeout: 2000, // Increased for CI with low resources
           enabled: true,
         },
       })

tests/query-injection.test.ts

@@ -33,7 +33,7 @@ beforeAll(async () => {
       if (response.ok) break
     } catch (e) {
       if (i === 19) throw new Error("Test server failed to start")
-      await new Promise((resolve) => setTimeout(resolve, 150))
+      await new Promise((resolve) => setTimeout(resolve, 250)) // Increased delay for CI with low resources
     }
   }
 })