Skip to content

Commit 1ea699c

Browse files
dorlugasigalCopilot
andcommitted
fix(test): correct actionLogs spawn opts assertion for cross-platform CI
The shell option is always present in spawn opts (true on Windows, false elsewhere). Updated deepStrictEqual to expect shell: os.platform() === 'win32'. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent fd12994 commit 1ea699c

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

test/cli/service.test.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -520,7 +520,7 @@ describe('actionLogs', () => {
520520
assert.deepStrictEqual(spawnCalls[0].args, ['logs', 'termbeam', '--lines', '200']);
521521
assert.deepStrictEqual(spawnCalls[0].opts, {
522522
stdio: 'inherit',
523-
...(os.platform() === 'win32' ? { shell: true } : {}),
523+
shell: os.platform() === 'win32',
524524
});
525525
});
526526

0 commit comments

Comments
 (0)