honour order of paths in configuration (#2345)

CHANGELOG.md

@@ -8,6 +8,8 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 Please see [CONTRIBUTING.md](./CONTRIBUTING.md) on how to contribute to Cucumber.
 
 ## [Unreleased]
+### Fixed
+- Honour order of paths in configuration ([#2345](https://github.com/cucumber/cucumber-js/pull/2345))
 
 ## [10.0.0] - 2023-10-09
 ### Added

src/api/paths.ts

@@ -72,11 +72,10 @@ async function expandPaths(
           return [match]
         })
       )
-      return expanded.flat()
+      return expanded.flat().sort()
     })
   )
   const normalized = expandedPaths.flat().map((x) => path.normalize(x))
-  normalized.sort()
   return [...new Set(normalized)]
 }
 

src/api/paths_spec.ts

@@ -200,6 +200,58 @@ describe('resolvePaths', () => {
     })
   })
 
+  describe('multiple paths ordering', async () => {
+    it('should honour the provided order of multiple files', async () => {
+      // Arrange
+      const cwd = await buildTestWorkingDirectory()
+      const featurePathA = path.join(cwd, 'features', 'a.feature')
+      const featurePathB = path.join(cwd, 'features', 'b.feature')
+      await fsExtra.outputFile(featurePathA, '')
+      await fsExtra.outputFile(featurePathB, '')
+      // Act
+      const { featurePaths } = await resolvePaths(
+        new FakeLogger(),
+        cwd,
+        {
+          paths: ['features/b.feature', 'features/a.feature'],
+        },
+        {
+          requireModules: [],
+          requirePaths: [],
+          importPaths: [],
+        }
+      )
+
+      // Assert
+      expect(featurePaths).to.eql([featurePathB, featurePathA])
+    })
+
+    it('should honour the provided order of multiple directories', async () => {
+      // Arrange
+      const cwd = await buildTestWorkingDirectory()
+      const featurePathA = path.join(cwd, 'features-a', 'something.feature')
+      const featurePathB = path.join(cwd, 'features-b', 'something.feature')
+      await fsExtra.outputFile(featurePathA, '')
+      await fsExtra.outputFile(featurePathB, '')
+      // Act
+      const { featurePaths } = await resolvePaths(
+        new FakeLogger(),
+        cwd,
+        {
+          paths: ['features-b', 'features-a'],
+        },
+        {
+          requireModules: [],
+          requirePaths: [],
+          importPaths: [],
+        }
+      )
+
+      // Assert
+      expect(featurePaths).to.eql([featurePathB, featurePathA])
+    })
+  })
+
   describe('path to an empty rerun file', () => {
     it('returns empty featurePaths and support code paths', async function () {
       // Arrange