From 115d9eb66a5276feb1148097e7217cb5202c531c Mon Sep 17 00:00:00 2001 From: Mikola Lysenko Date: Fri, 9 Oct 2026 13:10:36 -0400 Subject: [PATCH] test(patch): use get instead of the removed download alias socket-patch v5 removes the `download` alias of `get`. Call `patch get` in the dry-run integration test, and check that the command exits 0 and forwards `get express` to socket-patch. The old check accepted any exit code, so a usage error also passed. --- test/integration/binary/js.test.mts | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/test/integration/binary/js.test.mts b/test/integration/binary/js.test.mts index 653c0cd5a..7ded5b1f3 100644 --- a/test/integration/binary/js.test.mts +++ b/test/integration/binary/js.test.mts @@ -1727,21 +1727,22 @@ if (BINARY.enabled) { ) it.skipIf(!ENV.RUN_INTEGRATION_TESTS)( - 'should handle patch download --dry-run', + 'should handle patch get --dry-run', async () => { if (!binaryExists) { return } const result = await executeCliCommand( - ['patch', 'download', '--dry-run', '--config', '{}', 'express'], + ['patch', 'get', '--dry-run', '--config', '{}', 'express'], { binPath: BINARY.path, isolateConfig: false, }, ) - expect(result.code).toBeGreaterThanOrEqual(0) + expect(result.code).toBe(0) + expect(result.stderr).toContain('Arguments: get express') }, )