From 1cd736afd2f7a0a609ed2ebcdb03535659367578 Mon Sep 17 00:00:00 2001 From: Peter Steinberger Date: Tue, 12 May 2026 23:57:58 +0100 Subject: [PATCH] test: dedupe update cli mock reads --- src/cli/update-cli.option-collisions.test.ts | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/cli/update-cli.option-collisions.test.ts b/src/cli/update-cli.option-collisions.test.ts index 90994d2658d..eb5a04986af 100644 --- a/src/cli/update-cli.option-collisions.test.ts +++ b/src/cli/update-cli.option-collisions.test.ts @@ -34,6 +34,10 @@ vi.mock("../runtime.js", () => ({ defaultRuntime: mocks.defaultRuntime, })); +function firstCallOptions(mock: { mock: { calls: unknown[][] } }) { + return mock.mock.calls[0]?.[0]; +} + describe("update cli option collisions", () => { beforeEach(() => { updateCommand.mockClear(); @@ -52,7 +56,7 @@ describe("update cli option collisions", () => { argv: ["update", "status", "--json", "--timeout", "9"], assert: () => { expect(updateStatusCommand).toHaveBeenCalledTimes(1); - const [opts] = updateStatusCommand.mock.calls.at(0) ?? []; + const opts = firstCallOptions(updateStatusCommand); expect((opts as { json?: boolean; timeout?: string } | undefined)?.json).toBe(true); expect((opts as { json?: boolean; timeout?: string } | undefined)?.timeout).toBe("9"); }, @@ -62,7 +66,7 @@ describe("update cli option collisions", () => { argv: ["update", "wizard", "--timeout", "13"], assert: () => { expect(updateWizardCommand).toHaveBeenCalledTimes(1); - const [opts] = updateWizardCommand.mock.calls.at(0) ?? []; + const opts = firstCallOptions(updateWizardCommand); expect((opts as { timeout?: string } | undefined)?.timeout).toBe("13"); }, },