start-command 0.39.1 → 0.40.0

package/test/regression-187.js

@@ -78,6 +78,8 @@ function fixture() {
 }
 
 const stopped = () => ({ alive: false, state: SessionState.STOPPED });
+// The snapshot preflight must not depend on the free disk of the test host.
+const plentyOfDisk = { freeBytes: () => 100 * 1024 ** 3 };
 const accepted = (_bin, args = []) => ({
   success: true,
   stdout: args.includes('--size')
@@ -102,6 +104,7 @@ describe('issue #187: explicit resume attempts', () => {
             ? () => ({ alive: false, state: SessionState.MISSING })
             : stopped,
         runner: accepted,
+        snapshotOptions: plentyOfDisk,
         runIsolated: () => ({ success: true, containerId: 'new-container' }),
         startWatcher: (_name, _policy, _log, _uuid, options) => {
           expect(options.recoverOnKill).toBe(mode !== 'docker-snapshot');
@@ -357,6 +360,7 @@ esac
       command: 'replacement',
       probe: stopped,
       runner: accepted,
+      snapshotOptions: plentyOfDisk,
       startWatcher: () => {},
     });
     expect(result.success).toBe(true);

package/test/regression-224.js

@@ -59,6 +59,8 @@ afterEach(() => {
 const SAFETY = require.resolve('../src/lib/docker-snapshot-safety');
 
 /** Hold the snapshot lock in another process for `holdMs`. */
+// A waiter polls for 1 s before it reads the holder's name, so tests that
+// expect the name hold the lock well past that, even on a slow runner.
 async function holdLockElsewhere(lockPath, holder, holdMs) {
   const ready = `${lockPath}.ready`;
   const child = spawn(
@@ -264,7 +266,7 @@ describe('issue #224 item 1: waiting for the snapshot lock', () => {
 
   it('waits for the holder with --snapshot-wait and says so once', async () => {
     const lockPath = path.join(makeTempDir('lock-224-'), 'snapshot.lock');
-    await holdLockElsewhere(lockPath, 'task-a', 1500);
+    await holdLockElsewhere(lockPath, 'task-a', 3000);
     const lines = [];
     const started = Date.now();
     const lock = acquireSnapshotLock({
@@ -307,7 +309,7 @@ describe('issue #224 item 1: waiting for the snapshot lock', () => {
 
   it('serializes two concurrent CLI resumes instead of failing the second', async () => {
     const lockPath = path.join(makeTempDir('lock-224-'), 'snapshot.lock');
-    await holdLockElsewhere(lockPath, 'task-a', 1200);
+    await holdLockElsewhere(lockPath, 'task-a', 3000);
     const { store } = snapshotRecord();
     const calls = [];
     const lines = [];

package/test/regression-231.js

@@ -12,6 +12,7 @@ const { parseArgs } = require('../src/lib/args-parser');
 const {
   ExecutionStore,
   ExecutionRecord,
+  isClinkInstalled,
 } = require('../src/lib/execution-store');
 const {
   executionAliases,
@@ -126,6 +127,25 @@ describe('issue #231: execution names', () => {
     ]);
   });
 
+  // Issue #253: the rename used to reach only the lino store; clink 4.x
+  // rejected every links query the store built.
+  it.skipIf(!isClinkInstalled())(
+    '--rename reaches the links database',
+    () => {
+      const { store } = sandbox();
+      store.save(record(A, { sessionName: 'screen-a' }));
+      nameExecution(store, A, 'first', { rename: true });
+      const links = store.execClink('((($id: $s $t)) (($id: $s $t)))', [
+        '--changes',
+      ]).output;
+      expect(links).toContain(
+        `${A}.options: ${A} 'options={"sessionName":"screen-a","name":"first"}'`
+      );
+      expect(store.verifyConsistency().errors).toEqual([]);
+    },
+    30000
+  );
+
   it('a UUID-shaped alias resolves like any other name', () => {
     const { store } = sandbox();
     store.save(record(A));

package/test/release-git.mjs

@@ -87,7 +87,8 @@ describe('release-git (#211)', () => {
     await withRepo({ reject: true }, async () => {
       const result = await $`git push origin main`.run(quiet);
       expect(result.code).not.toBe(0);
-      expect(result.stderr).toContain('GH013');
+      // command-stream 2.x returns stderr as a readable snapshot (#253).
+      expect(String(result.stderr)).toContain('GH013');
     });
   });