Merge pull request #216 from cyqw/master

change type of possibleValues in EnumSchema from Set to List

Author: erosb

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

@@ -1,12 +1,13 @@
 package org.everit.json.schema;
 
-import static java.util.stream.Collectors.toSet;
+import static java.util.stream.Collectors.toList;
 
 import java.util.Collections;
-import java.util.HashSet;
+import java.util.ArrayList;
 import java.util.Objects;
+import java.util.List;
 import java.util.Set;
-
+import java.util.stream.Collectors;
 import org.everit.json.schema.internal.JSONPrinter;
 import org.json.JSONArray;
 import org.json.JSONObject;
@@ -28,16 +29,16 @@ static Object toJavaValue(Object orig) {
         }
     }
 
-    static Set<Object> toJavaValues(Set<Object> orgJsons) {
-        return orgJsons.stream().map(EnumSchema::toJavaValue).collect(toSet());
+    static List<Object> toJavaValues(List<Object> orgJsons) {
+        return orgJsons.stream().map(EnumSchema::toJavaValue).collect(toList());
     }
 
     /**
      * Builder class for {@link EnumSchema}.
      */
     public static class Builder extends Schema.Builder<EnumSchema> {
 
-        private Set<Object> possibleValues = new HashSet<>();
+        private List<Object> possibleValues = new ArrayList<>();
 
         @Override
         public EnumSchema build() {
@@ -49,24 +50,33 @@ public Builder possibleValue(Object possibleValue) {
             return this;
         }
 
-        public Builder possibleValues(Set<Object> possibleValues) {
+        public Builder possibleValues(List<Object> possibleValues) {
             this.possibleValues = possibleValues;
             return this;
         }
+
+        public Builder possibleValues(Set<Object> possibleValues) {
+            this.possibleValues = possibleValues.stream().collect(toList());
+            return this;
+        }
     }
 
     public static Builder builder() {
         return new Builder();
     }
 
-    private final Set<Object> possibleValues;
+    private final List<Object> possibleValues;
 
     public EnumSchema(Builder builder) {
         super(builder);
-        possibleValues = Collections.unmodifiableSet(toJavaValues(builder.possibleValues));
+        possibleValues = Collections.unmodifiableList(toJavaValues(builder.possibleValues));
     }
 
     public Set<Object> getPossibleValues() {
+        return possibleValues.stream().collect(Collectors.toSet());
+    }
+
+    public List<Object> getPossibleValuesAsList() {
         return possibleValues;
     }
 

core/src/main/java/org/everit/json/schema/loader/SchemaExtractor.java

@@ -143,7 +143,7 @@ class EnumSchemaExtractor extends AbstractSchemaExtractor {
             return emptyList();
         }
         EnumSchema.Builder builder = EnumSchema.builder();
-        Set<Object> possibleValues = new HashSet<>();
+        List<Object> possibleValues = new ArrayList<>();
         require("enum").requireArray().forEach((i, item) -> possibleValues.add(item.unwrap()));
         builder.possibleValues(possibleValues);
         return singletonList(builder);

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

@@ -20,10 +20,9 @@
 
 import java.io.StringWriter;
 import java.util.HashMap;
-import java.util.HashSet;
+import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
-import java.util.Set;
 import java.util.stream.Collectors;
 import java.util.stream.IntStream;
 
@@ -38,11 +37,11 @@
 
 public class EnumSchemaTest {
 
-    private Set<Object> possibleValues;
+    private List<Object> possibleValues;
 
     @Before
     public void before() {
-        possibleValues = new HashSet<>();
+        possibleValues = new ArrayList<>();
         possibleValues.add(true);
         possibleValues.add("foo");
     }
@@ -86,10 +85,10 @@ public void objectInArrayMatches() {
         subject.validate(list);
     }
 
-    private Set<Object> asSet(final JSONArray array) {
-        return new HashSet<>(IntStream.range(0, array.length())
+    private List<Object> asSet(final JSONArray array) {
+        return new ArrayList<>(IntStream.range(0, array.length())
                 .mapToObj(i -> array.get(i))
-                .collect(Collectors.toSet()));
+                .collect(Collectors.toList()));
     }
 
     @Test