@specific.dev/spectest 0.85.0 → 0.85.1

package/dist/coverage.js

@@ -57,10 +57,14 @@ export function validateCoverage(service, cov) {
     const obj = typeof cov === "object" && cov !== null ? cov : null;
     const adapters = obj?.adapters;
     const reports = obj?.reports;
-    const knownKeys = obj !== null && Object.keys(obj).every((k) => k === "adapters" || k === "reports");
+    // `explains` is what `applyCoverageAdapters` adds for the wire; the
+    // config is validated again after that, so it must pass here.
+    const explains = obj?.explains;
+    const knownKeys = obj !== null && Object.keys(obj).every((k) => k === "adapters" || k === "reports" || k === "explains");
     const okReports = reports === undefined || reports === "delta" || reports === "cumulative";
     const okAdapters = adapters === undefined || Array.isArray(adapters);
-    if (obj !== null && knownKeys && okReports && okAdapters && (adapters !== undefined || reports !== undefined)) {
+    const okExplains = explains === undefined || (Array.isArray(explains) && explains.every((e) => typeof e === "string"));
+    if (obj !== null && knownKeys && okReports && okAdapters && okExplains && (adapters !== undefined || reports !== undefined)) {
         for (const a of adapters ?? []) {
             const ok = typeof a === "object" &&
                 a !== null &&

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@specific.dev/spectest",
-  "version": "0.85.0",
+  "version": "0.85.1",
   "description": "Spectest SDK for defining test environments in TypeScript.",
   "type": "module",
   "main": "./dist/index.js",

package/src/coverage.test.ts

@@ -65,6 +65,12 @@ describe("validateCoverage", () => {
 });
 
 describe("applyCoverageAdapters", () => {
+  test("the applied (wire) form validates again, as defineEnvironment does after applying", () => {
+    const out = applyCoverageAdapters("api", { ...base, coverage: { adapters: [node()] } });
+    expect(() => validateCoverage("api", out.coverage)).not.toThrow();
+    expect(() => validateCoverage("api", JSON.parse(JSON.stringify(out.coverage)))).toThrow(); // names only: not a config
+    expect(() => validateCoverage("api", { adapters: [node()], explains: [1] })).toThrow();
+  });
   test("node: appends NODE_OPTIONS, sets the coverage dir env, mounts the hook — never NODE_V8_COVERAGE", () => {
     const out = applyCoverageAdapters("api", { ...base, coverage: { adapters: [node()] } });
     expect(out.env?.NODE_OPTIONS).toBe(`--max-old-space-size=512 --require ${NODE_COVERAGE_HOOK_PATH}`);

package/src/coverage.ts

@@ -184,10 +184,14 @@ export function validateCoverage(service: string, cov: unknown): void {
   const obj = typeof cov === "object" && cov !== null ? (cov as Record<string, unknown>) : null;
   const adapters = obj?.adapters;
   const reports = obj?.reports;
-  const knownKeys = obj !== null && Object.keys(obj).every((k) => k === "adapters" || k === "reports");
+  // `explains` is what `applyCoverageAdapters` adds for the wire; the
+  // config is validated again after that, so it must pass here.
+  const explains = obj?.explains;
+  const knownKeys = obj !== null && Object.keys(obj).every((k) => k === "adapters" || k === "reports" || k === "explains");
   const okReports = reports === undefined || reports === "delta" || reports === "cumulative";
   const okAdapters = adapters === undefined || Array.isArray(adapters);
-  if (obj !== null && knownKeys && okReports && okAdapters && (adapters !== undefined || reports !== undefined)) {
+  const okExplains = explains === undefined || (Array.isArray(explains) && explains.every((e) => typeof e === "string"));
+  if (obj !== null && knownKeys && okReports && okAdapters && okExplains && (adapters !== undefined || reports !== undefined)) {
     for (const a of (adapters as unknown[] | undefined) ?? []) {
       const ok =
         typeof a === "object" &&