invocation matching not taking verb into account correctly

Kong · Oct 26, 2023 · 85861ff · 85861ff
1 parent f14c353
commit 85861ff
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 1 deletion.
diff --git a/unirest-mocks/src/main/java/kong/unirest/core/Routes.java b/unirest-mocks/src/main/java/kong/unirest/core/Routes.java
@@ -54,7 +54,7 @@ Expectation newExpectation() {
     boolean matches(HttpRequest request) {
         Path p = new Path(request.getUrl());
         return this.method.equals(request.getHttpMethod())
-                && this.path == null || this.path.equalsIgnoreCase(p.baseUrl());
+                && (this.path == null || this.path.equalsIgnoreCase(p.baseUrl()));
     }
 
     RawResponse exchange(HttpRequest request, Config config) {

diff --git a/unirest-mocks/src/test/java/kong/tests/AssertTest.java b/unirest-mocks/src/test/java/kong/tests/AssertTest.java
@@ -250,6 +250,12 @@ void returnAMockResponseObjectWithoutStuff() {
         assertEquals(List.of(), response.getHeaders().get("cool"));
     }
 
+    @Test
+    void verbsAreImportant() {
+        client.expect(GET, path).thenReturn("hi");
+        assertNotEquals("hi", Unirest.post(path).asString().getBody());
+    }
+
     private static class BodyBuddy implements Supplier<String>{
         String body;
         @Override