From 8b57d0fe9ef612881fd62e4d5a0074448bb03dbd Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Fri, 8 May 2026 11:21:56 +0100 Subject: [PATCH] test: clarify update cli exit assertions --- src/cli/update-cli.test.ts | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/cli/update-cli.test.ts b/src/cli/update-cli.test.ts index 47b2bd67d04..4a3a33b3870 100644 --- a/src/cli/update-cli.test.ts +++ b/src/cli/update-cli.test.ts @@ -1429,7 +1429,7 @@ describe("update-cli", () => { await updateCommand({ dryRun: true }); }, assert: () => { - expect(vi.mocked(defaultRuntime.exit).mock.calls.some((call) => call[0] === 1)).toBe(false); + expect(defaultRuntime.exit).not.toHaveBeenCalledWith(1); expect(runGatewayUpdate).not.toHaveBeenCalled(); }, }, @@ -3298,9 +3298,11 @@ describe("update-cli", () => { .mocked(defaultRuntime.error) .mock.calls.some((call) => String(call[0]).includes("Downgrade confirmation required.")); expect(downgradeMessageSeen).toBe(shouldExit); - expect(vi.mocked(defaultRuntime.exit).mock.calls.some((call) => call[0] === 1)).toBe( - shouldExit, - ); + if (shouldExit) { + expect(defaultRuntime.exit).toHaveBeenCalledWith(1); + } else { + expect(defaultRuntime.exit).not.toHaveBeenCalledWith(1); + } expect(vi.mocked(runGatewayUpdate).mock.calls.length > 0).toBe(false); expect( vi