Improve path matching in RESTEasy Reactive for ambiguous path templates

This solves further problems in case when the paths are spread over
multiple resource classes. Improves earlier PR:

#47386

Author: TomasHofman

independent-projects/resteasy-reactive/server/runtime/src/main/java/org/jboss/resteasy/reactive/server/mapping/RequestMapper.java

@@ -72,16 +72,21 @@ public RequestMatch<T> continueMatching(String path, RequestMatch<T> lastMatch)
             return null;
         }
 
-        var initialMatches = requestPaths.match(path).get(0);
-        var result = mapFromPathMatcher(path, initialMatches, 0);
-        if (result != null) {
-            int idx = nextMatchStartingIndex(initialMatches, lastMatch);
-            return mapFromPathMatcher(path, initialMatches, idx);
+        var initialMatchesList = requestPaths.match(path);
+        for (var initialMatches : initialMatchesList) {
+            var result = mapFromPathMatcher(path, initialMatches, 0);
+            if (result != null) {
+                int idx = nextMatchStartingIndex(initialMatches, lastMatch);
+                RequestMatch<T> match = mapFromPathMatcher(path, initialMatches, idx);
+                if (match != null) {
+                    return match;
+                }
+            }
         }
 
         // the following code is meant to handle cases like https://github.com/quarkusio/quarkus/issues/30667
-        initialMatches = requestPaths.defaultMatch(path);
-        result = mapFromPathMatcher(path, initialMatches, 0);
+        var initialMatches = requestPaths.defaultMatch(path);
+        var result = mapFromPathMatcher(path, initialMatches, 0);
         if (result != null) {
             int idx = nextMatchStartingIndex(initialMatches, lastMatch);
             return mapFromPathMatcher(path, initialMatches, idx);
@@ -105,7 +110,7 @@ private int nextMatchStartingIndex(PathMatcher.PathMatch<ArrayList<RequestPath<T
             }
         }
 
-        return -1;
+        return 0;
     }
 
     @SuppressWarnings({ "rawtypes", "unchecked" })

independent-projects/resteasy-reactive/server/vertx/src/test/java/org/jboss/resteasy/reactive/server/vertx/test/matching/PathParamOverlapTest.java

@@ -25,7 +25,7 @@ public class PathParamOverlapTest {
                 @Override
                 public JavaArchive get() {
                     return ShrinkWrap.create(JavaArchive.class)
-                            .addClasses(TestResource.class);
+                            .addClasses(TestResource.class, AnotherResource.class);
                 }
             });
 
@@ -82,15 +82,11 @@ public void test() {
                 .then()
                 .statusCode(200)
                 .body(equalTo("Foo bar_value"));
-    }
 
-    // TODO: remove this test before commit
-    @Test
-    public void test2() {
-        get("/hello/foo/bar_value")
+        get("/hello/foo/bar/foo/bar")
                 .then()
                 .statusCode(200)
-                .body(equalTo("Foo bar_value"));
+                .body(equalTo("FooBarFooBar"));
     }
 
     @Path("/hello")
@@ -122,4 +118,16 @@ public String fooBar(@RestPath String param) {
         }
 
     }
+
+    /**
+     * This is to also test paths spread over multiple resource classes.
+     */
+    @Path("/hello/foo/bar")
+    public static class AnotherResource {
+        @GET
+        @Path("/foo/bar")
+        public String fooBarFooBar() {
+            return "FooBarFooBar";
+        }
+    }
 }