@norskvideo/ctl-test-harness 0.1.27 → 0.1.28

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@norskvideo/ctl-test-harness",
-  "version": "0.1.27",
+  "version": "0.1.28",
   "type": "module",
   "exports": {
     ".": {

package/smoke.js

@@ -187,6 +187,7 @@ export async function runProductSmoke(slug, spec, deps = defaultSmokeDeps) {
     };
     let daemon = null;
     let launched = false;
+    let registered = false;
     let handles = [];
     let journeyError;
     try {
@@ -226,6 +227,7 @@ export async function runProductSmoke(slug, spec, deps = defaultSmokeDeps) {
         if (added.name !== spec.product.name) {
             throw new Error(`product add registered '${added.name}', expected '${spec.product.name}'`);
         }
+        registered = true;
         // Template.
         let templateName;
         if ("name" in spec.template) {
@@ -327,6 +329,14 @@ export async function runProductSmoke(slug, spec, deps = defaultSmokeDeps) {
     }
     if (handles.length)
         await deps.stopSources(handles).catch(() => { });
+    // The control plane is a singleton that deliberately outlives its daemon, so
+    // stopping the daemon strands it holding 127.0.0.1:14321 and every later job
+    // on that runner fails to bind. Ask the daemon to remove it while there is
+    // still a daemon to ask. Best-effort: a teardown that throws here would mask
+    // the journey's own error, and a SIGKILLed job never reaches this line at all
+    // -- the workflow's reap covers that half.
+    if (registered)
+        await cli(["product", "remove", spec.product.name]).catch(() => { });
     // cleanupDaemon ends with rmSync(storeDir); Docker's root-owned bind-mount
     // targets make that EACCES on a non-root runner. Expected: the root nuke
     // below is what removes them. Only a store that survives the nuke fails.