From ab3df344b834e54a2299d1e0782d8b742e1cdbc8 Mon Sep 17 00:00:00 2001 From: Jarred Sumner Date: Fri, 23 Jan 2026 23:22:32 -0800 Subject: [PATCH] Delete slop test --- test/regression/issue/26394.test.ts | 86 ----------------------------- 1 file changed, 86 deletions(-) delete mode 100644 test/regression/issue/26394.test.ts diff --git a/test/regression/issue/26394.test.ts b/test/regression/issue/26394.test.ts deleted file mode 100644 index b186ee7f55..0000000000 --- a/test/regression/issue/26394.test.ts +++ /dev/null @@ -1,86 +0,0 @@ -import { expect, test } from "bun:test"; - -// https://github.com/oven-sh/bun/issues/26394 -// Race condition in Bun.serve() where requests can arrive before routes are fully registered, -// causing the default "Welcome to Bun!" response instead of the configured handler's response. - -test("concurrent Bun.serve instances should not return Welcome to Bun", async () => { - const serverCount = 60; - const servers: ReturnType[] = []; - - try { - // Create many servers concurrently - for (let i = 0; i < serverCount; i++) { - servers.push( - Bun.serve({ - port: 0, - fetch: () => new Response("OK"), - }), - ); - } - - // Make concurrent requests to all servers - const responses = await Promise.all( - servers.map(async server => { - const res = await fetch(`http://127.0.0.1:${server.port}/`); - return res.text(); - }), - ); - - // Verify no "Welcome to Bun!" responses - check for both debug mode message and production mode - for (let i = 0; i < responses.length; i++) { - expect(responses[i]).not.toContain("Welcome to Bun"); - expect(responses[i]).not.toBe(""); // Production mode returns empty for renderMissing - expect(responses[i]).toBe("OK"); - } - } finally { - // Clean up - guaranteed to run even if assertions fail - for (const server of servers) { - server.stop(); - } - } -}); - -test("Bun.serve should be ready to handle requests immediately after returning", async () => { - // Test a single server with immediate fetch - this tests if the server is ready synchronously - using server = Bun.serve({ - port: 0, - fetch: () => new Response("handler response"), - }); - - // Immediately fetch - if there's a race condition, this might return "Welcome to Bun!" - const response = await fetch(`http://127.0.0.1:${server.port}/`); - const text = await response.text(); - - expect(text).toBe("handler response"); -}); - -test("multiple sequential Bun.serve instances with immediate requests", async () => { - // Create servers sequentially and immediately request from each - const results: string[] = []; - const servers: ReturnType[] = []; - - try { - for (let i = 0; i < 20; i++) { - const server = Bun.serve({ - port: 0, - fetch: () => new Response(`server-${i}`), - }); - servers.push(server); - - // Immediately fetch from the server - const response = await fetch(`http://127.0.0.1:${server.port}/`); - results.push(await response.text()); - } - - // Verify all responses match expected - for (let i = 0; i < results.length; i++) { - expect(results[i]).toBe(`server-${i}`); - } - } finally { - // Clean up - guaranteed to run even if assertions fail - for (const server of servers) { - server.stop(); - } - } -});