[JUnit Platform] Use EngineDiscoveryRequestResolver (#2835)

Cucumber now uses JUnits `EngineDiscoveryRequestResolver`. JUnit
provides a `EngineDiscoveryRequestResolver` that handles the resolution
of most resources. This removes some complexity from Cucumber.  This
comes with several implications.

#### Classpath resources must be singular feature files. 

There is now a distinction between the classpath resource and package
discovery selectors. The former should be used for individual feature
files on the classpath, the latter for directories.

So instead of using a class path resource selector for a directory:

```java
@suite
@IncludeEngines("cucumber")
@SelectClasspathResource("com/example")
public class RunCucumberTest {
}
```

The package selector should be used instead:

```java
@suite
@IncludeEngines("cucumber")
@SelectPackages("com.example")
public class RunCucumberTest {
}
```

This is still okay:

```java
@suite
@IncludeEngines("cucumber")
@SelectClasspathResource("com/example/my.feature")
public class RunCucumberTest {
}
```

Using the classpath resource selector on a folder does still work, but
the functionality has been deprecated will result in a warning.

####  Support `cucumber.execution.order`

Features and scenarios are now discovered in arbitrary order and ordered
after discovery. This order can be controlled with the
`cucumber.execution.order` property. By default features are executed in
`lexical` uri order, with scenarios and examples from top to bottom.
Other valid options are `reverse` and `random`.

There is room to add custom orderer here but this hasn't been
implemented yet.

#### Use JUnits discovery issue reporting mechanism

Any problems with the build are now reported through JUnits discovery
issue reporting mechanism. This provides a more robust report than
logging warnings.

Author: mpkorstanje

.github/workflows/test-java.yml

@@ -33,8 +33,6 @@ jobs:
         run: ./mvnw install -Pinclude-extra-modules -DskipTests=true -DskipITs=true -D"archetype.test.skip=true" -D"maven.javadoc.skip=true" --batch-mode -D"style.color=always" --show-version
       - name: Test
         run: ./mvnw verify -Pinclude-extra-modules -D"style.color=always"
-        env:
-          CUCUMBER_PUBLISH_TOKEN: ${{ secrets.CUCUMBER_PUBLISH_TOKEN }}
 
   javadoc:
     name: 'Javadoc'

CHANGELOG.md

@@ -10,9 +10,19 @@ The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
 ## [Unreleased]
+### Added
+- [JUnit Platform Engine] Option to include a parameterized scenario name only if the scenario is parameterized ([#2835](https://github.com/cucumber/cucumber-jvm/pull/2835) M.P. Korstanje)
+- [JUnit Platform Engine] Option to order features and scenarios ([#2835](https://github.com/cucumber/cucumber-jvm/pull/2835) M.P. Korstanje)
+- [JUnit Platform Engine] Log discovery issues when a classpath resource selector is (e.g. `@SelectClasspathResource`) is used to select a directory. ([#2835](https://github.com/cucumber/cucumber-jvm/pull/2835) M.P. Korstanje)
+
 ### Changed
+- [JUnit Platform Engine] Use JUnit's `EngineDiscoveryRequestResolver` to resolve classpath based resources. ([#2835](https://github.com/cucumber/cucumber-jvm/pull/2835) M.P. Korstanje)
 - [JUnit Platform Engine] Use JUnit Platform 1.13.1 (JUnit Jupiter 5.13.1)
 
+### Fixed
+- [JUnit Platform Engine] Log discovery issues for feature files with parse errors. ([#2835](https://github.com/cucumber/cucumber-jvm/pull/2835) M.P. Korstanje)
+
+
 ## [7.23.0] - 2025-05-29
 ### Added
 - [JUnit Platform Engine, TestNG] Remove framework elements from `UndefinedStepException` stacktrace ([#3002](https://github.com/cucumber/cucumber-jvm/pull/3002) M.P. Korstanje)

cucumber-core/src/main/java/io/cucumber/core/feature/FeatureIdentifier.java

@@ -32,11 +32,15 @@ public static URI parse(URI featureIdentifier) {
     }
 
     public static boolean isFeature(URI featureIdentifier) {
-        return featureIdentifier.getSchemeSpecificPart().endsWith(FEATURE_FILE_SUFFIX);
+        return isFeature(featureIdentifier.getSchemeSpecificPart());
     }
 
     public static boolean isFeature(Path path) {
-        return path.getFileName().toString().endsWith(FEATURE_FILE_SUFFIX);
+        return isFeature(path.getFileName().toString());
+    }
+
+    public static boolean isFeature(String fileName) {
+        return fileName.endsWith(FEATURE_FILE_SUFFIX);
     }
 
 }

cucumber-core/src/main/java/io/cucumber/core/options/Constants.java

@@ -42,7 +42,8 @@ public final class Constants {
      * Valid values are {@code lexical}, {@code reverse}, {@code random} or
      * {@code random:[seed]}.
      * <p>
-     * By default features are executed in lexical file name order
+     * By default, features are executed in lexical file name order and
+     * scenarios in a feature from top to bottom.
      */
     public static final String EXECUTION_ORDER_PROPERTY_NAME = "cucumber.execution.order";
 

cucumber-core/src/main/java/io/cucumber/core/order/StandardPickleOrders.java

@@ -8,8 +8,9 @@
 
 public final class StandardPickleOrders {
 
-    private static final Comparator<Pickle> pickleUriComparator = Comparator.comparing(Pickle::getUri)
-            .thenComparing(pickle -> pickle.getLocation().getLine());
+    private static final Comparator<Pickle> pickleUriComparator = Comparator
+            .comparing(Pickle::getUri)
+            .thenComparing(Pickle::getLocation);
 
     private StandardPickleOrders() {
 

cucumber-core/src/main/java/io/cucumber/core/resource/PathScanner.java

@@ -2,6 +2,7 @@
 
 import io.cucumber.core.logging.Logger;
 import io.cucumber.core.logging.LoggerFactory;
+import org.apiguardian.api.API;
 
 import java.io.IOException;
 import java.net.URI;
@@ -20,8 +21,10 @@
 import static java.nio.file.FileVisitResult.CONTINUE;
 import static java.nio.file.Files.exists;
 import static java.nio.file.Files.walkFileTree;
+import static org.apiguardian.api.API.Status.INTERNAL;
 
-class PathScanner {
+@API(status = INTERNAL)
+public class PathScanner {
 
     private static final Logger log = LoggerFactory.getLogger(PathScanner.class);
 
@@ -48,10 +51,14 @@ void findResourcesForPath(Path path, Predicate<Path> filter, Function<Path, Cons
         if (!exists(path)) {
             throw new IllegalArgumentException("path must exist: " + path);
         }
+        findResourcesForPath(path, filter, consumer.apply(path));
+    }
 
+    public void findResourcesForPath(Path path, Predicate<Path> filter, Consumer<Path> consumer) {
         try {
-            walkFileTree(path, EnumSet.of(FileVisitOption.FOLLOW_LINKS), Integer.MAX_VALUE,
-                new ResourceFileVisitor(filter, consumer.apply(path)));
+            EnumSet<FileVisitOption> options = EnumSet.of(FileVisitOption.FOLLOW_LINKS);
+            ResourceFileVisitor visitor = new ResourceFileVisitor(filter, consumer);
+            walkFileTree(path, options, Integer.MAX_VALUE, visitor);
         } catch (IOException e) {
             throw new RuntimeException(e);
         }

cucumber-gherkin-messages/src/main/java/io/cucumber/core/gherkin/messages/GherkinMessagesExample.java

@@ -4,6 +4,7 @@
 import io.cucumber.plugin.event.Location;
 import io.cucumber.plugin.event.Node;
 
+import java.net.URI;
 import java.util.Optional;
 
 final class GherkinMessagesExample implements Node.Example {
@@ -20,6 +21,11 @@ final class GherkinMessagesExample implements Node.Example {
         this.rowIndex = rowIndex;
     }
 
+    @Override
+    public URI getUri() {
+        return parent.getUri();
+    }
+
     @Override
     public Location getLocation() {
         return GherkinMessagesLocation.from(tableRow.getLocation());

cucumber-gherkin-messages/src/main/java/io/cucumber/core/gherkin/messages/GherkinMessagesExamples.java

@@ -3,6 +3,7 @@
 import io.cucumber.plugin.event.Location;
 import io.cucumber.plugin.event.Node;
 
+import java.net.URI;
 import java.util.Collection;
 import java.util.List;
 import java.util.Optional;
@@ -31,6 +32,11 @@ public Collection<Example> elements() {
         return children;
     }
 
+    @Override
+    public URI getUri() {
+        return parent.getUri();
+    }
+
     @Override
     public Location getLocation() {
         return location;

cucumber-gherkin-messages/src/main/java/io/cucumber/core/gherkin/messages/GherkinMessagesRule.java

@@ -4,6 +4,7 @@
 import io.cucumber.plugin.event.Location;
 import io.cucumber.plugin.event.Node;
 
+import java.net.URI;
 import java.util.Collection;
 import java.util.List;
 import java.util.Optional;
@@ -42,6 +43,11 @@ public Collection<Node> elements() {
         return children;
     }
 
+    @Override
+    public URI getUri() {
+        return parent.getUri();
+    }
+
     @Override
     public Location getLocation() {
         return GherkinMessagesLocation.from(rule.getLocation());

cucumber-gherkin-messages/src/main/java/io/cucumber/core/gherkin/messages/GherkinMessagesScenario.java

@@ -3,6 +3,7 @@
 import io.cucumber.plugin.event.Location;
 import io.cucumber.plugin.event.Node;
 
+import java.net.URI;
 import java.util.Optional;
 
 final class GherkinMessagesScenario implements Node.Scenario {
@@ -20,6 +21,11 @@ public Optional<Node> getParent() {
         return Optional.of(parent);
     }
 
+    @Override
+    public URI getUri() {
+        return parent.getUri();
+    }
+
     @Override
     public Location getLocation() {
         return GherkinMessagesLocation.from(scenario.getLocation());