diff --git a/src/main/java/com/fasterxml/jackson/module/jsonSchema/types/ObjectSchema.java b/src/main/java/com/fasterxml/jackson/module/jsonSchema/types/ObjectSchema.java index 446cea18..0f06c889 100644 --- a/src/main/java/com/fasterxml/jackson/module/jsonSchema/types/ObjectSchema.java +++ b/src/main/java/com/fasterxml/jackson/module/jsonSchema/types/ObjectSchema.java @@ -1,7 +1,6 @@ package com.fasterxml.jackson.module.jsonSchema.types; import java.util.ArrayList; -import java.util.HashMap; import java.util.List; import java.util.Map; @@ -12,6 +11,7 @@ import com.fasterxml.jackson.databind.annotation.JsonDeserialize; import com.fasterxml.jackson.databind.jsonFormatVisitors.JsonFormatTypes; import com.fasterxml.jackson.module.jsonSchema.JsonSchema; +import java.util.LinkedHashMap; /** * This type represents a {@link JsonSchema} as an object type @@ -68,8 +68,8 @@ public class ObjectSchema extends ContainerTypeSchema public ObjectSchema() { dependencies = new ArrayList(); - patternProperties = new HashMap(); - properties = new HashMap(); + patternProperties = new LinkedHashMap(); + properties = new LinkedHashMap(); } public boolean addSchemaDependency(String depender, JsonSchema parentMustMatch) { diff --git a/src/test/java/com/fasterxml/jackson/module/jsonSchema/TestPropertyOrderInSchema.java b/src/test/java/com/fasterxml/jackson/module/jsonSchema/TestPropertyOrderInSchema.java new file mode 100644 index 00000000..3b09f55e --- /dev/null +++ b/src/test/java/com/fasterxml/jackson/module/jsonSchema/TestPropertyOrderInSchema.java @@ -0,0 +1,55 @@ +package com.fasterxml.jackson.module.jsonSchema; + +import com.fasterxml.jackson.annotation.JsonPropertyOrder; + +import com.fasterxml.jackson.databind.ObjectMapper; + +public class TestPropertyOrderInSchema extends SchemaTestBase { + + @JsonPropertyOrder({"a", "b", "c"}) + static class Bean { + + private int b; + private String c; + private String a; + + public int getB() { + return b; + } + + public void setB(int b) { + this.b = b; + } + + public String getC() { + return c; + } + + public void setC(String c) { + this.c = c; + } + + public String getA() { + return a; + } + + public void setA(String a) { + this.a = a; + } + } + + /* + /********************************************************** + /* Unit tests + /********************************************************** + */ + final ObjectMapper MAPPER = objectMapper(); + + public void testCorrectOrder() throws Exception { + JsonSchemaGenerator generator = new JsonSchemaGenerator(MAPPER); + JsonSchema jsonSchema = generator.generateSchema(Bean.class); + System.out.println(jsonSchema.asObjectSchema().getProperties().keySet()); + assertEquals(jsonSchema.asObjectSchema().getProperties().keySet().toString(), "[a, b, c]"); + } + +}