From 2bd58b5cfad4cf5079f4aa2e2b29220d87fa2f8e Mon Sep 17 00:00:00 2001 From: Shakker Date: Tue, 12 May 2026 18:48:33 +0100 Subject: [PATCH] test: distinguish allowed value summaries --- src/config/allowed-values.test.ts | 17 +++++++++++------ 1 file changed, 11 insertions(+), 6 deletions(-) diff --git a/src/config/allowed-values.test.ts b/src/config/allowed-values.test.ts index addb8d2fe33..805d441d991 100644 --- a/src/config/allowed-values.test.ts +++ b/src/config/allowed-values.test.ts @@ -4,16 +4,21 @@ import { summarizeAllowedValues } from "./allowed-values.js"; describe("summarizeAllowedValues", () => { it("does not collapse mixed-type entries that stringify similarly", () => { const summary = summarizeAllowedValues([1, "1", 1, "1"]); - expect(summary?.hiddenCount).toBe(0); - expect(summary?.formatted).toContain('1, "1"'); - expect(summary?.values).toHaveLength(2); + expect(summary).toStrictEqual({ + formatted: '1, "1"', + hiddenCount: 0, + values: ["1", "1"], + }); }); it("keeps distinct long values even when labels truncate the same way", () => { const prefix = "a".repeat(200); const summary = summarizeAllowedValues([`${prefix}x`, `${prefix}y`]); - expect(summary?.hiddenCount).toBe(0); - expect(summary?.values).toHaveLength(2); - expect(summary?.values[0]).not.toBe(summary?.values[1]); + expect(summary).toStrictEqual({ + formatted: + '"aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa... (+41 chars)", "aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa... (+41 chars)"', + hiddenCount: 0, + values: [`${prefix}x`, `${prefix}y`], + }); }); });