@bongos/core 1.19.398 → 1.19.400

package/tests/provisioning_teardown_intent.mjs

@@ -30,6 +30,9 @@ import http from 'node:http';
 const require = createRequire(import.meta.url);
 const provisioning = require('../modules/provisioning/provisioning.js');
 
+// The actions a teardown supersedes, mirroring enqueueTeardown's SQL.
+const SUPERSEDABLE = new Set(['provision', 'settings-apply', 'restart']);
+
 let passed = 0, failed = 0;
 function t(name, fn) {
   try { fn(); passed++; console.log(`  PASS  ${name}`); }
@@ -65,12 +68,13 @@ function fakeIntentDb(seed = []) {
       }
       if (/UPDATE provisioning_intents/.test(sql) && /state = 'error'/.test(sql)) {
         // The supersede's guard must live in the SQL itself: pending-only (a running
-        // intent is the runner's mid-flight work) and provision-or-settings-push only
-        // (task 1003140 widened it: a queued push into a project about to be torn
-        // down is dead work; an open teardown is left alone for the idempotent path).
-        assert.match(sql, /action IN \('provision', 'settings-apply'\) AND state = 'pending'/);
+        // intent is the runner's mid-flight work) and only the actions that become
+        // dead work in front of a teardown (task 1003140 widened it to the settings
+        // push, task 1003238 to the restart; an open teardown is left alone for the
+        // idempotent path).
+        assert.match(sql, /action IN \('provision', 'settings-apply', 'restart'\) AND state = 'pending'/);
         const hit = rows.filter((r) =>
-          String(r.instance_id) === String(params[0]) && (r.action === 'provision' || r.action === 'settings-apply') && r.state === 'pending');
+          String(r.instance_id) === String(params[0]) && SUPERSEDABLE.has(r.action) && r.state === 'pending');
         for (const r of hit) { r.state = 'error'; r.last_error = 'superseded: teardown requested'; }
         return { rows: hit.map((r) => ({ id: r.id })) };
       }