Refactor: Update Vitest configuration and enhance test setup

This commit updates the Vitest configuration to use Cloudflare Workers settings and improves the global fetch mock for API testing. It also removes the outdated GitHub Actions workflow and the script for converting tests to Vitest, streamlining the testing process. Additionally, the setup for Miniflare tests has been enhanced to better simulate the Cloudflare Workers environment.

Author: AaronDDM

.github/workflows/pull-reqeust.yml .github/workflows/pull-request.yml.github/workflows/pull-reqeust.yml renamed to .github/workflows/pull-request.yml

@@ -1,18 +1,106 @@
 import { vi } from 'vitest';
 
-// Mock fetch globally for Vitest
-global.fetch = vi.fn();
+// Mock fetch for API testing - this will be used by the SDK to make HTTP requests
+(global.fetch as any) = vi.fn().mockResolvedValue({
+  ok: true,
+  status: 200,
+  statusText: 'OK',
+  headers: new Map(),
+  json: vi.fn().mockResolvedValue({}),
+  text: vi.fn().mockResolvedValue(''),
+  arrayBuffer: vi.fn().mockResolvedValue(new ArrayBuffer(0)),
+  blob: vi.fn().mockResolvedValue(new Blob()),
+  formData: vi.fn().mockResolvedValue(new FormData()),
+  clone: vi.fn().mockReturnThis(),
+  body: null,
+  bodyUsed: false,
+  url: '',
+  type: 'basic',
+  redirected: false,
+});
 
-// Cloudflare Workers environment setup
-// This simulates the Cloudflare Workers environment for testing
-if (process.env.CLOUDFLARE_WORKERS === 'true') {
-  // Set up Cloudflare Workers globals
-  globalThis.CloudflareWorkers = true;
+// Store the original Request constructor
+const OriginalRequest = globalThis.Request;
 
-  // Mock Cloudflare Workers specific APIs if needed
-  // globalThis.caches = new Map();
-  // globalThis.crypto = require('crypto');
-}
+// Mock Request constructor to add headers.raw() method for compatibility with node-fetch tests
+globalThis.Request = class MockRequest extends OriginalRequest {
+  constructor(url: string | URL, options?: RequestInit) {
+    super(url, options);
+
+    // Store original header names to preserve casing
+    const originalHeaderNames: Record<string, string> = {};
+    if (options?.headers) {
+      if (options.headers instanceof Headers) {
+        options.headers.forEach((_value, key) => {
+          originalHeaderNames[key.toLowerCase()] = key;
+        });
+      } else if (Array.isArray(options.headers)) {
+        options.headers.forEach(([key]) => {
+          originalHeaderNames[key.toLowerCase()] = key;
+        });
+      } else {
+        Object.entries(options.headers).forEach(([key]) => {
+          originalHeaderNames[key.toLowerCase()] = key;
+        });
+      }
+    }
+
+    // Add raw() method to headers for compatibility with node-fetch
+    const originalHeaders = this.headers;
+    (originalHeaders as any).raw = (): Record<string, string[]> => {
+      const rawHeaders: Record<string, string[]> = {};
+      originalHeaders.forEach((value, key) => {
+        // Use original casing if available, otherwise use the key as-is
+        const originalKey = originalHeaderNames[key.toLowerCase()] || key;
+        rawHeaders[originalKey] = [value];
+      });
+      return rawHeaders;
+    };
+  }
+} as any;
+
+// Mock mime-types for Cloudflare Workers compatibility
+// This is needed because mime-types uses CommonJS which doesn't work in Workers
+vi.mock('mime-types', () => ({
+  default: {
+    lookup: vi.fn((filePath: string) => {
+      // Simple mock implementation for common file types
+      const ext = filePath.split('.').pop()?.toLowerCase();
+      const mimeTypes: Record<string, string> = {
+        txt: 'text/plain',
+        html: 'text/html',
+        css: 'text/css',
+        js: 'application/javascript',
+        json: 'application/json',
+        png: 'image/png',
+        jpg: 'image/jpeg',
+        jpeg: 'image/jpeg',
+        gif: 'image/gif',
+        pdf: 'application/pdf',
+        zip: 'application/zip',
+      };
+      return mimeTypes[ext || ''] || 'application/octet-stream';
+    }),
+  },
+  lookup: vi.fn((filePath: string) => {
+    // Simple mock implementation for common file types
+    const ext = filePath.split('.').pop()?.toLowerCase();
+    const mimeTypes: Record<string, string> = {
+      txt: 'text/plain',
+      html: 'text/html',
+      css: 'text/css',
+      js: 'application/javascript',
+      json: 'application/json',
+      png: 'image/png',
+      jpg: 'image/jpeg',
+      jpeg: 'image/jpeg',
+      gif: 'image/gif',
+      pdf: 'application/pdf',
+      zip: 'application/zip',
+    };
+    return mimeTypes[ext || ''] || 'application/octet-stream';
+  }),
+}));
 
 // Export for TypeScript module resolution
 export {};

scripts/convertTestsToVitest.js

@@ -1,9 +1,8 @@
-import { defineConfig } from 'vitest/config';
+import { defineWorkersConfig } from '@cloudflare/vitest-pool-workers/config';
 
-export default defineConfig({
+export default defineWorkersConfig({
   test: {
     globals: true,
-    environment: 'node',
     setupFiles: ['./tests/setupMiniflareTests.ts'],
     coverage: {
       provider: 'v8',
@@ -17,9 +16,16 @@ export default defineConfig({
     clearMocks: true,
     mockReset: true,
     restoreMocks: true,
-    // Environment variables for Cloudflare Workers testing
-    env: {
-      CLOUDFLARE_WORKERS: 'true',
+    poolOptions: {
+      workers: {
+        miniflare: {
+          // Miniflare configuration for Cloudflare Workers environment
+          compatibilityDate: '2023-05-18',
+          compatibilityFlags: ['nodejs_compat'],
+          // Enable Node.js compatibility for SDK dependencies
+          bindings: {},
+        },
+      },
     },
   },
   resolve: {
@@ -34,4 +40,22 @@ export default defineConfig({
   esbuild: {
     target: 'node16',
   },
+  // Configure Vite to handle CommonJS modules properly
+  optimizeDeps: {
+    include: ['mime-types', 'mime-db'],
+  },
+  // Bundle Node.js dependencies for Workers environment
+  ssr: {
+    external: [],
+    noExternal: [
+      'node-fetch',
+      'mime-types',
+      'mime-db',
+      'form-data-encoder',
+      'formdata-node',
+    ],
+    optimizeDeps: {
+      include: ['mime-types', 'mime-db'],
+    },
+  },
 });