@camperaid/watest 2.4.7 → 2.4.9

package/core/format.js

@@ -1,4 +1,4 @@
-'use strict';
+const esc_char = ''; // \x1b
 
 const colors = {
   group: '36',
@@ -13,17 +13,17 @@ const colors = {
   intermittents: '105',
   todos: '103',
   warnings: '103',
-  failures: '41m\x1b[37',
+  failures: `41m${esc_char}[37`,
   success: '102',
 };
 
 const codes = new Map(
-  Array.from(Object.keys(colors)).map(key => [colors[key], key])
+  Array.from(Object.keys(colors)).map(key => [colors[key], key]),
 );
 
 // Colorifies the message.
 function colorify(color, label, msg) {
-  return `\x1b[${colors[color]}m${label}\x1b[0m ${msg}`;
+  return `${esc_char}[${colors[color]}m${label}${esc_char}[0m ${msg}`;
 }
 
 // Parses the given message.
@@ -53,60 +53,100 @@ function parse_failure(str) {
   return null;
 }
 
-module.exports = {
-  colorify,
-  parse,
-  parse_failure,
+function format_started(msg) {
+  return colorify('started', 'Started', msg);
+}
+
+function format_completed(msg) {
+  return colorify('completed', 'Completed', msg);
+}
+
+function format_intermittent(msg) {
+  return colorify('intermittent', 'Intermittent:', msg);
+}
+
+function format_intermittents(count, context) {
+  let label = context ? `${context} > intermittents` : 'Intermittents';
+  return colorify('intermittents', label, `Total: ${count}`);
+}
 
-  format_started(msg) {
-    return colorify('started', 'Started', msg);
-  },
-  format_completed(msg) {
-    return colorify('completed', 'Completed', msg);
-  },
-  format_intermittent(msg) {
-    return colorify('intermittent', 'Intermittent:', msg);
-  },
-  format_intermittents(count, context) {
-    let label = context ? `${context} > intermittents` : 'Intermittents';
-    return colorify('intermittents', label, `Total: ${count}`);
-  },
-  format_failure(msg, label) {
-    return colorify('failure', label || 'Failed:', msg);
-  },
-  format_failures(count, context_or_okcount, context) {
-    if (typeof context_or_okcount == 'string') {
-      return colorify(
-        'failures',
-        `>${context_or_okcount}`,
-        `Failure count: ${count}`
-      );
-    }
-    let label = !context ? 'Failed!' : `${context} > failed`;
+function format_failure(msg, label) {
+  return colorify('failure', label || 'Failed:', msg);
+}
+
+function format_failures(count, context_or_okcount, context) {
+  if (typeof context_or_okcount == 'string') {
     return colorify(
       'failures',
-      label,
-      `Passed: ${context_or_okcount}. Failed: ${count}`
+      `>${context_or_okcount}`,
+      `Failure count: ${count}`,
     );
-  },
-  format_ok(msg) {
-    return colorify('ok', 'Ok:', msg);
-  },
-  format_success(okcount, context) {
-    return colorify('success', context || 'Success!', `Total: ${okcount}`);
-  },
-  format_todo(msg) {
-    return colorify('todo', 'Todo:', msg);
-  },
-  format_todos(count, context) {
-    let label = context ? `${context} > todos` : 'Todos';
-    return colorify('todos', label, `Total: ${count}`);
-  },
-  format_warning(msg) {
-    return colorify('warning', 'Warning:', msg);
-  },
-  format_warnings(count, context) {
-    let label = context ? `${context} > warnings` : 'Warnings';
-    return colorify('warnings', label, `Total: ${count}`);
-  },
+  }
+  let label = !context ? 'Failed!' : `${context} > failed`;
+  return colorify(
+    'failures',
+    label,
+    `Passed: ${context_or_okcount}. Failed: ${count}`,
+  );
+}
+
+function format_ok(msg) {
+  return colorify('ok', 'Ok:', msg);
+}
+
+function format_success(okcount, context) {
+  return colorify('success', context || 'Success!', `Total: ${okcount}`);
+}
+
+function format_todo(msg) {
+  return colorify('todo', 'Todo:', msg);
+}
+
+function format_todos(count, context) {
+  let label = context ? `${context} > todos` : 'Todos';
+  return colorify('todos', label, `Total: ${count}`);
+}
+
+function format_warning(msg) {
+  return colorify('warning', 'Warning:', msg);
+}
+
+function format_warnings(count, context) {
+  let label = context ? `${context} > warnings` : 'Warnings';
+  return colorify('warnings', label, `Total: ${count}`);
+}
+
+// Node v20 wraps stderr by '\[31m' and '\[39m' characters.
+function stderr_wrap(s) {
+  return `${s}`;
+}
+
+function stderr_unwrap(s) {
+  // eslint-disable-next-line no-control-regex
+  return s.replace(/\[31m(.*?)\[39m(\n?)/gs, '$1$2');
+}
+
+function stderr_format_failure(...args) {
+  return stderr_wrap(format_failure(...args));
+}
+
+export {
+  colorify,
+  parse,
+  parse_failure,
+  format_started,
+  format_completed,
+  format_intermittent,
+  format_intermittents,
+  format_failure,
+  format_failures,
+  format_ok,
+  format_success,
+  format_todo,
+  format_todos,
+  format_warning,
+  format_warnings,
+  stderr_wrap,
+  stderr_unwrap,
+  stderr_format_failure,
 };

package/core/process_args.js

@@ -1,5 +1,3 @@
-'use strict';
-
 class ProcessArgs {
   static asObject() {
     let obj = {
@@ -68,6 +66,4 @@ class ProcessArgs {
   }
 }
 
-module.exports = {
-  ProcessArgs,
-};
+export { ProcessArgs };

package/core/series.js

@@ -1,21 +1,18 @@
-'use strict';
-
-const fs = require('fs');
-const path = require('path');
-const nodepath = path;
-
-const { log, log_error } = require('../logging/logging.js');
-const { DriverBase } = require('../webdriver/driver_base.js');
-const testflow = require('./core.js');
-const { parse, parse_failure } = require('./format.js');
-const { ProcessArgs } = require('./process_args.js');
-const settings = require('./settings.js');
-const { spawn } = require('./spawn');
-const { stringify } = require('./util.js');
-
-const { assert, fail } = testflow;
-
-const {
+import fs from 'fs';
+import nodepath from 'path';
+import { fileURLToPath } from 'url';
+
+import { assert, fail, testflow } from './core.js';
+import { parse, parse_failure } from './format.js';
+import { ProcessArgs } from './process_args.js';
+import settings from './settings.js';
+import { spawn } from './spawn.js';
+import { stringify } from './util.js';
+import { log, log_error } from '../logging/logging.js';
+import { LogPipe } from '../logging/logpipe.js';
+import { DriverBase } from '../webdriver/driver_base.js';
+
+import {
   format_started,
   format_completed,
   format_failure,
@@ -25,10 +22,13 @@ const {
   format_todos,
   format_warnings,
   colorify,
-} = require('./format.js');
+} from './format.js';
+
+const __filename = fileURLToPath(import.meta.url);
+const __dirname = nodepath.dirname(__filename);
 
 const root_folder = 'tests';
-const root_dir = path.resolve('.');
+const root_dir = nodepath.resolve('.');
 
 const kKungFuDeathGripTimeout = {};
 const kKungFuDeathGripCancelled = {};
@@ -44,7 +44,7 @@ process.on('unhandledRejection', error => {
 class Series {
   static run(patterns, options) {
     if (!('LogPipe' in options)) {
-      options.LogPipe = require('../logging/logpipe.js').LogPipe;
+      options.LogPipe = LogPipe;
     }
     options.LogPipe.suppress_logging = options.childProcess;
 
@@ -57,7 +57,7 @@ class Series {
           .run()
           .then(() =>
             // shutdown returns list of failures or null if no failures
-            options.LogPipe.release()
+            options.LogPipe.release(),
           )
           .then(
             () => series.shutdown(),
@@ -66,7 +66,7 @@ class Series {
               fail(e.message);
               log_error(e);
               return failures;
-            }
+            },
           );
       });
   }
@@ -86,7 +86,7 @@ class Series {
       LogPipe,
       webdriver,
       webdrivers = settings.webdrivers,
-    }
+    },
   ) {
     this.debunk = debunk;
     this.invocation = invocation || settings.invocation;
@@ -107,7 +107,7 @@ class Series {
     this.ocnt = 0;
 
     this.core = core || testflow.core;
-    testflow.lock(this.core);
+    testflow.lock({ core: this.core });
 
     this.core.setTimeout(timeout);
     this.core.clearStats();
@@ -128,7 +128,7 @@ class Series {
       this.patterns.map(pattern => ({
         path: pattern,
         webdriver: this.webdriver,
-      }))
+      })),
     );
 
     // In debunk mode re-run tests until it fails.
@@ -171,8 +171,8 @@ class Series {
         colorify(
           'failures',
           '!Failed:',
-          `no tests matching '${stringify(patterns)}' pattern(s) found`
-        )
+          `no tests matching '${stringify(patterns)}' pattern(s) found`,
+        ),
       );
       return Promise.resolve();
     }
@@ -206,7 +206,7 @@ class Series {
 
       if (test_module.expected_failures) {
         inherited_expected_failures = inherited_expected_failures.concat(
-          test_module.expected_failures.filter(v => v[0] == '**')
+          test_module.expected_failures.filter(v => v[0] == '**'),
         );
       }
 
@@ -251,7 +251,7 @@ class Series {
       // A separate folder for the webdriver tests.
       assert(
         this.webdrivers instanceof Array,
-        `Webdrivers are misconfigured, got: ${JSON.stringify(this.webdrivers)}`
+        `Webdrivers are misconfigured, got: ${JSON.stringify(this.webdrivers)}`,
       );
 
       // Build the tests for webdrivers. Filter the list according traversed
@@ -378,8 +378,8 @@ class Series {
         }).then(subtests => ({
           subfolder,
           subtests,
-        }))
-      )
+        })),
+      ),
     );
 
     return subtests_for_subfolders
