Use LinkedHashSet instead of HashSet to kept item in the same order

Author: quen2404

src/main/java/com/qdesrame/openapi/diff/compare/ApiResponseDiff.java

@@ -6,7 +6,7 @@
 import io.swagger.v3.oas.models.responses.ApiResponse;
 import io.swagger.v3.oas.models.responses.ApiResponses;
 
-import java.util.HashMap;
+import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
@@ -29,7 +29,7 @@ public Optional<ChangedApiResponse> diff(ApiResponses left, ApiResponses right,
         changedApiResponse.setAddResponses(responseMapKeyDiff.getIncreased());
         changedApiResponse.setMissingResponses(responseMapKeyDiff.getMissing());
         List<String> sharedResponseCodes = responseMapKeyDiff.getSharedKey();
-        Map<String, ChangedResponse> resps = new HashMap<>();
+        Map<String, ChangedResponse> resps = new LinkedHashMap<>();
         for (String responseCode : sharedResponseCodes) {
             openApiDiff.getResponseDiff().diff(left.get(responseCode), right.get(responseCode), context)
                     .ifPresent(changedResponse -> resps.put(responseCode, changedResponse));

src/main/java/com/qdesrame/openapi/diff/compare/ContentDiff.java

@@ -31,7 +31,7 @@ public Optional<ChangedContent> diff(Content left, Content right, DiffContext co
         changedContent.setIncreased(mediaTypeDiff.getIncreased());
         changedContent.setMissing(mediaTypeDiff.getMissing());
         List<String> sharedMediaTypes = mediaTypeDiff.getSharedKey();
-        Map<String, ChangedMediaType> changedMediaTypes = new HashMap<>();
+        Map<String, ChangedMediaType> changedMediaTypes = new LinkedHashMap<>();
         for (String mediaTypeKey : sharedMediaTypes) {
             MediaType oldMediaType = left.get(mediaTypeKey);
             MediaType newMediaType = right.get(mediaTypeKey);

src/main/java/com/qdesrame/openapi/diff/compare/ExtensionsDiff.java

@@ -22,9 +22,9 @@ public ExtensionsDiff(OpenApiDiff openApiDiff) {
     }
 
     public Optional<ChangedExtensions> diff(Map<String, Object> left, Map<String, Object> right, DiffContext context) {
-        if (null == left) left = new HashMap<>();
-        if (null == right) right = new HashMap<>();
-        ChangedExtensions changedExtensions = new ChangedExtensions(left, new HashMap<>(right), context);
+        if (null == left) left = new LinkedHashMap<>();
+        if (null == right) right = new LinkedHashMap<>();
+        ChangedExtensions changedExtensions = new ChangedExtensions(left, new LinkedHashMap<>(right), context);
         changedExtensions.getIncreased().putAll(right);
         for (String key : left.keySet()) {
             if (changedExtensions.getIncreased().containsKey(key)) {

src/main/java/com/qdesrame/openapi/diff/compare/HeadersDiff.java

@@ -5,7 +5,7 @@
 import com.qdesrame.openapi.diff.model.DiffContext;
 import io.swagger.v3.oas.models.headers.Header;
 
-import java.util.HashMap;
+import java.util.LinkedHashMap;
 import java.util.List;
 import java.util.Map;
 import java.util.Optional;
@@ -29,7 +29,7 @@ public Optional<ChangedHeaders> diff(Map<String, Header> left, Map<String, Heade
         changedHeaders.setMissing(headerMapDiff.getMissing());
         List<String> sharedHeaderKeys = headerMapDiff.getSharedKey();
 
-        Map<String, ChangedHeader> changed = new HashMap<>();
+        Map<String, ChangedHeader> changed = new LinkedHashMap<>();
         for (String headerKey : sharedHeaderKeys) {
             Header oldHeader = left.get(headerKey);
             Header newHeader = right.get(headerKey);

src/main/java/com/qdesrame/openapi/diff/compare/MapKeyDiff.java

@@ -1,6 +1,9 @@
 package com.qdesrame.openapi.diff.compare;
 
-import java.util.*;
+import java.util.ArrayList;
+import java.util.LinkedHashMap;
+import java.util.List;
+import java.util.Map;
 import java.util.Map.Entry;
 
 /**
@@ -16,8 +19,8 @@ public class MapKeyDiff<K, V> {
 
     private MapKeyDiff() {
         this.sharedKey = new ArrayList<>();
-        this.increased = new HashMap<>();
-        this.missing = new HashMap<>();
+        this.increased = new LinkedHashMap<>();
+        this.missing = new LinkedHashMap<>();
     }
 
     public static <K, V> MapKeyDiff<K, V> diff(Map<K, V> mapLeft,

src/main/java/com/qdesrame/openapi/diff/compare/PathsDiff.java

@@ -45,7 +45,7 @@ public Optional<ChangedPaths> diff(final Map<String, PathItem> left, final Map<S
                     throw new IllegalArgumentException("Two path items have the same signature: " + template);
                 }
                 PathItem rightPath = changedPaths.getIncreased().remove(result.get());
-                Map<String, String> params = new HashMap<>();
+                Map<String, String> params = new LinkedHashMap<>();
                 if (!url.equals(result.get())) {
                     List<String> oldParams = extractParameters(url);
                     List<String> newParams = extractParameters(result.get());

src/main/java/com/qdesrame/openapi/diff/compare/SchemaDiff.java

@@ -21,7 +21,7 @@ public class SchemaDiff extends ReferenceDiffCache<Schema, ChangedSchema> {
     private OpenApiDiff openApiDiff;
     private static RefPointer<Schema> refPointer = new RefPointer<>(RefType.SCHEMAS);
 
-    private static Map<Class<? extends Schema>, Class<? extends SchemaDiffResult>> schemaDiffResultClassMap = new HashMap<>();
+    private static Map<Class<? extends Schema>, Class<? extends SchemaDiffResult>> schemaDiffResultClassMap = new LinkedHashMap<>();
 
     static {
         schemaDiffResultClassMap.put(Schema.class, SchemaDiffResult.class);

src/main/java/com/qdesrame/openapi/diff/compare/schemadiffresult/ComposedSchemaDiffResult.java

@@ -13,8 +13,8 @@
 import io.swagger.v3.oas.models.media.Schema;
 import org.apache.commons.collections4.CollectionUtils;
 
-import java.util.HashMap;
 import java.util.HashSet;
+import java.util.LinkedHashMap;
 import java.util.Map;
 import java.util.Optional;
 import java.util.stream.Collectors;
@@ -59,7 +59,7 @@ public Optional<ChangedSchema> diff(HashSet<String> refSet, Components leftCompo
                 changedOneOfSchema.setIncreasedMapping(mappingDiff.getIncreased());
                 changedOneOfSchema.setMissingMapping(mappingDiff.getMissing());
 
-                Map<String, ChangedSchema> changedMapping = new HashMap<>();
+                Map<String, ChangedSchema> changedMapping = new LinkedHashMap<>();
                 changedOneOfSchema.setChangedMapping(changedMapping);
 
                 for (String key : mappingDiff.getSharedKey()) {
@@ -79,13 +79,13 @@ public Optional<ChangedSchema> diff(HashSet<String> refSet, Components leftCompo
     }
 
     private Map<String, Schema> getSchema(Components components, Map<String, String> mapping) {
-        Map<String, Schema> result = new HashMap<>();
+        Map<String, Schema> result = new LinkedHashMap<>();
         mapping.forEach((key, value) -> result.put(key, refPointer.resolveRef(components, new Schema(), value)));
         return result;
     }
 
     private Map<String, String> getMapping(ComposedSchema composedSchema) {
-        Map<String, String> reverseMapping = new HashMap<>();
+        Map<String, String> reverseMapping = new LinkedHashMap<>();
         for (Schema schema : composedSchema.getOneOf()) {
             String ref = schema.get$ref();
             if (ref == null) {

src/main/java/com/qdesrame/openapi/diff/model/ChangedApiResponse.java

@@ -5,7 +5,7 @@
 import lombok.Getter;
 import lombok.Setter;
 
-import java.util.HashMap;
+import java.util.LinkedHashMap;
 import java.util.Map;
 
 /**
@@ -25,9 +25,9 @@ public ChangedApiResponse(ApiResponses oldApiResponses, ApiResponses newApiRespo
         this.oldApiResponses = oldApiResponses;
         this.newApiResponses = newApiResponses;
         this.context = context;
-        this.missingResponses = new HashMap<>();
-        this.addResponses = new HashMap<>();
-        this.changedResponses = new HashMap<>();
+        this.missingResponses = new LinkedHashMap<>();
+        this.addResponses = new LinkedHashMap<>();
+        this.changedResponses = new LinkedHashMap<>();
     }
 
     @Override

src/main/java/com/qdesrame/openapi/diff/model/ChangedContent.java

@@ -5,7 +5,7 @@
 import lombok.Getter;
 import lombok.Setter;
 
-import java.util.HashMap;
+import java.util.LinkedHashMap;
 import java.util.Map;
 
 /**
@@ -26,9 +26,9 @@ public ChangedContent(Content oldContent, Content newContent, DiffContext contex
         this.oldContent = oldContent;
         this.newContent = newContent;
         this.context = context;
-        this.increased = new HashMap<>();
-        this.missing = new HashMap<>();
-        this.changed = new HashMap<>();
+        this.increased = new LinkedHashMap<>();
+        this.missing = new LinkedHashMap<>();
+        this.changed = new LinkedHashMap<>();
     }
 
     @Override

src/main/java/com/qdesrame/openapi/diff/model/ChangedSchema.java

@@ -9,7 +9,7 @@
 import lombok.Setter;
 import org.apache.commons.collections4.CollectionUtils;
 
-import java.util.HashMap;
+import java.util.LinkedHashMap;
 import java.util.Map;
 
 /**
@@ -43,9 +43,9 @@ public class ChangedSchema implements Changed {
     protected ChangedExtensions changedExtensions;
 
     public ChangedSchema() {
-        increasedProperties = new HashMap<>();
-        missingProperties = new HashMap<>();
-        changedProperties = new HashMap<>();
+        increasedProperties = new LinkedHashMap<>();
+        missingProperties = new LinkedHashMap<>();
+        changedProperties = new LinkedHashMap<>();
     }
 
     @Override