From ad1554a36f0beb9b94d8c31a92f4cb3c9b48d2fd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marc-Andr=C3=A9=20Bombeck?= Date: Sat, 1 Aug 2026 17:56:20 +0200 Subject: [PATCH] Make the round-trip case actually round-trip The flip from an authored recipe back to the defaults was written with the retired pillar as the thing left out. With that pillar gone, both halves of the case sent the same full list, so the first read false where it expected true and the flip was never exercised at all. It now takes out a pillar that still exists, and says at the assertion why the two halves have to differ. --- tests/integration/health-score-configured-flag.test.ts | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/tests/integration/health-score-configured-flag.test.ts b/tests/integration/health-score-configured-flag.test.ts index ecce1f999..b36609a20 100644 --- a/tests/integration/health-score-configured-flag.test.ts +++ b/tests/integration/health-score-configured-flag.test.ts @@ -361,6 +361,9 @@ describe("the resolved configured flag, on every wire that carries it", () => { expectEveryWire(await readWires(), false); + // Narrowed: one pillar taken out. If both halves of this case send the + // same list, the round trip proves nothing — it would read false at both + // ends and the flip would never be exercised. await saveSelection([ "BLOOD_PRESSURE", "GLYCAEMIA", @@ -368,7 +371,6 @@ describe("the resolved configured flag, on every wire that carries it", () => { "SLEEP", "ADIPOSITY", "WELLBEING", - "LIPIDS", ]); expectEveryWire(await readWires(), true);