fix brittle test that relies on order

hgschmie · hgschmie · commit b850d6e09e0b · 2023-08-22T11:35:58.000-07:00
diff --git a/plexus-java/src/main/java/org/codehaus/plexus/languages/java/jpms/LocationManager.java b/plexus-java/src/main/java/org/codehaus/plexus/languages/java/jpms/LocationManager.java
@@ -250,8 +250,8 @@ public String extract(Path path) throws IOException {
                     request.isIncludeStatic());
         }
 
-        Map<T, JavaModuleDescriptor> definedModules = new HashMap<>();
-        Map<T, JavaModuleDescriptor> automaticModules = new HashMap<>();
+        Map<T, JavaModuleDescriptor> definedModules = new LinkedHashMap<>();
+        Map<T, JavaModuleDescriptor> automaticModules = new LinkedHashMap<>();
 
         pathElements.forEach((k, v) -> {
             if (v != null && !v.isAutomatic()) {

Original file line number	Diff line number	Diff line change
`@@ -250,8 +250,8 @@ public String extract(Path path) throws IOException {`
`250`	`250`	`request.isIncludeStatic());`
`251`	`251`	`}`
`252`	`252`
`253`		`- Map<T, JavaModuleDescriptor> definedModules = new HashMap<>();`
`254`		`- Map<T, JavaModuleDescriptor> automaticModules = new HashMap<>();`
	`253`	`+ Map<T, JavaModuleDescriptor> definedModules = new LinkedHashMap<>();`
	`254`	`+ Map<T, JavaModuleDescriptor> automaticModules = new LinkedHashMap<>();`
`255`	`255`
`256`	`256`	`pathElements.forEach((k, v) -> {`
`257`	`257`	`if (v != null && !v.isAutomatic()) {`