WIP

mpkorstanje · mpkorstanje · commit 9c02efcc1805 · 2024-06-21T15:29:14.000+02:00
diff --git a/java/src/main/java/io/cucumber/query/Query.java b/java/src/main/java/io/cucumber/query/Query.java
@@ -186,10 +186,11 @@ public String findNameOf(TableRow element, NamingStrategy namingStrategy) {
                 .orElseThrow(createElementWasNotPartOfThisQueryObject());
     }
 
-    public Optional<String> findNameOf(Pickle element, NamingStrategy namingStrategy) {
+    public String findNameOf(Pickle element, NamingStrategy namingStrategy) {
         requireNonNull(element);
         requireNonNull(namingStrategy);
-        return reduceLinageOf(element, namingStrategy);
+        return reduceLinageOf(element, namingStrategy)
+                .orElseGet(element::getName);
     }
 
     private static Supplier<IllegalArgumentException> createElementWasNotPartOfThisQueryObject() {

Original file line number	Diff line number	Diff line change
`@@ -186,10 +186,11 @@ public String findNameOf(TableRow element, NamingStrategy namingStrategy) {`
`186`	`186`	`.orElseThrow(createElementWasNotPartOfThisQueryObject());`
`187`	`187`	`}`
`188`	`188`
`189`		`- public Optional<String> findNameOf(Pickle element, NamingStrategy namingStrategy) {`
	`189`	`+ public String findNameOf(Pickle element, NamingStrategy namingStrategy) {`
`190`	`190`	`requireNonNull(element);`
`191`	`191`	`requireNonNull(namingStrategy);`
`192`		`- return reduceLinageOf(element, namingStrategy);`
	`192`	`+ return reduceLinageOf(element, namingStrategy)`
	`193`	`+ .orElseGet(element::getName);`
`193`	`194`	`}`
`194`	`195`
`195`	`196`	`private static Supplier<IllegalArgumentException> createElementWasNotPartOfThisQueryObject() {`