Fix equals/hashcode

Author: gnodet

impl/maven-impl/src/main/java/org/apache/maven/internal/impl/model/DefaultProfileActivationContext.java

@@ -32,7 +32,9 @@
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
+import java.util.Set;
 import java.util.concurrent.atomic.AtomicBoolean;
+import java.util.stream.Collectors;
 
 import org.apache.maven.api.model.Model;
 import org.apache.maven.api.services.Interpolator;
@@ -58,7 +60,7 @@ enum ModelInfo {
         RootDirectory
     }
 
-    class Record {
+    static class Record {
         private final Map<String, Boolean> usedActiveProfiles = new HashMap<>();
         private final Map<String, Boolean> usedInactiveProfiles = new HashMap<>();
         private final Map<String, String> usedSystemProperties = new HashMap<>();
@@ -67,6 +69,36 @@ class Record {
         private final Map<ModelInfo, String> usedModelInfos = new HashMap<>();
         private final Map<ExistRequest, Boolean> usedExists = new HashMap<>();
 
+        @Override
+        public boolean equals(Object o) {
+            if (o instanceof Record record) {
+                return Objects.equals(values(usedActiveProfiles), values(record.usedActiveProfiles))
+                        && Objects.equals(values(usedInactiveProfiles), values(record.usedInactiveProfiles))
+                        && Objects.equals(usedSystemProperties, record.usedSystemProperties)
+                        && Objects.equals(usedUserProperties, record.usedUserProperties)
+                        && Objects.equals(usedModelProperties, record.usedModelProperties)
+                        && Objects.equals(usedModelInfos, record.usedModelInfos)
+                        && Objects.equals(usedExists, record.usedExists);
+            }
+            return false;
+        }
+
+        private static Set<String> values(Map<String, Boolean> map) {
+            return map.entrySet().stream().filter(e -> e.getValue()).map(e -> e.getKey()).collect(Collectors.toSet());
+        }
+
+        @Override
+        public int hashCode() {
+            return Objects.hash(
+                    usedActiveProfiles,
+                    usedInactiveProfiles,
+                    usedSystemProperties,
+                    usedUserProperties,
+                    usedModelProperties,
+                    usedModelInfos,
+                    usedExists);
+        }
+
         boolean matches(DefaultProfileActivationContext context) {
             for (Map.Entry<String, Boolean> e : usedActiveProfiles.entrySet()) {
                 if (!Objects.equals(e.getValue(), context.activeProfileIds.contains(e.getKey()))) {
@@ -107,7 +139,8 @@ boolean matches(DefaultProfileActivationContext context) {
             }
             for (Map.Entry<ExistRequest, Boolean> e : usedExists.entrySet()) {
                 if (!Objects.equals(
-                        e.getValue(), doExists(e.getKey().path(), e.getKey().enableGlob()))) {
+                        e.getValue(),
+                        context.doExists(e.getKey().path(), e.getKey().enableGlob()))) {
                     return false;
                 }
             }