fix: ignore other errors when unable to match path/method (#45)

Author: vwong

src/__tests__/fixtures/request-parameters-path/baseline.json

@@ -338,5 +338,90 @@
       }
     },
     "type": "error"
+  },
+  {
+    "code": "request.path-or-method.unknown",
+    "message": "Path or method not defined in spec file: POST /integer/1",
+    "mockDetails": {
+      "interactionDescription": "should not raise subsequent errors when request.path-or-method.unknown",
+      "interactionState": "[none]",
+      "location": "[root].interactions[10].request.path",
+      "value": "/integer/1"
+    },
+    "specDetails": {
+      "location": "[root].paths",
+      "pathMethod": null,
+      "pathName": null,
+      "value": {
+        "/integer/{id}": {
+          "get": {
+            "parameters": [
+              {
+                "in": "path",
+                "name": "id",
+                "required": true,
+                "schema": {
+                  "type": "integer"
+                }
+              }
+            ],
+            "responses": {
+              "200": {
+                "description": "description"
+              }
+            }
+          }
+        },
+        "/boolean/{id}": {
+          "get": {
+            "parameters": [
+              {
+                "in": "path",
+                "name": "id",
+                "required": true,
+                "schema": {
+                  "type": "boolean"
+                }
+              }
+            ],
+            "responses": {
+              "200": {
+                "description": "description"
+              }
+            }
+          }
+        },
+        "/exploded-object/{id}": {
+          "get": {
+            "parameters": [
+              {
+                "in": "path",
+                "name": "id",
+                "required": true,
+                "schema": {
+                  "type": "object",
+                  "properties": {
+                    "string": {
+                      "type": "string"
+                    },
+                    "number": {
+                      "type": "number"
+                    }
+                  }
+                },
+                "style": "simple",
+                "explode": true
+              }
+            ],
+            "responses": {
+              "200": {
+                "description": "description"
+              }
+            }
+          }
+        }
+      }
+    },
+    "type": "error"
   }
 ]

src/__tests__/fixtures/request-parameters-path/pact.json

@@ -101,6 +101,22 @@
       "response": {
         "status": 200
       }
+    },
+    {
+      "description": "should not raise subsequent errors when request.path-or-method.unknown",
+      "request": {
+        "method": "POST",
+        "path": "/integer/1",
+        "headers": {
+          "additional": "ignored error"
+        }
+      },
+      "response": {
+        "status": 200,
+        "headers": {
+          "accept": "should not be here!"
+        }
+      }
     }
   ]
 }

src/__tests__/fixtures/request-parameters-path/results.json

@@ -182,5 +182,21 @@
       "pathName": null
     },
     "type": "error"
+  },
+  {
+    "code": "request.path-or-method.unknown",
+    "message": "Path or method not defined in spec file: POST /integer/1",
+    "mockDetails": {
+      "interactionDescription": "should not raise subsequent errors when request.path-or-method.unknown",
+      "interactionState": "[none]",
+      "location": "[root].interactions[10].request.path",
+      "value": "/integer/1"
+    },
+    "specDetails": {
+      "location": "[root].paths",
+      "pathMethod": null,
+      "pathName": null
+    },
+    "type": "error"
   }
 ]

src/compare/index.ts

@@ -100,7 +100,15 @@ export class Comparator {
         continue;
       }
 
-      yield* compareReqPath(this.#ajvCoerce, route, interaction, index);
+      const results = Array.from(
+        compareReqPath(this.#ajvCoerce, route, interaction, index),
+      );
+
+      if (results.length) {
+        yield* results;
+        continue;
+      }
+
       yield* compareReqHeader(this.#ajvCoerce, route, interaction, index);
       yield* compareReqQuery(this.#ajvCoerce, route, interaction, index);
       yield* compareReqBody(this.#ajvNocoerce, route, interaction, index);