esm: use specific option for import vs require for support code

compatibility/cck_spec.ts

@@ -36,8 +36,11 @@ describe('Cucumber Compatibility Kit', () => {
           paths: [`${CCK_FEATURES_PATH}/${suiteName}/${suiteName}${extension}`],
         },
         support: {
-          transpileWith: ['ts-node/register'],
-          paths: [`${CCK_IMPLEMENTATIONS_PATH}/${suiteName}/${suiteName}.ts`],
+          requireModules: ['ts-node/register'],
+          requirePaths: [
+            `${CCK_IMPLEMENTATIONS_PATH}/${suiteName}/${suiteName}.ts`,
+          ],
+          importPaths: [],
         },
         formats: {
           stdout: 'message',

features/esm.feature

@@ -3,7 +3,7 @@ Feature: ES modules support
 
   cucumber-js works with native ES modules
 
-  Scenario Outline: native module syntax works in support code, formatters and snippets
+  Scenario: native module syntax works in support code, formatters and snippets
     Given a file named "features/a.feature" with:
       """
       Feature:
@@ -39,18 +39,38 @@ Feature: ES modules support
         'default': '--format summary'
       }
       """
-    When I run cucumber-js with `<options> --format ./custom-formatter.js --format-options '{"snippetSyntax": "./custom-snippet-syntax.js"}' <args>`
-    Then it passes
-    Examples:
-      | args         |
-      |              |
-      | --parallel 2 |
+    When I run cucumber-js with `--import features/**/*.js --format ./custom-formatter.js --format-options '{"snippetSyntax": "./custom-snippet-syntax.js"}'`
+    Then it runs 2 scenarios
+    And it passes
 
-  Scenario: .mjs support code files are matched by default
+  Scenario: native modules work with parallel runtime
     Given a file named "features/a.feature" with:
       """
       Feature:
-        Scenario:
+        Scenario: one
+          Given a step passes
+
+        Scenario: two
+          Given a step passes
+      """
+    And a file named "features/step_definitions/cucumber_steps.js" with:
+      """
+      import {Given} from '@cucumber/cucumber'
+
+      Given(/^a step passes$/, function() {});
+      """
+    When I run cucumber-js with `--import features/**/*.js' --parallel 2`
+    Then it runs 2 scenarios
+    And it passes
+
+  Scenario: .mjs support code files are discovered automatically if no requires or imports specified
+    Given a file named "features/a.feature" with:
+      """
+      Feature:
+        Scenario: one
+          Given a step passes
+
+        Scenario: two
           Given a step passes
       """
     And a file named "features/step_definitions/cucumber_steps.mjs" with:
@@ -60,4 +80,5 @@ Feature: ES modules support
       Given(/^a step passes$/, function() {});
       """
     When I run cucumber-js
-    Then it passes
+    Then it runs 2 scenarios
+    And it passes

src/cli/argv_parser.ts

@@ -30,6 +30,7 @@ export interface IParsedArgvOptions {
   formatOptions: IParsedArgvFormatOptions
   i18nKeywords: string
   i18nLanguages: boolean
+  import: string[]
   language: string
   name: string[]
   order: PickleOrder
@@ -140,6 +141,12 @@ const ArgvParser = {
         ''
       )
       .option('--i18n-languages', 'list languages', false)
+      .option(
+        '--import <GLOB|DIR|FILE>',
+        'import files before executing features (repeatable)',
+        ArgvParser.collect,
+        []
+      )
       .option(
         '--language <ISO 639-1>',
         'provide the default language for feature files',

src/cli/configuration_builder.ts

@@ -16,8 +16,9 @@ export async function buildConfiguration(
       order: options.order,
     },
     support: {
-      transpileWith: options.requireModule,
-      paths: options.require,
+      requireModules: options.requireModule,
+      requirePaths: options.require,
+      importPaths: options.import,
     },
     runtime: {
       dryRun: options.dryRun,

src/cli/configuration_builder_spec.ts

@@ -33,8 +33,9 @@ describe('buildConfiguration', () => {
         tagExpression: '',
       },
       support: {
-        paths: [],
-        transpileWith: [],
+        requireModules: [],
+        requirePaths: [],
+        importPaths: [],
       },
     })
   })

src/configuration/types.ts

@@ -11,8 +11,9 @@ export interface IRunConfiguration {
     order?: PickleOrder
   }
   support: {
-    transpileWith: string[]
-    paths: string[]
+    requireModules: string[]
+    requirePaths: string[]
+    importPaths: string[]
   }
   runtime?: Partial<IRuntimeOptions> & { parallel?: number }
   formats?: IFormatterConfiguration

src/run/paths.ts

@@ -10,7 +10,8 @@ export async function resolvePaths(
 ): Promise<{
   unexpandedFeaturePaths: string[]
   featurePaths: string[]
-  supportCodePaths: string[]
+  requirePaths: string[]
+  importPaths: string[]
 }> {
   const unexpandedFeaturePaths = await getUnexpandedFeaturePaths(
     cwd,
@@ -20,19 +21,17 @@ export async function resolvePaths(
     cwd,
     unexpandedFeaturePaths
   )
-  let unexpandedSupportCodePaths = configuration.support.paths ?? []
-  if (unexpandedSupportCodePaths.length === 0) {
-    unexpandedSupportCodePaths = getFeatureDirectoryPaths(cwd, featurePaths)
-  }
-  const supportCodePaths = await expandPaths(
+  const { requirePaths, importPaths } = await deriveSupportPaths(
     cwd,
-    unexpandedSupportCodePaths,
-    '.@(js|mjs)'
+    featurePaths,
+    configuration.support.requirePaths,
+    configuration.support.importPaths
   )
   return {
     unexpandedFeaturePaths,
     featurePaths,
-    supportCodePaths,
+    requirePaths,
+    importPaths,
   }
 }
 
@@ -114,3 +113,32 @@ async function expandFeaturePaths(
   featurePaths = [...new Set(featurePaths)] // Deduplicate the feature files
   return await expandPaths(cwd, featurePaths, '.feature')
 }
+
+async function deriveSupportPaths(
+  cwd: string,
+  featurePaths: string[],
+  unexpandedRequirePaths: string[],
+  unexpandedImportPaths: string[]
+): Promise<{
+  requirePaths: string[]
+  importPaths: string[]
+}> {
+  if (
+    unexpandedRequirePaths.length === 0 &&
+    unexpandedImportPaths.length === 0
+  ) {
+    const defaultPaths = getFeatureDirectoryPaths(cwd, featurePaths)
+    const requirePaths = await expandPaths(cwd, defaultPaths, '.js')
+    const importPaths = await expandPaths(cwd, defaultPaths, '.mjs')
+    return { requirePaths, importPaths }
+  }
+  const requirePaths =
+    unexpandedRequirePaths.length > 0
+      ? await expandPaths(cwd, unexpandedRequirePaths, '.js')
+      : []
+  const importPaths =
+    unexpandedImportPaths.length > 0
+      ? await expandPaths(cwd, unexpandedImportPaths, '.@(js|cjs|mjs)')
+      : []
+  return { requirePaths, importPaths }
+}