admin/everything-claude-code
1
0
Fork 0
mirror of https://github.com/affaan-m/everything-claude-code.git synced 2026-02-16 19:33:11 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix: use execFileSync with input option for Windows-compatible stdin tests

Browse Source 
Windows cmd.exe treats single quotes literally, so `echo '...' | node -e '...'`
fails. Switched to execFileSync with the `input` option to pipe stdin data
directly without shell quoting issues.
This commit is contained in:
Affaan Mustafa
2026-02-12 17:14:03 -08:00
parent 8248310181
commit c0c54d0dae
1 changed files with 12 additions and 24 deletions
Download Patch File Download Diff File Expand all files Collapse all files

36
tests/lib/utils.test.js
View File

@@ -627,46 +627,34 @@ function runTests() {
})) passed++; else failed++;
// readStdinJson tests (via subprocess — safe hardcoded inputs)
// Use execFileSync with input option instead of shell echo|pipe for Windows compat
console.log('\nreadStdinJson():');
const stdinScript = 'const u=require("./scripts/lib/utils");u.readStdinJson({timeoutMs:2000}).then(d=>{process.stdout.write(JSON.stringify(d))})';
const stdinOpts = { encoding: 'utf8', cwd: path.join(__dirname, '..', '..'), timeout: 5000 };
if (test('readStdinJson parses valid JSON from stdin', () => {
const { execSync } = require('child_process');
const script = 'const u=require("./scripts/lib/utils");u.readStdinJson({timeoutMs:2000}).then(d=>{process.stdout.write(JSON.stringify(d))})';
const result = execSync(
`echo '{"tool_input":{"command":"ls"}}' | node -e '${script}'`,
{ encoding: 'utf8', cwd: path.join(__dirname, '..', '..'), timeout: 5000 }
);
const { execFileSync } = require('child_process');
const result = execFileSync('node', ['-e', stdinScript], { ...stdinOpts, input: '{"tool_input":{"command":"ls"}}' });
const parsed = JSON.parse(result);
assert.deepStrictEqual(parsed, { tool_input: { command: 'ls' } });
})) passed++; else failed++;
if (test('readStdinJson returns {} for invalid JSON', () => {
const { execSync } = require('child_process');
const script = 'const u=require("./scripts/lib/utils");u.readStdinJson({timeoutMs:2000}).then(d=>{process.stdout.write(JSON.stringify(d))})';
const result = execSync(
`echo 'not json' | node -e '${script}'`,
{ encoding: 'utf8', cwd: path.join(__dirname, '..', '..'), timeout: 5000 }
);
const { execFileSync } = require('child_process');
const result = execFileSync('node', ['-e', stdinScript], { ...stdinOpts, input: 'not json' });
assert.deepStrictEqual(JSON.parse(result), {});
})) passed++; else failed++;
if (test('readStdinJson returns {} for empty stdin', () => {
const { execSync } = require('child_process');
const script = 'const u=require("./scripts/lib/utils");u.readStdinJson({timeoutMs:2000}).then(d=>{process.stdout.write(JSON.stringify(d))})';
const result = execSync(
`echo '' | node -e '${script}'`,
{ encoding: 'utf8', cwd: path.join(__dirname, '..', '..'), timeout: 5000 }
);
const { execFileSync } = require('child_process');
const result = execFileSync('node', ['-e', stdinScript], { ...stdinOpts, input: '' });
assert.deepStrictEqual(JSON.parse(result), {});
})) passed++; else failed++;
if (test('readStdinJson handles nested objects', () => {
const { execSync } = require('child_process');
const script = 'const u=require("./scripts/lib/utils");u.readStdinJson({timeoutMs:2000}).then(d=>{process.stdout.write(JSON.stringify(d))})';
const result = execSync(
`echo '{"a":{"b":1},"c":[1,2]}' | node -e '${script}'`,
{ encoding: 'utf8', cwd: path.join(__dirname, '..', '..'), timeout: 5000 }
);
const { execFileSync } = require('child_process');
const result = execFileSync('node', ['-e', stdinScript], { ...stdinOpts, input: '{"a":{"b":1},"c":[1,2]}' });
const parsed = JSON.parse(result);
assert.deepStrictEqual(parsed, { a: { b: 1 }, c: [1, 2] });
})) passed++; else failed++;