Make locations unique in MergedContextConfiguration

Prior to this commit the same location could be added twice and this would cause the Context loader to load it multiple times.

See spring-projectsgh-32534

Author: andrea-mauro

Diff for: spring-test/src/main/java/org/springframework/test/context/MergedContextConfiguration.java

@@ -288,10 +288,10 @@ public MergedContextConfiguration(Class<?> testClass, @Nullable String[] locatio
 			@Nullable MergedContextConfiguration parent) {
 
 		this.testClass = testClass;
-		this.locations = processStrings(locations);
+		this.locations = processUniqueStrings(locations);
 		this.classes = processClasses(classes);
 		this.contextInitializerClasses = processContextInitializerClasses(contextInitializerClasses);
-		this.activeProfiles = processActiveProfiles(activeProfiles);
+		this.activeProfiles = processUniqueStrings(activeProfiles);
 		this.propertySourceDescriptors = Collections.unmodifiableList(propertySourceDescriptors);
 		this.propertySourceLocations = this.propertySourceDescriptors.stream()
 				.map(PropertySourceDescriptor::locations)
@@ -597,13 +597,13 @@ private static Set<ContextCustomizer> processContextCustomizers(
 				Collections.unmodifiableSet(contextCustomizers) : EMPTY_CONTEXT_CUSTOMIZERS);
 	}
 
-	private static String[] processActiveProfiles(@Nullable String[] activeProfiles) {
-		if (activeProfiles == null) {
+	private static String[] processUniqueStrings(@Nullable String[] array) {
+		if (array == null) {
 			return EMPTY_STRING_ARRAY;
 		}
 
 		// Active profiles must be unique
-		Set<String> profilesSet = new LinkedHashSet<>(Arrays.asList(activeProfiles));
+		Set<String> profilesSet = new LinkedHashSet<>(Arrays.asList(array));
 		return StringUtils.toStringArray(profilesSet);
 	}
 

Diff for: spring-test/src/test/java/org/springframework/test/context/MergedContextConfigurationTests.java

@@ -105,6 +105,18 @@ void hashCodeWithDifferentLocations() {
 		assertThat(mergedConfig2.hashCode()).isNotEqualTo(mergedConfig1.hashCode());
 	}
 
+	@Test
+	void hashCodeWithSameDuplicateLocations() {
+		String[] locations1 = new String[] { "foo", "bar}" };
+		String[] locations2 = new String[] { "foo", "bar}", "foo", "bar}" };
+		MergedContextConfiguration mergedConfig1 = new MergedContextConfiguration(getClass(), locations1,
+																				  EMPTY_CLASS_ARRAY, EMPTY_STRING_ARRAY, loader);
+		MergedContextConfiguration mergedConfig2 = new MergedContextConfiguration(getClass(), locations2,
+																				  EMPTY_CLASS_ARRAY, EMPTY_STRING_ARRAY, loader);
+		assertThat(mergedConfig2.hashCode()).isEqualTo(mergedConfig1.hashCode());
+	}
+
+
 	@Test
 	void hashCodeWithSameConfigClasses() {
 		Class<?>[] classes = new Class<?>[] { String.class, Integer.class };