DATAREST-994 - Fixed two argument constructor of RepositoryRestHandlerMapping.

Repositories in RepositoryCorsConfigurationAccessor may be null now. findCorsConfiguration returns null when no repositories are provided. Order of parameters changed to make the optional parameter last.

Author: schauder

spring-data-rest-webmvc/src/test/java/org/springframework/data/rest/webmvc/RepositoryCorsConfigurationAccessorUnitTests.java

@@ -53,7 +53,7 @@ public class RepositoryCorsConfigurationAccessorUnitTests {
 
 @Before
 public void before() throws Exception {
-accessor = new RepositoryCorsConfigurationAccessor(mappings, repositories, NoOpStringValueResolver.INSTANCE);
+accessor = new RepositoryCorsConfigurationAccessor(mappings, NoOpStringValueResolver.INSTANCE, repositories);
 }
 
 @Test // DATAREST-573
@@ -86,11 +86,10 @@ public void createConfigurationShouldConstructFullCorsConfiguration() {
 assertThat(configuration.getMaxAge(), is(1234L));
 }
 
-
 @Test // DATAREST-994
 public void returnsNullCorsConfigurationWithNullRepositories() {
 
-accessor = new RepositoryCorsConfigurationAccessor(mappings, null, NoOpStringValueResolver.INSTANCE);
+accessor = new RepositoryCorsConfigurationAccessor(mappings, NoOpStringValueResolver.INSTANCE, null);
 
 ResourceMetadata resourceMetadata = mock(ResourceMetadata.class);
 when(resourceMetadata.getPath()).thenReturn(new Path("/people"));