fix: derive matches based on matchAll length

Author: kettanaito

src/core/utils/matching/matchRequestUrl.test.ts

@@ -35,6 +35,25 @@ it('returns false when a RegExp does not match', () => {
       params: {},
     },
   )
+
+  expect(
+    matchRequestUrl(new URL('https://test.mswjs.io'), /foo\.bar/g),
+  ).toEqual({
+    matches: false,
+    params: {},
+  })
+
+  expect(matchRequestUrl(new URL('https://test.mswjs.io'), /foo(.+)/)).toEqual({
+    matches: false,
+    params: {},
+  })
+
+  expect(matchRequestUrl(new URL('https://test.mswjs.io'), /foo(.+)/g)).toEqual(
+    {
+      matches: false,
+      params: {},
+    },
+  )
 })
 
 it('ignores query parameters when matching against a RegExp', () => {

src/core/utils/matching/matchRequestUrl.ts

@@ -32,7 +32,7 @@ export function matchRequestUrl(
       : cleanUrl.match(pattern)
 
     return {
-      matches: match != null,
+      matches: match != null && match.length > 0,
       params: Object.fromEntries(
         (match ?? []).slice(1).map((value, index) => [index, value]),
       ),