fix(V2): revert xlsx round-trip commit folding

TheLastCicada · TheLastCicada · commit 31f82938fec3 · 2026-04-24T11:38:38.000-07:00
Cross-process module-level variables (beforeProjects, beforeUnits) set
in a separate mocha subprocess cannot be read by a subsequent subprocess
invocation.  Additionally, committing XLSX round-trip and PUT changes in
the same cycle would cause Step 15a fidelity assertions to fail for any
project/unit also modified by PUT tests.

Restore the original single-pass XLSX invocation and the standalone
Step 15 commit it() block.  The 3-pass interleaved approach is dropped;
Opts 1-3 (cascade-delete CI split, merged it() blocks, 3s poll interval)
are retained.
diff --git a/tests/v2/live-api/data-short.js b/tests/v2/live-api/data-short.js
@@ -394,20 +394,12 @@ async function main() {
     console.log('');
 
     const xlsxTestFiles = ['xlsx-import-export.live.spec.js'];
+    await runMochaTests('Step 1[1-6]a?:', 'XLSX Import/Export', xlsxTestFiles);
 
-    // Pass 1: XLSX imports + verify + export (1 on-chain commit from Step 11)
-    await runMochaTests('Step 11:|Step 11a:|Step 12a:|Step 13:|Step 14:', 'XLSX Import/Export (imports)', xlsxTestFiles);
-
-    // Pass 2: XLSX round-trip staging only (no separate commit — absorbed into PUT commit below)
-    await runMochaTests('Step 15: Round-trip re-import \\(batch\\)', 'XLSX Round-trip Staging', xlsxTestFiles);
-
-    // Phase 4: PUT tests (stages on top of XLSX round-trip; one commit covers both)
+    // Phase 4: PUT tests
     await runMochaTests('Step 7: PUT Request Tests', 'PUT Operations');
     await commitAndWait('PUT');
 
-    // Pass 3: XLSX round-trip verify + filter export (reads now-committed data)
-    await runMochaTests('Step 15a:|Step 16:', 'XLSX Round-trip Verify + Filter Export', xlsxTestFiles);
-
     // Phase 5: DELETE tests
     await runMochaTests('Step 9: DELETE Request Tests', 'DELETE Operations');
     await commitAndWait('DELETE');
diff --git a/tests/v2/live-api/xlsx-import-export.live.spec.js b/tests/v2/live-api/xlsx-import-export.live.spec.js
@@ -515,6 +515,11 @@ describe('XLSX Import/Export Live API Tests', function () {
       expect(importResponse.body.success).to.be.true;
     });
 
+    it('should commit all round-trip staged records', async function () {
+      await commitStagedRecords(request, [], true);
+      await waitForPendingCommits(request);
+      await waitForStagingEmpty(request);
+    });
   });
 
   describe('Step 15a: Verify round-trip fidelity', function () {
