[JUnit Platform Engine] Accept partial matches in name filter (#3174)

In 'CucumberTestDescriptor', 'cucumber.filter.name' was evaluated using
'matches()'which requires the pattern to cover the full scenario name.
Replaced with 'find()' to align with 'NamePredicate' used by JUnit 4
and CLI.

Fixes #3166. Non-anchored patterns like 'Cool test' now correctly match
any scenario whose name contains that substring, consistent with JUnit
4 and the CLI.

Author: mars1589

CHANGELOG.md

@@ -13,6 +13,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Added
 - [Java] Add optional names to `@Before`, `@After`, `@BeforeStep` and `@AfterStep` hooks and emit hook names in messages ([#2917](https://github.com/cucumber/cucumber-jvm/issues/2917), [#3173](https://github.com/cucumber/cucumber-jvm/pull/3173))
 
+### Fixed
+- [JUnit Platform Engine] Accept partial matches with `cucumber.filter.name` and align the behavior with JUnit 4 and CLI  ([#3174](https://github.com/cucumber/cucumber-jvm/pull/3174))
+
 ### Changed
 - [All] Set baseline to Java 17 ([#3116](https://github.com/cucumber/cucumber-jvm/pull/3116))
 - [All] Adopt [JSpecify](https://jspecify.dev/) to declare nullability ([#3116](https://github.com/cucumber/cucumber-jvm/pull/3116))

cucumber-junit-platform-engine/src/main/java/io/cucumber/junit/platform/engine/CucumberTestDescriptor.java

@@ -292,7 +292,7 @@ private Optional<SkipResult> shouldBeSkippedByTagFilter(CucumberEngineExecutionC
 
         private Optional<SkipResult> shouldBeSkippedByNameFilter(CucumberEngineExecutionContext context) {
             return context.getConfiguration().nameFilter().map(pattern -> {
-                if (pattern.matcher(pickle.getName()).matches()) {
+                if (pattern.matcher(pickle.getName()).find()) {
                     return SkipResult.doNotSkip();
                 }
                 return SkipResult

cucumber-junit-platform-engine/src/test/java/io/cucumber/junit/platform/engine/CucumberTestEngineTest.java

@@ -690,6 +690,51 @@ void selectAndSkipDisabledScenarioByName() {
                     event(skippedWithReason("'cucumber.filter.name=^Nothing$' did not match this scenario"))));
     }
 
+    @Test
+    void anchoredNamePatternMatchesExactName() {
+        EngineTestKit.engine(ENGINE_ID)
+                .configurationParameter(FILTER_NAME_PROPERTY_NAME, "^A single scenario$")
+                .selectors(selectFile("src/test/resources/io/cucumber/junit/platform/engine/single.feature"))
+                .execute()
+                .testEvents()
+                .assertThatEvents()
+                .haveExactly(1, event(test(), finishedSuccessfully()));
+    }
+
+    @Test
+    void anchoredNamePatternDoesNotMatchPartOfName() {
+        EngineTestKit.engine(ENGINE_ID)
+                .configurationParameter(FILTER_NAME_PROPERTY_NAME, "^A single$")
+                .selectors(selectFile("src/test/resources/io/cucumber/junit/platform/engine/single.feature"))
+                .execute()
+                .testEvents()
+                .assertThatEvents()
+                .haveExactly(1, event(test(),
+                    event(skippedWithReason("'cucumber.filter.name=^A single$' did not match this scenario"))));
+    }
+
+    @Test
+    void nonAnchoredNamePatternMatchesPartOfName() {
+        EngineTestKit.engine(ENGINE_ID)
+                .configurationParameter(FILTER_NAME_PROPERTY_NAME, "single")
+                .selectors(selectFile("src/test/resources/io/cucumber/junit/platform/engine/single.feature"))
+                .execute()
+                .testEvents()
+                .assertThatEvents()
+                .haveExactly(1, event(test(), finishedSuccessfully()));
+    }
+
+    @Test
+    void wildcardNamePatternMatchesPartOfName() {
+        EngineTestKit.engine(ENGINE_ID)
+                .configurationParameter(FILTER_NAME_PROPERTY_NAME, "single .*")
+                .selectors(selectFile("src/test/resources/io/cucumber/junit/platform/engine/single.feature"))
+                .execute()
+                .testEvents()
+                .assertThatEvents()
+                .haveExactly(1, event(test(), finishedSuccessfully()));
+    }
+
     @Test
     void cucumberTagsAreConvertedToJunitTags() {
         EngineTestKit.engine(ENGINE_ID)