fixing #345

Author: erosb

core/src/main/java/org/everit/json/schema/ToStringVisitor.java

@@ -144,8 +144,9 @@ void visitBooleanSchema(BooleanSchema schema) {
 
     @Override void visitNotSchema(NotSchema notSchema) {
         printInJsonObject(() -> {
+            visitSchema(notSchema);
             writer.key("not");
-            super.visitNotSchema(notSchema);
+            notSchema.getMustNotMatch().accept(this);
         });
     }
 

core/src/test/java/org/everit/json/schema/NotSchemaTest.java

@@ -15,16 +15,22 @@
  */
 package org.everit.json.schema;
 
+import static org.everit.json.schema.JSONMatcher.sameJsonAs;
 import static org.everit.json.schema.TestSupport.buildWithLocation;
+import static org.hamcrest.MatcherAssert.assertThat;
 import static org.junit.Assert.assertEquals;
 
+import org.everit.json.schema.loader.SchemaLoader;
+import org.json.JSONObject;
 import org.junit.Test;
 
 import nl.jqno.equalsverifier.EqualsVerifier;
 import nl.jqno.equalsverifier.Warning;
 
 public class NotSchemaTest {
 
+    private static final ResourceLoader LOADER = ResourceLoader.DEFAULT;
+
     @Test
     public void failure() {
         NotSchema subject = buildWithLocation(NotSchema.builder().mustNotMatch(BooleanSchema.INSTANCE));
@@ -57,4 +63,13 @@ public void toStringTest() {
         assertEquals("{\"not\":{\"type\":\"boolean\"}}", actual);
     }
 
+    @Test
+    public void issue345() {
+        JSONObject rawSchema = LOADER.readObj("issue345.json");
+        Schema notSchema = SchemaLoader.builder()
+            .schemaJson(rawSchema)
+            .build().load().build();
+        assertThat(new JSONObject(notSchema.toString()), sameJsonAs(rawSchema));
+    }
+
 }

core/src/test/resources/org/everit/jsonvalidator/issue345.json

@@ -0,0 +1,11 @@
+{
+  "$schema":"http://json-schema.org/draft-04/schema#",
+  "not":{
+    "type":"object",
+    "properties":{
+      "name":{
+        "type":"string"
+      }
+    }
+  }
+}