fix: include original coordinates in support code library (#2197)

CHANGELOG.md

@@ -8,6 +8,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 Please see [CONTRIBUTING.md](https://github.com/cucumber/cucumber/blob/master/CONTRIBUTING.md) on how to contribute to Cucumber.
 
 ## [Unreleased]
+### Fixed
+- Include original coordinates in `loadSupport` result ([#2197](https://github.com/cucumber/cucumber-js/pull/2197))
 
 ## [8.9.0] - 2022-11-24
 ### Added

src/api/load_support_spec.ts

@@ -0,0 +1,25 @@
+import { IRunEnvironment } from './types'
+import path from 'path'
+import { loadSupport } from './load_support'
+import { loadConfiguration } from './load_configuration'
+import { expect } from 'chai'
+import { setupEnvironment, teardownEnvironment } from './test_helpers'
+
+describe('loadSupport', () => {
+  let environment: IRunEnvironment
+  beforeEach(async () => {
+    environment = await setupEnvironment()
+  })
+  afterEach(async () => teardownEnvironment(environment))
+
+  it('should include original paths in the returned support code library', async () => {
+    const { runConfiguration } = await loadConfiguration({}, environment)
+    const support = await loadSupport(runConfiguration, environment)
+
+    expect(support.originalCoordinates).to.deep.eq({
+      requireModules: ['ts-node/register'],
+      requirePaths: [path.join(environment.cwd, 'features', 'steps.ts')],
+      importPaths: [],
+    })
+  })
+})

src/api/run_cucumber_spec.ts

@@ -1,44 +1,10 @@
-import { Envelope, TestStepResultStatus, IdGenerator } from '@cucumber/messages'
-import fs from 'mz/fs'
-import path from 'path'
-import { reindent } from 'reindent-template-literals'
-import { PassThrough } from 'stream'
+import { Envelope, TestStepResultStatus } from '@cucumber/messages'
 import { expect } from 'chai'
 import { runCucumber } from './run_cucumber'
 import { IRunEnvironment } from './types'
 import { loadSupport } from './load_support'
 import { loadConfiguration } from './load_configuration'
-
-const newId = IdGenerator.uuid()
-
-async function setupEnvironment(): Promise<Partial<IRunEnvironment>> {
-  const cwd = path.join(__dirname, '..', '..', 'tmp', `runCucumber_${newId()}`)
-  await fs.mkdir(path.join(cwd, 'features'), { recursive: true })
-  await fs.writeFile(
-    path.join(cwd, 'features', 'test.feature'),
-    reindent(`Feature: test fixture
-      Scenario: one
-        Given a step
-        Then another step`)
-  )
-  await fs.writeFile(
-    path.join(cwd, 'features', 'steps.ts'),
-    reindent(`import { Given, Then } from '../../../src'
-    Given('a step', function () {})
-    Then('another step', function () {})`)
-  )
-  await fs.writeFile(
-    path.join(cwd, 'cucumber.mjs'),
-    `export default {paths: ['features/test.feature'], requireModule: ['ts-node/register'], require: ['features/steps.ts']}`
-  )
-  const stdout = new PassThrough()
-  return { cwd, stdout }
-}
-
-async function teardownEnvironment(environment: IRunEnvironment) {
-  await fs.rmdir(environment.cwd, { recursive: true })
-  environment.stdout.end()
-}
+import { setupEnvironment, teardownEnvironment } from './test_helpers'
 
 describe('runCucumber', () => {
   describe('preloading support code', () => {

src/api/support.ts

@@ -19,7 +19,11 @@ export async function getSupportCodeLibrary({
   requirePaths: string[]
   importPaths: string[]
 }): Promise<ISupportCodeLibrary> {
-  supportCodeLibraryBuilder.reset(cwd, newId)
+  supportCodeLibraryBuilder.reset(cwd, newId, {
+    requireModules,
+    requirePaths,
+    importPaths,
+  })
   requireModules.map((module) => require(module))
   requirePaths.map((path) => require(path))
   for (const path of importPaths) {

src/api/test_helpers.ts

@@ -0,0 +1,37 @@
+import { IRunEnvironment } from './types'
+import path from 'path'
+import fs from 'mz/fs'
+import { reindent } from 'reindent-template-literals'
+import { PassThrough } from 'stream'
+import { IdGenerator } from '@cucumber/messages'
+
+const newId = IdGenerator.uuid()
+
+export async function setupEnvironment(): Promise<Partial<IRunEnvironment>> {
+  const cwd = path.join(__dirname, '..', '..', 'tmp', `api_${newId()}`)
+  await fs.mkdir(path.join(cwd, 'features'), { recursive: true })
+  await fs.writeFile(
+    path.join(cwd, 'features', 'test.feature'),
+    reindent(`Feature: test fixture
+      Scenario: one
+        Given a step
+        Then another step`)
+  )
+  await fs.writeFile(
+    path.join(cwd, 'features', 'steps.ts'),
+    reindent(`import { Given, Then } from '../../../src'
+    Given('a step', function () {})
+    Then('another step', function () {})`)
+  )
+  await fs.writeFile(
+    path.join(cwd, 'cucumber.mjs'),
+    `export default {paths: ['features/test.feature'], requireModule: ['ts-node/register'], require: ['features/steps.ts']}`
+  )
+  const stdout = new PassThrough()
+  return { cwd, stdout }
+}
+
+export async function teardownEnvironment(environment: IRunEnvironment) {
+  await fs.rmdir(environment.cwd, { recursive: true })
+  environment.stdout.end()
+}

src/runtime/parallel/worker.ts

@@ -70,7 +70,11 @@ export default class Worker {
     supportCodeIds,
     options,
   }: IWorkerCommandInitialize): Promise<void> {
-    supportCodeLibraryBuilder.reset(this.cwd, this.newId)
+    supportCodeLibraryBuilder.reset(this.cwd, this.newId, {
+      requireModules,
+      requirePaths,
+      importPaths,
+    })
     requireModules.map((module) => require(module))
     requirePaths.map((module) => require(module))
     for (const path of importPaths) {