fix: test reliability

Author: abose

test/test-dir.browser.js

@@ -58,11 +58,12 @@ function _setupTests(testType) {
     }
 
     before(async function () {
+        this.timeout(15000); // Increase timeout for Node WebSocket setup
         switch (testType) {
         case TEST_TYPE_FS_ACCESS: testPath = window.mountTestPath;break;
         case TEST_TYPE_FILER: testPath = window.virtualTestPath;break;
         case TEST_TYPE_TAURI_WS:
-            await window.waitForTrue(()=>{return window.isNodeSetup;},1000);
+            await window.waitForTrue(()=>{return window.isNodeSetup;}, 10000);
             fs.forceUseNodeWSEndpoint(true);
             testPath = fs.getTauriVirtualPath(`${await _getTestBaseDir()}test-phoenix-fs`);
             consoleLogToShell("using tauri websocket test path: "+ testPath);
@@ -82,6 +83,7 @@ function _setupTests(testType) {
     });
 
     beforeEach(async function () {
+        this.timeout(10000); // Increase timeout for setup operations
         // setup test folders
         await _clean();
         console.log(`mkdirs: `, testPath);
@@ -573,7 +575,7 @@ describe(`Should phoenix be able to read root dir`, async function () {
     if(window.__TAURI__ || window.__ELECTRON__){
 
         before(async ()=>{
-            await window.waitForTrue(()=>{return window.isNodeSetup;},1000);
+            await window.waitForTrue(()=>{return window.isNodeSetup;}, 10000);
         });
 
         it(`Should read root /tauri dir`, async function () {

test/test-file.browser.js

@@ -79,7 +79,7 @@ function _setupTests(testType) {
         case TEST_TYPE_FS_ACCESS: testPath = window.mountTestPath;break;
         case TEST_TYPE_FILER: testPath = window.virtualTestPath;break;
         case TEST_TYPE_TAURI_WS:
-            await window.waitForTrue(()=>{return window.isNodeSetup;},1000);
+            await window.waitForTrue(()=>{return window.isNodeSetup;}, 10000);
             fs.forceUseNodeWSEndpoint(true);
             testPath = fs.getTauriVirtualPath(`${await _getTestBaseDir()}test-phoenix-fs`);
             consoleLogToShell("using tauri websocket test path: "+ testPath);

test/test-node.browser.js

@@ -8,7 +8,7 @@ describe(`node ws fs tests`, function () {
     let wssPort, wssEndpoint;
 
     before(async function () {
-        await window.waitForTrue(()=>{return window.isNodeSetup;},1000);
+        await window.waitForTrue(()=>{return window.isNodeSetup;}, 10000);
         let message = await execNode(NODE_COMMANDS.GET_PORT);
         expect(message.port).to.be.a('number');
         wssPort = message.port;

test/test-watcher.browser.js

@@ -148,7 +148,7 @@ function _setupTests(testType) {
         case TEST_TYPE_FS_ACCESS: testPath = window.mountTestPath;break;
         case TEST_TYPE_FILER: testPath = window.virtualTestPath;break;
         case TEST_TYPE_TAURI_WS:
-            await window.waitForTrue(()=>{return window.isNodeSetup;},1000);
+            await window.waitForTrue(()=>{return window.isNodeSetup;}, 10000);
             fs.forceUseNodeWSEndpoint(true);
             testPath = fs.getTauriVirtualPath(`${await _getTestBaseDir()}test-phoenix-fs`);
             consoleLogToShell("using tauri websocket test path: "+ testPath);

test/test.worker.js

@@ -49,7 +49,7 @@ function _setupTests(testType) {
         case TEST_TYPE_FS_ACCESS: testPath = window.mountTestPath;break;
         case TEST_TYPE_FILER: testPath = window.virtualTestPath;break;
         case TEST_TYPE_TAURI_WS:
-            await window.waitForTrue(()=>{return window.isNodeSetup;},1000);
+            await window.waitForTrue(()=>{return window.isNodeSetup;}, 10000);
             fs.forceUseNodeWSEndpoint(true);
             testPath = fs.getTauriVirtualPath(`${await window.__TAURI__.path.appLocalDataDir()}test-phoenix-fs`);
             consoleLogToShell("using tauri websocket test path: "+ testPath);
@@ -69,8 +69,11 @@ function _setupTests(testType) {
             console.log(`From Worker:`, event);
             messageFromWorker = event.data;
         };
+        // Wait for worker to be ready before running tests
+        let workerReady = await waitForWorkerMessage(`worker.ready`, 5000);
+        expect(workerReady).to.be.true;
         if(testType === TEST_TYPE_TAURI_WS){
-            await window.waitForTrue(()=>{return window.isNodeSetup;},1000);
+            await window.waitForTrue(()=>{return window.isNodeSetup;}, 10000);
             worker.postMessage({command: `tauriWSInit`, wsEndpoint: window.nodeWSEndpoint});
             let status = await waitForWorkerMessage(`tauriWSInit.ok`, 1000);
             expect(status).to.be.true;

test/testInit.js

@@ -25,6 +25,12 @@ window.waitForTrue = async function(checkFn, timeoutMs) {
 
 if(window.__TAURI__ || window.__ELECTRON__) {
     before(async function () {
-        await window.waitForTrue(()=>{return window.isNodeSetup;},1000);
+        this.timeout(15000); // Increase mocha timeout for Node WebSocket setup
+        // Wait up to 10 seconds for Node WebSocket to be ready
+        // This is important for slower systems or first-time loads
+        let ready = await window.waitForTrue(()=>{return window.isNodeSetup;}, 10000);
+        if (!ready) {
+            throw new Error('Timeout waiting for Node WebSocket setup');
+        }
     });
 }

test/worker-task.js

@@ -96,3 +96,6 @@ self.addEventListener('message', (event) => {
     default: console.error('unknown worker command: ', command);
     }
 }, false);
+
+// Signal that the worker is ready
+postMessage('worker.ready');