@camperaid/watest 2.5.4 → 2.5.5

package/core/series.js

@@ -500,13 +500,17 @@ class Series {
         testname: path,
       });
 
-      tests.push({
+      const testObj = {
         name,
         path,
         func: test_wrap,
         webdriver,
         failures_info,
-      });
+      };
+      if (test_module.timeout) {
+        testObj.timeout = test_module.timeout;
+      }
+      tests.push(testObj);
     }
 
     // Add tests from subfolders.
@@ -653,6 +657,7 @@ class Series {
         skip_on_fail,
         run_in_child_process,
         init_or_uninit,
+        timeout,
       } = test;
 
       if (stop && !name.endsWith('uninit')) {
@@ -678,6 +683,7 @@ class Series {
 
       // Invoke a test.
       log(`\n!Running: ${name}, path: ${path}\n`);
+
       let start_time = new Date();
       let kungFuDeathGrip = null;
       let kungFuDeathGripResolve = null;
@@ -695,21 +701,23 @@ class Series {
       try {
         this.core.setExpectedFailures(failures_info);
 
-        // If timeout is given then race it against the test.
-        if (settings.timeout) {
+        // If timeout is given (from meta.js or settings) then race it against the test.
+        const effectiveTimeout = timeout ?? settings.timeout;
+        if (effectiveTimeout) {
+          const timeoutMs = effectiveTimeout * 1000;
           kungFuDeathGrip = new Promise(
             resolve => (kungFuDeathGripResolve = resolve),
           ).then(value => {
             if (value != kKungFuDeathGripCancelled) {
               fail(
-                `Test ${name} takes longer than ${settings.timeout}ms. It's either slow or never ends.`,
+                `Test ${name} takes longer than ${effectiveTimeout}s. It's either slow or never ends.`,
               );
               return kKungFuDeathGripTimeout;
             }
           });
           kungFuDeathGripTimer = setTimeout(
             kungFuDeathGripResolve,
-            settings.timeout,
+            timeoutMs,
           );
           let retval = await Promise.race([func(), kungFuDeathGrip]);
           if (retval != kKungFuDeathGripTimeout) {

package/core/settings.js

@@ -54,11 +54,11 @@ class Settings {
   }
 
   get testFilePattern() {
-    return this.rc.test_file_pattern || /^t[-_]/;
+    return this.rc?.test_file_pattern || /^t[-_]/;
   }
 
   get timeout() {
-    return parseInt(this.rc.timeout) || 0;
+    return parseInt(this.rc?.timeout) || 0;
   }
 
   setupTmpStorageDir() {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@camperaid/watest",
-  "version": "2.5.4",
+  "version": "2.5.5",
   "description": "Web Application Testsuite",
   "type": "module",
   "engines": {

package/tests/series/perform/t-timeout.js

@@ -0,0 +1,107 @@
+import { is_test_output } from '../test.js';
+import { success } from '../../../core/core.js';
+import { Core } from '../../../core/core.js';
+import { Series } from '../../../core/series.js';
+
+class MockLogPipe {
+  attach() {
+    return Promise.resolve();
+  }
+  release() {
+    return Promise.resolve();
+  }
+  logToFile() {}
+}
+
+function make_perform_with_timeout(tests) {
+  return async () => {
+    const series = new Series('tests/', {
+      core: new Core(),
+      LogPipe: new MockLogPipe(),
+    });
+    try {
+      await series.perform({ folder: 'tests/', tests });
+    } finally {
+      series.shutdown();
+    }
+  };
+}
+
+export async function test() {
+  // Test that timeout from meta.js causes test to fail after specified time
+  await is_test_output(
+    make_perform_with_timeout([
+      {
+        name: 't-slow.js',
+        path: 'tests/t-slow.js',
+        func: () => new Promise(resolve => setTimeout(resolve, 500)),
+        failures_info: [],
+        timeout: 0.05, // 50ms timeout in seconds
+      },
+    ]),
+    [
+      '\x1B[38;5;99mStarted\x1B[0m tests/',
+      '!Running: t-slow.js, path: tests/t-slow.js',
+      '>t-slow.js completed in',
+      '\x1B[38;5;243mCompleted\x1B[0m tests/',
+      'Testsuite: shutdown',
+    ],
+    [
+      "\x1B[31mFailed:\x1B[0m Test t-slow.js takes longer than 0.05s. It's either slow or never ends.",
+      '\x1B[31m>t-slow.js\x1B[0m has 1 failure(s)',
+    ],
+    'meta timeout causes test to fail',
+  );
+
+  // Test that test completes normally when timeout is not exceeded
+  await is_test_output(
+    make_perform_with_timeout([
+      {
+        name: 't-fast.js',
+        path: 'tests/t-fast.js',
+        func: () => {
+          success('fast test passes');
+          return Promise.resolve();
+        },
+        failures_info: [],
+        timeout: 1, // 1 second timeout
+      },
+    ]),
+    [
+      '\x1B[38;5;99mStarted\x1B[0m tests/',
+      '!Running: t-fast.js, path: tests/t-fast.js',
+      '\x1B[32mOk:\x1B[0m fast test passes',
+      '>t-fast.js completed in',
+      '\x1B[38;5;243mCompleted\x1B[0m tests/',
+      'Testsuite: shutdown',
+    ],
+    [],
+    'test completes within timeout',
+  );
+
+  // Test that no timeout applies when timeout is undefined
+  await is_test_output(
+    make_perform_with_timeout([
+      {
+        name: 't-no-timeout.js',
+        path: 'tests/t-no-timeout.js',
+        func: () => {
+          success('no timeout test passes');
+          return new Promise(resolve => setTimeout(resolve, 10));
+        },
+        failures_info: [],
+        // timeout: undefined - no timeout set
+      },
+    ]),
+    [
+      '\x1B[38;5;99mStarted\x1B[0m tests/',
+      '!Running: t-no-timeout.js, path: tests/t-no-timeout.js',
+      '\x1B[32mOk:\x1B[0m no timeout test passes',
+      '>t-no-timeout.js completed in',
+      '\x1B[38;5;243mCompleted\x1B[0m tests/',
+      'Testsuite: shutdown',
+    ],
+    [],
+    'test runs without timeout when not specified',
+  );
+}

package/webdriver/driver-base.js

@@ -30,7 +30,7 @@ function getChromeOptions() {
     height: settings.webdriver_window_height,
   });
   if (settings.webdriver_headless) {
-    chromeOptions.addArguments('headless');
+    chromeOptions.addArguments('headless=new');
   }
   if (settings.webdriver == 'chrome-mobile') {
     chromeOptions.setMobileEmulation({