From ced1ff617a21927f9626ebe32eff05a3a6b01f62 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E4=BE=9D=E7=91=AA=E8=B2=93?= Date: Tue, 10 Mar 2026 09:30:45 +0800 Subject: [PATCH] Replace void with expect().toBeDefined() for getter side-effect access (S3735) Co-Authored-By: Claude Opus 4.6 --- tests/stores/allMapData.test.js | 4 ++-- tests/stores/conformance.test.js | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/stores/allMapData.test.js b/tests/stores/allMapData.test.js index 8c2da60..eebc05c 100644 --- a/tests/stores/allMapData.test.js +++ b/tests/stores/allMapData.test.js @@ -236,8 +236,8 @@ describe("allMapDataStore", () => { store.allFilterAttrs = [ { type: "date", min: originalDate, max: "2023-12-31T23:59:00Z" }, ]; - // Access the getter once - void store.filterAttrs; + // Access the getter once to trigger any side effects + expect(store.filterAttrs).toBeDefined(); // The original state should still contain the ISO date, not a formatted string expect(store.allFilterAttrs[0].min).toBe(originalDate); }); diff --git a/tests/stores/conformance.test.js b/tests/stores/conformance.test.js index 14d078a..b2a0873 100644 --- a/tests/stores/conformance.test.js +++ b/tests/stores/conformance.test.js @@ -204,7 +204,7 @@ describe("conformanceStore", () => { attributes: [], }, ]; - void store.cases; + expect(store.cases).toBeDefined(); // Original state should still contain the ISO date expect(store.allCases[0].started_at).toBe(originalDate); });