Merge pull request #9 from orangain/fix-object-match

Fix match of empty object literal

src/main/java/io/github/orangain/jsonmatch/pattern/ObjectLiteralPatternNode.java

@@ -23,6 +23,10 @@ public ObjectLiteralPatternNode(@NotNull String expected, @NotNull Map<String, J
     @NotNull
     @Override
     public Optional<JsonMatchErrorDetail> matches(@NotNull JsonPath path, @NotNull JsonNode actualNode) {
+        if (!actualNode.isObject()) {
+            String reason = "not a json object";
+            return Optional.of(error(path, actualNode, reason));
+        }
         Set<String> actualFieldNames = new HashSet<>();
         actualNode.fieldNames().forEachRemaining(actualFieldNames::add);
 

src/test/kotlin/io/github/orangain/jsonmatch/EmptyObjectLiteralTest.kt

@@ -0,0 +1,50 @@
+package io.github.orangain.jsonmatch
+
+import org.assertj.core.api.Assertions
+import org.junit.Test
+
+class EmptyObjectLiteralTest {
+    val patternJson = "{}"
+
+    @Test
+    fun matchEmptyObject() {
+        // language=JSON
+        JsonStringAssert.assertThat("{}").jsonMatches(patternJson)
+    }
+
+    @Test
+    fun doesNotMatchArray() {
+        Assertions.assertThatThrownBy {
+            // language=JSON
+            JsonStringAssert.assertThat("[]").jsonMatches(patternJson)
+        }.isInstanceOf(AssertionError::class.java)
+            .hasMessageContaining("""path: $, actual: [], expected: {}, reason: not a json object""")
+    }
+
+    @Test
+    fun doesNotMatchNumber() {
+        Assertions.assertThatThrownBy {
+            // language=JSON
+            JsonStringAssert.assertThat("0").jsonMatches(patternJson)
+        }.isInstanceOf(AssertionError::class.java)
+            .hasMessageContaining("""path: $, actual: 0, expected: {}, reason: not a json object""")
+    }
+
+    @Test
+    fun doesNotMatchString() {
+        Assertions.assertThatThrownBy {
+            // language=JSON
+            JsonStringAssert.assertThat("\"foo\"").jsonMatches(patternJson)
+        }.isInstanceOf(AssertionError::class.java)
+            .hasMessageContaining("""path: $, actual: "foo", expected: {}, reason: not a json object""")
+    }
+
+    @Test
+    fun doesNotMatchNull() {
+        Assertions.assertThatThrownBy {
+            // language=JSON
+            JsonStringAssert.assertThat("null").jsonMatches(patternJson)
+        }.isInstanceOf(AssertionError::class.java)
+            .hasMessageContaining("""path: $, actual: null, expected: {}, reason: not a json object""")
+    }
+}