Update unit tests

Author: DefCon-007

test/apiUtils.test.ts

@@ -105,49 +105,6 @@ describe('apiUtils', () => {
       expect(result).toBe(successResponse)
     })
 
-    it('should throw error after max retries', async () => {
-      // Set up multiple error responses
-      const errorResponse = new Response('Service Unavailable', {
-        status: 503,
-        statusText: 'Service Unavailable',
-      })
-
-      vi.mocked(global.fetch).mockResolvedValue(errorResponse)
-
-      // Mock setTimeout to speed up tests
-      vi.useFakeTimers()
-
-      // Start the fetch with retry
-      const resultPromise = fetchWithRetry(
-        'https://api.example.com/test',
-        {},
-        {
-          initialDelayMs: 10,
-          maxRetries: 2,
-        }
-      )
-
-      // Fast-forward timer for all retries
-      await vi.runAllTimersAsync()
-
-      // Verify it throws after max retries
-      try {
-        await resultPromise
-        // If we get here, the test should fail
-        expect('Promise should have been rejected').toBe('but it was resolved')
-      } catch (error) {
-        // We expect an error to be thrown
-        expect(error).toBeInstanceOf(Error)
-        expect((error as Error).message).toContain('HTTP error 503')
-      }
-
-      // Restore timers
-      vi.useRealTimers()
-
-      // Should be called initial + maxRetries times
-      expect(global.fetch).toHaveBeenCalledTimes(3)
-    })
-
     it('should retry on network errors', async () => {
       // Set up a network error followed by a successful response
       vi.mocked(global.fetch)

test/k6helper.test.ts

@@ -6,11 +6,11 @@ import {
   executeRunK6Command,
   extractTestRunId,
   fetchChecks,
+  fetchTestRunSummary,
   generateK6RunCommand,
   isCloudIntegrationEnabled,
   validateTestPaths,
 } from '../src/k6helper'
-import { generateMarkdownSummary } from '../src/markdownRenderer'
 import { TestRunUrlsMap } from '../src/types'
 
 // Mock child_process.spawn
@@ -29,6 +29,7 @@ vi.mock('child_process', () => ({
     stderr: { on: vi.fn() },
     pid: 123,
   })),
+  execSync: vi.fn().mockReturnValue(Buffer.from('k6 v0.38.0')),
 }))
 
 // Mock @actions/core
@@ -248,50 +249,52 @@ describe('extractTestRunId', () => {
   })
 })
 
-describe('generateMarkdownSummary', () => {
-  it('should return a default message for null metrics', () => {
-    expect(generateMarkdownSummary(null, null)).toBe(
-      'No metrics data available.'
-    )
-  })
-
-  it('should return a default message for undefined metrics', () => {
-    expect(generateMarkdownSummary(undefined, null)).toBe(
-      'No metrics data available.'
-    )
+describe('fetchTestRunSummary', () => {
+  beforeEach(() => {
+    vi.resetAllMocks()
   })
 
-  it('should return a default message for empty metrics', () => {
-    const emptyMetrics = {
-      http_metric_summary: null,
-      ws_metric_summary: null,
-      grpc_metric_summary: null,
-      checks_metric_summary: null,
-      thresholds_summary: null,
-      browser_metric_summary: null,
+  it('should return test run summary when API request succeeds', async () => {
+    // Mock response from the API
+    const mockTestRunSummary = {
+      metrics_summary: {
+        http_metric_summary: {
+          requests: 100,
+          failed_requests: 5,
+        },
+        checks_metric_summary: {
+          total: 200,
+          successes: 190,
+        },
+      },
+      baseline_test_run_details: null,
     }
 
-    expect(generateMarkdownSummary(emptyMetrics, null)).toBe(
-      'No metrics data available.'
+    // Mock the apiRequest function to return our mock response
+    vi.mocked(apiRequest).mockResolvedValueOnce(mockTestRunSummary)
+
+    // Call the function
+    const result = await fetchTestRunSummary('1234')
+
+    // Verify the result
+    expect(result).toEqual(mockTestRunSummary)
+    expect(apiRequest).toHaveBeenCalledWith(
+      expect.stringContaining('/test_runs(1234)/result_summary')
     )
   })
 
-  it('should display checks summary when present', () => {
-    const metrics = {
-      http_metric_summary: null,
-      ws_metric_summary: null,
-      grpc_metric_summary: null,
-      checks_metric_summary: {
-        total: 10,
-        successes: 8,
-      },
-      thresholds_summary: null,
-      browser_metric_summary: null,
-    }
+  it('should return undefined when API request fails', async () => {
+    // Mock the apiRequest function to return undefined (API failure)
+    vi.mocked(apiRequest).mockResolvedValueOnce(undefined)
 
-    const result = generateMarkdownSummary(metrics, null)
+    // Call the function
+    const result = await fetchTestRunSummary('1234')
 
-    expect(result).toContain('checks were not successful')
+    // Verify the result
+    expect(result).toBeUndefined()
+    expect(apiRequest).toHaveBeenCalledWith(
+      expect.stringContaining('/test_runs(1234)/result_summary')
+    )
   })
 })