From 2a9eb71976f0a55820d46421e4e54f6fd5bff95a Mon Sep 17 00:00:00 2001 From: erik Date: Sat, 11 Jul 2026 00:30:49 -0500 Subject: [PATCH] fix: Refactor function calls in JobExtractionWizard to remove unnecessary void usage. Enhance test setup by adding mock implementations for update checking and installation processes, improving test coverage for application updates. --- .../src/renderer/modules/extraction/JobExtractionWizard.tsx | 4 ++-- services/desktop_app/src/renderer/test/setup.ts | 6 ++++++ 2 files changed, 8 insertions(+), 2 deletions(-) diff --git a/services/desktop_app/src/renderer/modules/extraction/JobExtractionWizard.tsx b/services/desktop_app/src/renderer/modules/extraction/JobExtractionWizard.tsx index 3a58981..f0a7503 100644 --- a/services/desktop_app/src/renderer/modules/extraction/JobExtractionWizard.tsx +++ b/services/desktop_app/src/renderer/modules/extraction/JobExtractionWizard.tsx @@ -529,8 +529,8 @@ export function JobExtractionWizard({ error={error} statusText={statusText} selectRoot={selectRoot} - discoverSchema={() => void discoverSchema()} - rediscoverWithOtherSample={() => void rediscoverWithOtherSample()} + discoverSchema={() => discoverSchema()} + rediscoverWithOtherSample={() => rediscoverWithOtherSample()} discoverySampleLabels={discoverySampleLabels} canRediscoverWithOtherSample={ form.mode === "general" diff --git a/services/desktop_app/src/renderer/test/setup.ts b/services/desktop_app/src/renderer/test/setup.ts index 7aa4ccd..87f8456 100644 --- a/services/desktop_app/src/renderer/test/setup.ts +++ b/services/desktop_app/src/renderer/test/setup.ts @@ -13,5 +13,11 @@ beforeEach(() => { { radicado: "R1", filename: "factura.pdf", docType: "Factura", localPath: "C:\\docs\\R1\\factura.pdf", size: 10 }, ]), readFile: vi.fn().mockResolvedValue({ filename: "factura.pdf", size: 10, data: new ArrayBuffer(8) }), + checkForUpdates: vi.fn().mockResolvedValue({ + updateAvailable: false, + currentVersion: "1.0.0", + }), + installUpdate: vi.fn().mockResolvedValue({ ok: true }), + onUpdateProgress: vi.fn().mockReturnValue(() => undefined), }; });