fix(ci): use a local test server for fetch test (#207)

Author: vados-cosmonic

package-lock.json

@@ -8,7 +8,7 @@ import {
 } from '@bytecodealliance/jco';
 import { spawnSync } from 'node:child_process';
 import { tmpdir } from 'node:os';
-import { resolve, join, dirname } from 'node:path';
+import { join, dirname } from 'node:path';
 import { readFile, writeFile, mkdir, rm } from 'node:fs/promises';
 import { rmSync, existsSync } from 'node:fs';
 import { createHash } from 'node:crypto';
@@ -17,26 +17,13 @@ import {
   stubWasi,
 } from '../lib/spidermonkey-embedding-splicer.js';
 import { fileURLToPath } from 'node:url';
-import { cwd, stdout, platform } from 'node:process';
+import { cwd, stdout } from 'node:process';
+
+import { maybeWindowsPath } from './platform.js';
+
 export const { version } = JSON.parse(
   await readFile(new URL('../package.json', import.meta.url), 'utf8'),
 );
-const isWindows = platform === 'win32';
-
-function maybeWindowsPath(path) {
-  if (!path) return path;
-  const resolvedPath = resolve(path);
-  if (!isWindows) return resolvedPath;
-
-  // Strip any existing UNC prefix check both the format we add as well as what
-  //  the windows API returns when using path.resolve
-  let cleanPath = resolvedPath;
-  while (cleanPath.startsWith('\\\\?\\') || cleanPath.startsWith('//?/')) {
-    cleanPath = cleanPath.substring(4);
-  }
-
-  return '//?/' + cleanPath.replace(/\\/g, '/');
-}
 
 /**
  * Clean up the given input string by removing the given patterns if

src/componentize.js

@@ -0,0 +1,19 @@
+import { resolve } from 'node:path';
+import { platform } from 'node:process';
+
+export const isWindows = platform === 'win32';
+
+export function maybeWindowsPath(path) {
+  if (!path) return path;
+  const resolvedPath = resolve(path);
+  if (!isWindows) return resolvedPath;
+
+  // Strip any existing UNC prefix check both the format we add as well as what
+  //  the windows API returns when using path.resolve
+  let cleanPath = resolvedPath;
+  while (cleanPath.startsWith('\\\\?\\') || cleanPath.startsWith('//?/')) {
+    cleanPath = cleanPath.substring(4);
+  }
+
+  return '//?/' + cleanPath.replace(/\\/g, '/');
+}

src/platform.js

@@ -1,8 +1,18 @@
+import { URL, fileURLToPath } from 'node:url';
+import { createServer } from 'node:http';
+
 import { strictEqual, ok } from 'node:assert';
 
+import { maybeWindowsPath } from '../../src/platform.js';
+import { getRandomPort } from '../util.js';
+
+const FETCH_URL = 'http://localhost';
+
+const port = await getRandomPort();
+
 export const source = `
   export async function run () {
-    const res = await fetch('https://httpbin.org/anything');
+    const res = await fetch('${FETCH_URL}:${port}');
     const source = await res.json();
     console.log(source.url);
   }
@@ -14,7 +24,34 @@ export const source = `
 export const enableFeatures = ['http'];
 
 export async function test(run) {
+  const url = `${FETCH_URL}:${port}`;
+
+  // Run a local server on some port
+  const server = createServer(async (req, res) => {
+    res.writeHead(200, { 'Content-Type': 'application/json; charset=utf-8' });
+    res.write(
+      JSON.stringify({
+        status: 'ok',
+        url,
+      }),
+    );
+    res.end();
+  }).listen(port);
+
+  // Wait until the server is ready
+  let ready = false;
+  while (!ready) {
+    try {
+      const res = await fetch(url);
+      ready = true;
+    } catch (err) {
+      await new Promise((resolve) => setTimeout(resolve, 250));
+    }
+  }
+
   const { stdout, stderr } = await run();
   strictEqual(stderr, '');
-  strictEqual(stdout.trim(), 'https://httpbin.org/anything');
+  strictEqual(stdout.trim(), url);
+
+  server.close();
 }

test/builtins/fetch.js

@@ -0,0 +1,13 @@
+import { createServer } from 'node:net';
+
+// Utility function for getting a random port
+export async function getRandomPort() {
+  return await new Promise((resolve) => {
+    const server = createServer();
+    server.listen(0, function () {
+      const port = this.address().port;
+      server.on('close', () => resolve(port));
+      server.close();
+    });
+  });
+}