Moved the shared parse-excel test helper back under packages/excel-parser/src/__tests__/helpers/parse-excel.ts and tightened packages/excel-parser/jest.config.ts so only real *.test.ts / *.spec.ts files under __tests__ are discovered by Jest. This keeps helpers colocated with tests without executing them as suites.

Also fixed temp workbook filename collisions in the shared helper by switching to randomUUID()-based filenames, which resolved intermittent failures in the full npm run test:unit --workspace=@supplier-config/excel-parser run.

Author: m-houston

packages/excel-parser/jest.config.ts

@@ -13,7 +13,7 @@ const config: Config = {
   coverageDirectory: "./.reports/unit/coverage",
 
   // Indicates which provider should be used to instrument code for coverage
-  coverageProvider: "v8",
+  coverageProvider: "babel",
 
   coverageThreshold: {
     global: {
@@ -48,7 +48,7 @@ const config: Config = {
 
   testEnvironment: "node",
   testPathIgnorePatterns: ["/node_modules/", "/dist/"],
-  testMatch: ["**/__tests__/**/*.ts", "**/?(*.)+(spec|test).ts"],
+  testMatch: ["**/__tests__/**/*.test.ts", "**/__tests__/**/*.spec.ts"],
   moduleFileExtensions: ["ts", "js", "json", "node"],
 
   transform: {

…l-parser/src/test-helpers/parse-excel.ts …ser/src/__tests__/helpers/parse-excel.tspackages/excel-parser/src/test-helpers/parse-excel.ts renamed to packages/excel-parser/src/__tests__/helpers/parse-excel.ts packages/excel-parser/src/test-helpers/parse-excel.ts renamed to packages/excel-parser/src/__tests__/helpers/parse-excel.ts

@@ -1,8 +1,9 @@
+import { randomUUID } from "node:crypto";
 import * as XLSX from "xlsx";
 import { tmpdir } from "node:os";
 import path from "node:path";
 
-import { parseExcelFile } from "../parse-excel";
+import { parseExcelFile } from "../../parse-excel";
 
 export interface WorkbookData {
   packs?: Record<string, unknown>[];
@@ -180,7 +181,7 @@ export function buildWorkbookOmitting(omit: string): XLSX.WorkBook {
 }
 
 export function writeWorkbook(workbook: XLSX.WorkBook): string {
-  const filePath = path.join(tmpdir(), `test-${Date.now()}.xlsx`);
+  const filePath = path.join(tmpdir(), `test-${randomUUID()}.xlsx`);
   XLSX.writeFile(workbook, filePath);
   return filePath;
 }

packages/excel-parser/src/__tests__/parse-excel.letter-variants.test.ts

@@ -2,7 +2,7 @@ import {
   makePackRow,
   makeVariantRow,
   parseWorkbook,
-} from "../test-helpers/parse-excel";
+} from "./helpers/parse-excel";
 
 describe("parse-excel letter variants", () => {
   it("parses canonical enum values", () => {

packages/excel-parser/src/__tests__/parse-excel.pack-specifications.test.ts

@@ -2,7 +2,7 @@ import {
   makePackRow,
   makeVariantRow,
   parseWorkbook,
-} from "../test-helpers/parse-excel";
+} from "./helpers/parse-excel";
 
 describe("parse-excel pack specifications", () => {
   it("throws on invalid postage size", () => {

packages/excel-parser/src/__tests__/parse-excel.suppliers.test.ts

@@ -5,7 +5,7 @@ import {
   makeSupplierRow,
   makeVolumeGroupRow,
   parseWorkbook,
-} from "../test-helpers/parse-excel";
+} from "./helpers/parse-excel";
 
 describe("parse-excel suppliers, allocations, and supplier packs", () => {
   it("parses Suppliers of all channel types", () => {

packages/excel-parser/src/__tests__/parse-excel.test.ts

@@ -1,8 +1,5 @@
 import { parseExcelFile } from "../parse-excel";
-import {
-  buildWorkbookOmitting,
-  writeWorkbook,
-} from "../test-helpers/parse-excel";
+import { buildWorkbookOmitting, writeWorkbook } from "./helpers/parse-excel";
 
 describe("parse-excel workbook structure", () => {
   it("throws when LetterVariant sheet is missing", () => {

packages/excel-parser/src/__tests__/parse-excel.volume-groups.test.ts

@@ -2,7 +2,7 @@ import {
   makePackRow,
   makeVariantRow,
   parseWorkbook,
-} from "../test-helpers/parse-excel";
+} from "./helpers/parse-excel";
 
 describe("parse-excel volume groups", () => {
   it("parses VolumeGroup with description and endDate", () => {