Make notebook tests more reliable

packages/test-harness/package.json

@@ -39,7 +39,7 @@
     "@types/glob": "^8.1.0",
     "@types/mocha": "^10.0.9",
     "@types/tail": "2.2.3",
-    "@vscode/test-electron": "^2.4.1",
+    "@vscode/test-electron": "^2.5.2",
     "cross-spawn": "7.0.5",
     "mocha": "^10.7.3"
   },

packages/test-harness/src/launchVscodeAndRunTests.ts

@@ -27,8 +27,8 @@ export async function launchVscodeAndRunTests(extensionTestsPath: string) {
       "packages/cursorless-vscode/dist",
     );
 
-    const crashDir = getEnvironmentVariableStrict("VSCODE_CRASH_DIR");
-    const logsDir = getEnvironmentVariableStrict("VSCODE_LOGS_DIR");
+    // const crashDir = getEnvironmentVariableStrict("VSCODE_CRASH_DIR");
+    // const logsDir = getEnvironmentVariableStrict("VSCODE_LOGS_DIR");
     const useLegacyVscode =
       getEnvironmentVariableStrict("APP_VERSION") === "legacy";
 
@@ -81,10 +81,7 @@ export async function launchVscodeAndRunTests(extensionTestsPath: string) {
       // don't bother.  Can be re-enabled if we ever need it; on windows it only
       // hangs some of the time, so might be enough to get a crash dump when you
       // need it.
-      launchArgs:
-        useLegacyVscode || os.platform() === "win32"
-          ? undefined
-          : [`--crash-reporter-directory=${crashDir}`, `--logsPath=${logsDir}`],
+      launchArgs: ["--force-node-api-uncaught-exceptions-policy=true"],
     });
 
     console.log(`Returned from "runTests" with value: ${code}`);

packages/test-harness/src/runAllTests.ts

@@ -71,6 +71,9 @@ async function runTestsInDir(
   // Add files to the test suite
   files.forEach((f) => mocha.addFile(path.resolve(testRoot, f)));
 
+  console.log(`Running tests in ${testRoot} for ${files.length} files`);
+  console.log(process.argv);
+
   try {
     // Run the mocha test
     await new Promise<void>((resolve, reject) => {