Fix test flakiness from stale git locks after worker termination

undefined · claude · undefined · commit dfceba195abe · 2026-03-15T06:28:20.000+05:00
- Wrap exec in runner.js to auto-remove index.lock before git commands
- Add 1s delay after worker.terminate() to let native threads release locks

Co-Authored-By: Claude Opus 4.6 (1M context) &lt;noreply@anthropic.com&gt;
diff --git a/test/runner.js b/test/runner.js
@@ -1,13 +1,23 @@
 var fse = require("fs-extra");
 var path = require("path");
 var local = path.join.bind(path, __dirname);
-var exec = require('../utils/execPromise');
+var _exec = require('../utils/execPromise');
 
 var NodeGit = require('..');
 
 var workdirPath = local("repos/workdir");
 var constWorkdirPath = local("repos/constworkdir");
 var masterSha;
+
+// Wrap exec to remove stale git locks before any git command
+function exec(command, opts) {
+  if (opts && opts.cwd && command.startsWith("git ")) {
+    try {
+      fse.removeSync(path.join(opts.cwd, ".git", "index.lock"));
+    } catch (e) {}
+  }
+  return _exec(command, opts);
+}
 var gitDir;
 var gitBackupDir;
 
diff --git a/test/tests/worker.js b/test/tests/worker.js
@@ -101,7 +101,8 @@ if (Worker) {
         worker.on("error", () => {});
         worker.on("exit", (code) => {
           if (code === TERMINATED_EXIT_CODE) {
-            done();
+            // Delay to let native threads release locks after termination
+            setTimeout(done, 1000);
           } else {
             assert.fail();
           }
@@ -144,7 +145,7 @@ if (Worker) {
       worker.on("error", () => {});
       worker.on("exit", (code) => {
         if (code === TERMINATED_EXIT_CODE && testOk === true) {
-          done();
+          setTimeout(done, 1000);
         }
         else {
           assert.fail();
@@ -217,7 +218,7 @@ if (Worker) {
       worker.on("error", () => {});
       worker.on("exit", (code) => {
         if (code === TERMINATED_EXIT_CODE && testOk === true) {
-          done();
+          setTimeout(done, 1000);
         }
         else {
           assert.fail();
