mirrors/bun.sh
1
0
Fork 0
mirror of https://github.com/oven-sh/bun synced 2026-02-09 10:28:47 +00:00
Code Issues Packages Projects Releases Wiki Activity

test: use -e inline scripts instead of tempDir where possible

Browse Source 
Replace tempDir file creation with -e inline script execution for the
first three test cases to avoid filesystem I/O. The fourth test case
(bun test) still requires tempDir since bun test needs a file to run.

Note: bunExe() returns process.execPath which points to the debug build
when running via `bun bd test`, so spawned processes already use the
correct binary without needing to add "bd" as a subcommand.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This commit is contained in:
Claude Bot
2026-02-04 01:26:21 +00:00
parent 4a6f4280f0
commit bf992ecb2c
1 changed files with 17 additions and 48 deletions
Download Patch File Download Diff File Expand all files Collapse all files

65
test/regression/issue/26724.test.ts
View File

@@ -13,21 +13,14 @@ describe("issue #26724: verbose fetch logging should not be enabled by default",
},
});
using dir = tempDir("issue-26724", {
"test.ts": `
const url = process.argv[2];
const response = await fetch(url);
console.log("status:", response.status);
`,
});
const script = `const response = await fetch("http://localhost:${server.port}/get"); console.log("status:", response.status);`;
await using proc = Bun.spawn({
cmd: [bunExe(), "run", "test.ts", `http://localhost:${server.port}/get`],
cmd: [bunExe(), "-e", script],
env: {
...bunEnv,
BUN_CONFIG_VERBOSE_FETCH: undefined, // Explicitly ensure it's not set
},
cwd: String(dir),
stdout: "pipe",
stderr: "pipe",
});
@@ -52,21 +45,14 @@ describe("issue #26724: verbose fetch logging should not be enabled by default",
},
});
using dir = tempDir("issue-26724-verbose", {
"test.ts": `
const url = process.argv[2];
const response = await fetch(url);
console.log("status:", response.status);
`,
});
const script = `const response = await fetch("http://localhost:${server.port}/get"); console.log("status:", response.status);`;
await using proc = Bun.spawn({
cmd: [bunExe(), "run", "test.ts", `http://localhost:${server.port}/get`],
cmd: [bunExe(), "-e", script],
env: {
...bunEnv,
BUN_CONFIG_VERBOSE_FETCH: "1",
},
cwd: String(dir),
stdout: "pipe",
stderr: "pipe",
});
@@ -90,42 +76,24 @@ describe("issue #26724: verbose fetch logging should not be enabled by default",
},
});
using dir = tempDir("issue-26724-nodehttp", {
"test.ts": `
import http from 'node:http';
const port = parseInt(process.argv[2], 10);
const options = {
hostname: 'localhost',
port: port,
path: '/get',
method: 'GET',
};
const req = http.request(options, (res) => {
let data = '';
res.on('data', (chunk) => { data += chunk; });
res.on('end', () => {
console.log('status:', res.statusCode);
});
});
req.on('error', (err) => {
console.error('error:', err.message);
});
req.end();
`,
});
const script = `
import http from 'node:http';
const options = { hostname: 'localhost', port: ${server.port}, path: '/get', method: 'GET' };
const req = http.request(options, (res) => {
let data = '';
res.on('data', (chunk) => { data += chunk; });
res.on('end', () => { console.log('status:', res.statusCode); });
});
req.on('error', (err) => { console.error('error:', err.message); });
req.end();
`;
await using proc = Bun.spawn({
cmd: [bunExe(), "run", "test.ts", String(server.port)],
cmd: [bunExe(), "-e", script],
env: {
...bunEnv,
BUN_CONFIG_VERBOSE_FETCH: undefined, // Explicitly ensure it's not set
},
cwd: String(dir),
stdout: "pipe",
stderr: "pipe",
});
@@ -150,6 +118,7 @@ describe("issue #26724: verbose fetch logging should not be enabled by default",
},
});
// This test requires a file since bun test needs a test file to run
using dir = tempDir("issue-26724-buntest", {
"fetch.test.ts": `
import { test, expect } from "bun:test";