fix: more tests

Author: kobenguyent

.eslintrc.json

@@ -43,6 +43,7 @@
     "no-loop-func": 0,
     "arrow-parens": 0,
     "default-param-last": 0,
-    "import/extensions": 0
+    "import/extensions": 0,
+    "no-return-await": "off"
   }
 }

bin/codecept.js

@@ -1,8 +1,23 @@
 #!/usr/bin/env node
-import program from 'commander';
-import Codecept from '../lib/codecept';
+import { program } from 'commander';
+import Codecept from '../lib/codecept.js';
 import { print, error } from '../lib/output.js';
 import { printError } from '../lib/command/utils.js';
+import * as init from '../lib/command/init.js';
+import * as configMigrate from '../lib/command/configMigrate.js';
+import * as interactive from '../lib/command/interactive.js';
+import * as definitions from '../lib/command/definitions.js';
+import * as list from '../lib/command/list.js';
+import * as gherkinInit from '../lib/command/gherkin/init.js';
+import * as gherkinSteps from '../lib/command/gherkin/steps.js';
+import * as gherkinSnippets from '../lib/command/gherkin/snippets.js';
+import * as generate from '../lib/command/generate.js';
+import * as run from '../lib/command/run.js';
+import * as runWorkers from '../lib/command/run-workers.js';
+import * as runMultiple from '../lib/command/run-multiple.js';
+import * as rerun from '../lib/command/run-rerun.js';
+import * as dryRun from '../lib/command/dryRun.js';
+import * as info from '../lib/command/info.js';
 
 const errorHandler = (fn) => async (...args) => {
   try {
@@ -26,42 +41,42 @@ program.version(Codecept.version());
 
 program.command('init [path]')
   .description('Creates dummy config in current dir or [path]')
-  .action(errorHandler(require('../lib/command/init')));
+  .action(errorHandler(init));
 
 program.command('migrate [path]')
   .description('Migrate json config to js config in current dir or [path]')
-  .action(errorHandler(require('../lib/command/configMigrate')));
+  .action(errorHandler(configMigrate));
 
 program.command('shell [path]')
   .alias('sh')
   .description('Interactive shell')
   .option('--verbose', 'output internal logging information')
   .option('--profile [value]', 'configuration profile to be used')
   .option('-c, --config [file]', 'configuration file to be used')
-  .action(errorHandler(require('../lib/command/interactive')));
+  .action(errorHandler(interactive));
 
 program.command('list [path]')
   .alias('l')
   .description('List all actions for I.')
-  .action(errorHandler(require('../lib/command/list')));
+  .action(errorHandler(list));
 
 program.command('def [path]')
   .description('Generates TypeScript definitions for all I actions.')
   .option('-c, --config [file]', 'configuration file to be used')
   .option('-o, --output [folder]', 'target folder to paste definitions')
-  .action(errorHandler(require('../lib/command/definitions')));
+  .action(errorHandler(definitions));
 
 program.command('gherkin:init [path]')
   .alias('bdd:init')
   .description('Prepare CodeceptJS to run feature files.')
   .option('-c, --config [file]', 'configuration file to be used')
-  .action(errorHandler(require('../lib/command/gherkin/init')));
+  .action(errorHandler(gherkinInit));
 
 program.command('gherkin:steps [path]')
   .alias('bdd:steps')
   .description('Prints all defined gherkin steps.')
   .option('-c, --config [file]', 'configuration file to be used')
-  .action(errorHandler(require('../lib/command/gherkin/steps')));
+  .action(errorHandler(gherkinSteps));
 
 program.command('gherkin:snippets [path]')
   .alias('bdd:snippets')
@@ -70,28 +85,28 @@ program.command('gherkin:snippets [path]')
   .option('-c, --config [file]', 'configuration file to be used')
   .option('--feature [file]', 'feature files(s) to scan')
   .option('--path [file]', 'file in which to place the new snippets')
-  .action(errorHandler(require('../lib/command/gherkin/snippets')));
+  .action(errorHandler(gherkinSnippets));
 
 program.command('generate:test [path]')
   .alias('gt')
   .description('Generates an empty test')
-  .action(errorHandler(require('../lib/command/generate').test));
+  .action(errorHandler(generate.test));
 
 program.command('generate:pageobject [path]')
   .alias('gpo')
   .description('Generates an empty page object')
-  .action(errorHandler(require('../lib/command/generate').pageObject));
+  .action(errorHandler(generate.pageObject));
 
 program.command('generate:object [path]')
   .alias('go')
   .option('--type, -t [kind]', 'type of object to be created')
   .description('Generates an empty support object (page/step/fragment)')
-  .action(errorHandler(require('../lib/command/generate').pageObject));
+  .action(errorHandler(generate.pageObject));
 
 program.command('generate:helper [path]')
   .alias('gh')
   .description('Generates a new helper')
-  .action(errorHandler(require('../lib/command/generate').helper));
+  .action(errorHandler(generate.helper));
 
 program.command('run [test]')
   .description('Executes tests')
@@ -128,7 +143,7 @@ program.command('run [test]')
   .option('--recursive', 'include sub directories')
   .option('--trace', 'trace function calls')
   .option('--child <string>', 'option for child processes')
-  .action(errorHandler(require('../lib/command/run')));
+  .action(errorHandler(run));
 
 program.command('run-workers <workers> [selectedRuns...]')
   .description('Executes tests in workers')
@@ -145,7 +160,7 @@ program.command('run-workers <workers> [selectedRuns...]')
   .option('-p, --plugins <k=v,k2=v2,...>', 'enable plugins, comma-separated')
   .option('-O, --reporter-options <k=v,k2=v2,...>', 'reporter-specific options')
   .option('-R, --reporter <name>', 'specify the reporter to use')
-  .action(errorHandler(require('../lib/command/run-workers')));
+  .action(errorHandler(runWorkers));
 
 program.command('run-multiple [suites...]')
   .description('Executes tests multiple')
@@ -169,12 +184,12 @@ program.command('run-multiple [suites...]')
   // mocha options
   .option('--colors', 'force enabling of colors')
 
-  .action(errorHandler(require('../lib/command/run-multiple')));
+  .action(errorHandler(runMultiple));
 
 program.command('info [path]')
   .description('Print debugging information concerning the local environment')
   .option('-c, --config', 'your config file path')
-  .action(errorHandler(require('../lib/command/info')));
+  .action(errorHandler(info));
 
 program.command('dry-run [test]')
   .description('Prints step-by-step scenario for a test without actually running it')
@@ -190,7 +205,7 @@ program.command('dry-run [test]')
   .option('--steps', 'show step-by-step execution')
   .option('--verbose', 'output internal logging information')
   .option('--debug', 'output additional information')
-  .action(errorHandler(require('../lib/command/dryRun')));
+  .action(errorHandler(dryRun));
 
 program.command('run-rerun [test]')
   .description('Executes tests in more than one test suite run')
@@ -227,7 +242,7 @@ program.command('run-rerun [test]')
   .option('--trace', 'trace function calls')
   .option('--child <string>', 'option for child processes')
 
-  .action(require('../lib/command/run-rerun'));
+  .action(rerun);
 
 program.on('command:*', (cmd) => {
   console.log(`\nUnknown command ${cmd}\n`);

lib/actor.js

@@ -1,4 +1,4 @@
-import Step from './step.js';
+import { Step } from './step.js';
 import { MetaStep } from './step.js';
 import container from './container.js';
 import { methodsOfObject } from './utils.js';
@@ -11,7 +11,7 @@ import output from './output.js';
  * @interface
  * @alias ActorStatic
  */
-class Actor {
+export class Actor {
   /**
    * Print the comment on log. Also, adding a step in the `Test.steps` object
    * @param {string} msg
@@ -69,7 +69,8 @@ class Actor {
  * Wraps helper methods into promises.
  * @ignore
  */
-export default function (obj = {}) {
+// eslint-disable-next-line import/prefer-default-export
+export function actor(obj = {}) {
   if (!store.actor) {
     store.actor = new Actor();
   }
@@ -119,7 +120,7 @@ export default function (obj = {}) {
   return actor;
 }
 
-function recordStep(step, args) {
+export function recordStep(step, args) {
   step.status = 'queued';
   step.setArguments(args);
 

lib/assert/include.js

@@ -1,7 +1,7 @@
 import Assertion from '../assert.js';
 import AssertionFailedError from './error.js';
 import { template } from '../utils.js';
-import output from '../output.js';
+import * as output from '../output.js';
 
 const MAX_LINES = 10;
 

lib/cli.js

@@ -23,7 +23,8 @@ class Cli extends Base {
     if (opts.debug) level = 2;
     if (opts.verbose) level = 3;
     output.level(level);
-    output.print(`CodeceptJS v${require('./codecept').version()} ${output.standWithUkraine()}`);
+    const version = require('./codecept.js').version();
+    output.print(`CodeceptJS v${version} ${output.standWithUkraine()}`);
     output.print(`Using test root "${global.codecept_dir}"`);
 
     const showSteps = level >= 1;

lib/codecept.js

@@ -12,7 +12,7 @@ import path from "path";
 import generated from "@codeceptjs/helper";
 import * as index from './index.js';
 
-import actor0 from "./actor.js";
+import * as actor0 from "./actor.js";
 
 import pause0 from "./pause.js";
 

lib/command/definitions.js

@@ -1,10 +1,12 @@
 import fs from 'fs';
 import path from 'path';
 import { getConfig, getTestRoot } from './utils.js';
-import Codecept from '../codecept';
+import Codecept from '../../lib/codecept.js';
 import container from '../container.js';
 import output from '../output.js';
-const actingHelpers = [...require('../plugin/standardActingHelpers.js'), 'REST'];
+import plugin from "../plugin/standardActingHelpers.js";
+
+const actingHelpers = [...(plugin), 'REST'];
 
 /**
  * Prepare data and generate content of definitions file

lib/command/dryRun.js

@@ -1,6 +1,6 @@
 import { getConfig, getTestRoot } from './utils.js';
 import Config from '../config.js';
-import Codecept from '../codecept';
+import Codecept from '../codecept.js';
 import output from '../output.js';
 import * as event from '../event.js';
 import { store } from '../store.js';

lib/command/generate.js

@@ -5,7 +5,7 @@ import mkdirp from 'mkdirp';
 import path from 'path';
 import { fileExists, ucfirst, lcfirst, beautify } from '../utils.js';
 import output from '../output.js';
-import generateDefinitions from './definitions';
+import generateDefinitions from './definitions.js';
 import { getConfig, getTestRoot, safeFileWrite, readConfig } from './utils.js';
 
 let extension = 'js';

lib/command/gherkin/snippets.js

@@ -5,7 +5,7 @@ import * as Messages from '@cucumber/messages';
 import glob from 'glob';
 import fsPath from 'path';
 import { getConfig, getTestRoot } from '../utils.js';
-import Codecept from '../../codecept';
+import Codecept from '../../codecept.js';
 import output from '../../output.js';
 import { matchStep } from '../../interfaces/bdd.js';