@ripplo/testing 0.5.5 → 0.5.6

package/dist/lockfile.js

@@ -611,6 +611,12 @@ async function hashFixturesIntoCompileResult({
   return { ...result, fixtures: Object.fromEntries(hashed) };
 }
 async function hashOneFixture({ fixturesRoot, name }) {
+  const rawName = name;
+  if (typeof rawName !== "string" || rawName.length === 0) {
+    throw new Error(
+      `Internal error: upload step produced a non-string fixture name (got ${rawName === null ? "null" : typeof rawName}). This usually means a test passed a non-Fixture value to upload() \u2014 wrap the filename with fixture("file.png").`
+    );
+  }
   if (name.includes("..") || path.isAbsolute(name)) {
     throw new Error(`Invalid fixture name "${name}": must be a path under .ripplo/fixtures/`);
   }
@@ -634,10 +640,19 @@ async function hashOneFixture({ fixturesRoot, name }) {
 function collectFixtureReferences(result) {
   const names = /* @__PURE__ */ new Set();
   result.tests.forEach((test) => {
-    Object.values(test.spec.nodes).forEach((node) => {
-      if (node.type === "upload") {
-        node.files.forEach((name) => names.add(name));
+    Object.entries(test.spec.nodes).forEach(([stepId, node]) => {
+      if (node.type !== "upload") {
+        return;
       }
+      node.files.forEach((name, i) => {
+        const raw = name;
+        if (typeof raw !== "string" || raw.length === 0) {
+          throw new Error(
+            `Test "${test.slug}" step "${stepId}" upload files[${String(i)}] is not a non-empty string (got ${raw === null ? "null" : typeof raw}). Wrap filenames with fixture(): upload(loc, fixture("file.png")).`
+          );
+        }
+        names.add(raw);
+      });
     });
   });
   return names;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@ripplo/testing",
   "description": "TypeScript DSL for defining and running Ripplo e2e workflow tests",
-  "version": "0.5.5",
+  "version": "0.5.6",
   "type": "module",
   "files": [
     "dist"
@@ -99,8 +99,8 @@
     "tsup": "^8.5.1",
     "typescript": "catalog:",
     "vitest": "^4.1.4",
-    "@ripplo/spec": "^0.0.0",
-    "@ripplo/eslint-config": "0.0.0"
+    "@ripplo/eslint-config": "0.0.0",
+    "@ripplo/spec": "^0.0.0"
   },
   "peerDependencies": {
     "@nestjs/common": "^10.0.0 || ^11.0.0",