From 44af64418a66562e6c4e4cd02e80e6f96f90376a Mon Sep 17 00:00:00 2001 From: Jill Regan Date: Sun, 22 Feb 2026 12:37:21 -0500 Subject: [PATCH] Update unit test --- src/utils.test.ts | 16 ++++++++-------- src/utils.ts | 5 +++-- 2 files changed, 11 insertions(+), 10 deletions(-) diff --git a/src/utils.test.ts b/src/utils.test.ts index 576ae9c..e3f03ef 100644 --- a/src/utils.test.ts +++ b/src/utils.test.ts @@ -835,21 +835,21 @@ describe("findMatchingFieldAndFile", () => { describe("findSectionIdsByQuery", () => { it("throws when sections is empty", () => { expect(() => findSectionIdsByQuery([], "section-1")).toThrow( - /section section-1 could not be found/, + /Item has no sections; cannot resolve section "section-1"/, ); }); it("throws when sections is null/undefined", () => { expect(() => findSectionIdsByQuery(undefined as unknown as FullItem["sections"], "x"), - ).toThrow(/could not be found/); + ).toThrow(/Item has no sections; cannot resolve section "x"/); }); - it("returns section id when section matches by id", () => { - const sections = [{ id: "sec-1", label: "Section 1" }]; - expect( - findSectionIdsByQuery(sections as FullItem["sections"], "sec-1"), - ).toEqual(["sec-1"]); + it("throws when section query matches no section", () => { + const sections = [{ id: "sec-1", label: "Other" }]; + expect(() => + findSectionIdsByQuery(sections as FullItem["sections"], "nonexistent"), + ).toThrow(/No section matching "nonexistent" found in specified item/); }); it("returns section id when section matches by label", () => { @@ -863,7 +863,7 @@ describe("findSectionIdsByQuery", () => { const sections = [{ id: "sec-1", label: "Other" }]; expect(() => findSectionIdsByQuery(sections as FullItem["sections"], "nonexistent"), - ).toThrow(/could not be found/); + ).toThrow(/No section matching "nonexistent" found in specified item/); }); it("returns multiple ids when multiple sections match", () => { diff --git a/src/utils.ts b/src/utils.ts index 3534833..d1f1752 100644 --- a/src/utils.ts +++ b/src/utils.ts @@ -113,7 +113,6 @@ const getFileContentWithRetry = async ( itemId: string, fileId: string, ): Promise => { - const maxAttempts = 3; const retryDelayMs = 2000; for (let attempt = 1; attempt <= maxAttempts; attempt++) { @@ -299,7 +298,9 @@ const validateSecretRefs = (envNames: string[]): void => { // Throw an error if any secret references are invalid if (invalid.length > 0) { - const details = invalid.map(({ name, message }) => `${name}: ${message}`).join("; "); + const details = invalid + .map(({ name, message }) => `${name}: ${message}`) + .join("; "); throw new Error(`Invalid secret reference(s): ${details}`); } };