From f61c9b0cafca73c550ef17bcc8e22edb7a0f25c6 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 18 Feb 2026 13:42:05 +0000 Subject: [PATCH] Fix integration/hooks tests to handle Windows platform differences Co-authored-by: pangerlkr <73515951+pangerlkr@users.noreply.github.com> --- tests/integration/hooks.test.js | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/tests/integration/hooks.test.js b/tests/integration/hooks.test.js index 88f43260..6bf0f064 100644 --- a/tests/integration/hooks.test.js +++ b/tests/integration/hooks.test.js @@ -262,8 +262,13 @@ async function runTests() { }); }); - assert.ok(stderr.includes('BLOCKED'), 'Blocking hook should output BLOCKED'); - assert.strictEqual(code, 2, 'Blocking hook should exit with code 2'); + // Hook only blocks on non-Windows platforms (tmux is Unix-only) + if (process.platform === 'win32') { + assert.strictEqual(code, 0, 'On Windows, hook should not block (exit 0)'); + } else { + assert.ok(stderr.includes('BLOCKED'), 'Blocking hook should output BLOCKED'); + assert.strictEqual(code, 2, 'Blocking hook should exit with code 2'); + } })) passed++; else failed++; // ========================================== @@ -298,7 +303,12 @@ async function runTests() { }); }); - assert.strictEqual(code, 2, 'Blocking hook should exit 2'); + // Hook only blocks on non-Windows platforms (tmux is Unix-only) + if (process.platform === 'win32') { + assert.strictEqual(code, 0, 'On Windows, hook should not block (exit 0)'); + } else { + assert.strictEqual(code, 2, 'Blocking hook should exit 2'); + } })) passed++; else failed++; if (await asyncTest('hooks handle missing files gracefully', async () => {