@@ -425,7 +425,7 @@ class Series {
     }
 
     const expected_failures = inherited_expected_failures.concat(
-      test_module.expected_failures || []
+      test_module.expected_failures || [],
     );
 
     // Initialize
@@ -494,7 +494,7 @@ class Series {
         await Promise.all(
           [...(test_module.services || [])]
             .reverse()
-            .map(s => settings.servicer.stop(s))
+            .map(s => settings.servicer.stop(s)),
         );
       };
       tests.push({
@@ -533,7 +533,7 @@ class Series {
       list = list.filter(
         test =>
           this.matchedPatterns({ path: test.path, webdriver, patterns })
-            .length != 0
+            .length != 0,
       );
     }
 
@@ -545,7 +545,7 @@ class Series {
       pattern =>
         (!webdriver || !pattern.webdriver || pattern.webdriver == webdriver) &&
         (path.startsWith(pattern.path) ||
-          (path_is_not_final && pattern.path.startsWith(path)))
+          (path_is_not_final && pattern.path.startsWith(path))),
     );
   }
 
@@ -647,18 +647,18 @@ class Series {
         // If timeout is given then race it against the test.
         if (settings.timeout) {
           kungFuDeathGrip = new Promise(
-            resolve => (kungFuDeathGripResolve = resolve)
+            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 ${settings.timeout}ms. It's either slow or never ends.`,
               );
               return kKungFuDeathGripTimeout;
             }
           });
           kungFuDeathGripTimer = setTimeout(
             kungFuDeathGripResolve,
-            settings.timeout
+            settings.timeout,
           );
           let retval = await Promise.race([func(), kungFuDeathGrip]);
           if (retval != kKungFuDeathGripTimeout) {
@@ -825,8 +825,8 @@ class Series {
         format_failures(
           this.core.failureCount - fcnt,
           this.core.okCount - ocnt,
-          !is_root && folder
-        )
+          !is_root && folder,
+        ),
       );
     } else {
       log_func(format_success(this.core.okCount - ocnt, !is_root && folder));
@@ -835,7 +835,7 @@ class Series {
 
   static failuresInfo({ failures, webdriver, platform, testname }) {
     let filtered_failures = failures.filter(
-      v => testname.includes(v[0]) || v[0] == '*' || v[0] == '**'
+      v => testname.includes(v[0]) || v[0] == '*' || v[0] == '**',
     );
 
     return (
@@ -865,13 +865,14 @@ class Series {
   async loadTestMeta(folder) {
     try {
       return await import(this.getTestMetaPath(folder));
+      // eslint-disable-next-line no-unused-vars
     } catch (e) {
       return {}; // no meta.js
     }
   }
 
   loadTest(test_path) {
-    return import(path.join(root_dir, test_path)).then(
+    return import(nodepath.join(root_dir, test_path)).then(
       test_module => {
         if (!test_module.test) {
           throw new Error(`No test was found in ${test_path}`);
@@ -881,32 +882,33 @@ class Series {
       e => {
         log_error(e);
         throw new Error(`Failed to load test: ${test_path}`);
-      }
+      },
     );
   }
 
   getTestMetaPath(folder) {
-    let meta_path = path.join(root_dir, `${folder}/meta.js`);
+    let meta_path = nodepath.join(root_dir, `${folder}/meta.js`);
     return (
       (fs.existsSync(meta_path) && meta_path) ||
-      path.join(root_dir, `${folder}/meta.mjs`)
+      nodepath.join(root_dir, `${folder}/meta.mjs`)
     );
   }
 
   getTestFileList(folder) {
     return fs
-      .readdirSync(path.join(root_dir, folder))
+      .readdirSync(nodepath.join(root_dir, folder))
       .filter(
         n =>
           n.startsWith('t_') &&
-          (!settings.ignorePattern || !settings.ignorePattern.test(n))
+          (!settings.ignorePattern || !settings.ignorePattern.test(n)),
       );
   }
 
   performInChildProcess({ name, path, loader, webdriver }) {
     let args = [];
     if (loader) {
-      args.push('--loader', loader);
+      // Use the new --import flag with register() API instead of deprecated --loader
+      args.push('--import', `data:text/javascript,import { register } from "node:module"; import { pathToFileURL } from "node:url"; register("${loader}", pathToFileURL("./"));`);
     }
     const watest_bin = nodepath.join(__dirname, '../bin/watest.js');
     args.push(
@@ -916,14 +918,14 @@ class Series {
       '--root-folder',
       nodepath.join(name, '../'),
       ...ProcessArgs.controlArguments,
-      path
+      path,
     );
     if (webdriver) {
       args.push('--webdriver', webdriver);
     }
 
     return spawn('node', args, {}, buffer =>
-      this.processChildProcessOutput(name, buffer)
+      this.processChildProcessOutput(name, buffer),
     ).catch(e => {
       log_error(e);
       fail(`Failed to process child process output`);
@@ -944,22 +946,22 @@ class Series {
         if (line.startsWith('console.debug')) {
           line = line.replace(
             /console\.debug: "(.+)"/,
-            (match, p) => `[DEBUG] ${p}`
+            (match, p) => `[DEBUG] ${p}`,
           );
         } else if (line.startsWith('console.log')) {
           line = line.replace(
             /console\.log: "(.+)"/,
-            (match, p) => `[INFO] ${p}`
+            (match, p) => `[INFO] ${p}`,
           );
         } else if (line.startsWith('console.assert')) {
           line = line.replace(
             /console\.assert: "(.+)"/,
-            (match, p) => `[SEVERE] ${p}`
+            (match, p) => `[SEVERE] ${p}`,
           );
         } else if (line.startsWith('console.error')) {
           line = line.replace(
             /console\.error: "(.+)"/,
-            (match, p) => `[SEVERE] ${p}`
+            (match, p) => `[SEVERE] ${p}`,
           );
         } else if (line.startsWith('data:text/html,')) {
           line = line.replace(/\S*/, '@dataurl_placeholder');
@@ -1018,5 +1020,8 @@ class Series {
   }
 }
 
-module.exports.Series = Series;
-module.exports.runSeries = (...args) => Series.run(...args);
+export { Series };
+
+export function runSeries(...args) {
+  return Series.run(...args);
+}

package/core/settings.js

@@ -1,13 +1,4 @@
-'use strict';
-
-const path = require('path');
-
-let rc = null;
-try {
-  rc = require(path.resolve('.', './.watestrc.js'));
-} catch (e) {
-  rc = require(path.resolve('.', './.watestrc.cjs'));
-}
+import path from 'path';
 
 class Settings {
   constructor() {
@@ -16,20 +7,22 @@ class Settings {
     this.tmp_storage_dir = '';
     this.logger = null;
     this.servicer = null;
-
-    this.setupTmpStorageDir();
-    this.setupLogDir();
-    this.setupWebdrivers();
   }
 
   async initialize() {
+    this.rc = (await import(path.resolve('.', './.watestrc.js'))).default;
+
     this.logger = (
-      await import(rc.logger || '../interfaces/logger.js')
+      await import(this.rc.logger || '../interfaces/logger.js')
     ).default;
 
     this.servicer = (
-      await import(rc.servicer || '../interfaces/servicer.js')
+      await import(this.rc.servicer || '../interfaces/servicer.js')
     ).default;
+
+    this.setupTmpStorageDir();
+    this.setupLogDir();
+    this.setupWebdrivers();
   }
 
   get invocation() {
@@ -41,64 +34,71 @@ class Settings {
           break;
       }
 
-      this._invocation = rc.invocation || `${platform}`;
+      this._invocation = this.rc.invocation || `${platform}`;
     }
     return this._invocation;
   }
 
   get ignorePattern() {
-    return rc.ignore_pattern && new RegExp(rc.ignore_pattern);
+    return this.rc.ignore_pattern && new RegExp(this.rc.ignore_pattern);
   }
 
   get debunkLimit() {
-    return parseInt(rc.debunk_limit) || 5;
+    return parseInt(this.rc.debunk_limit) || 5;
   }
 
   get timeout() {
-    return parseInt(rc.timeout) || 0;
+    return parseInt(this.rc.timeout) || 0;
   }
 
   setupTmpStorageDir() {
-    if (!rc.tmp_dir) {
+    if (!this.rc.tmp_dir) {
       console.log(`Settings: no temporary storage dir`);
       return;
     }
 
-    this.tmp_storage_dir = path.join(rc.tmp_dir, 'watest-tmpstorage');
+    this.tmp_storage_dir = path.join(this.rc.tmp_dir, 'watest-tmpstorage');
     console.log(
-      `Settings: temporary storage dir is at ${this.tmp_storage_dir}`
+      `Settings: temporary storage dir is at ${this.tmp_storage_dir}`,
     );
   }
 
   setupLogDir() {
-    const log_dir = rc.log_dir;
+    const log_dir = this.rc.log_dir;
     if (!log_dir) {
       console.log('Settings: no file logging');
       return;
     }
 
-    this.run = rc.run || `${parseInt(Date.now() / 1000)}`;
+    this.run = this.rc.run || `${parseInt(Date.now() / 1000)}`;
 
     this.log_dir = path.join(log_dir, this.run);
     console.log(`Settings: logging into ${log_dir}`);
   }
 
   setupWebdrivers() {
-    this.webdrivers = rc.webdrivers;
-    if (typeof rc.webdrivers == 'string') {
+    this.webdrivers = this.rc.webdrivers;
+    if (typeof this.rc.webdrivers == 'string') {
       try {
-        this.webdrivers = JSON.parse(rc.webdrivers);
+        this.webdrivers = JSON.parse(this.rc.webdrivers);
       } catch (e) {
         console.error(
-          `Settings: failed to parse webdrivers '${rc.webdrivers}'`
+          `Settings: failed to parse webdrivers '${this.rc.webdrivers}'`,
+          e,
         );
       }
     }
 
     this.webdriver = null;
     this.webdriver_headless =
-      rc.webdriver_headless == true || rc.webdriver_headless == 'true';
-    this.webdriver_loglevel = rc.webdriver_loglevel;
+      this.rc.webdriver_headless == true ||
+      this.rc.webdriver_headless == 'true';
+    this.webdriver_loglevel = this.rc.webdriver_loglevel;
+
+    this.webdriver_window_width =
+      parseInt(this.rc.webdriver_window_width) || 1366;
+    this.webdriver_window_height =
+      parseInt(this.rc.webdriver_window_height) || 768;
 
     if (this.webdrivers) {
       console.log(`Settings: ${this.webdrivers.join(', ')} webdrivers`);
@@ -106,4 +106,4 @@ class Settings {
   }
 }
 
-module.exports = new Settings();
+export default new Settings();