DATAREST-994 - Fixed two argument constructor of RepositoryRestHandlerMapping.

Repositories in RepositoryCorsConfigurationAccessor may be null now. findCorsConfiguration returns null when no repositories are provided.

Author: schauder

spring-data-rest-webmvc/src/main/java/org/springframework/data/rest/webmvc/RepositoryRestHandlerMapping.java

@@ -1,5 +1,5 @@
 /*
- * Copyright 2012-2016 the original author or authors.
+ * Copyright 2012-2017 the original author or authors.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -266,14 +266,14 @@ public String resolveStringValue(String value) {
 static class RepositoryCorsConfigurationAccessor {
 
 private final @NonNull ResourceMappings mappings;
-private final @NonNull Repositories repositories;
+private final Repositories repositories;
 private final @NonNull StringValueResolver embeddedValueResolver;
 
 CorsConfiguration findCorsConfiguration(String lookupPath) {
 
 ResourceMetadata resource = getResourceMetadata(getRepositoryBasePath(lookupPath));
 
-return resource != null ? createConfiguration(
+return resource != null && repositories != null ? createConfiguration(
 repositories.getRepositoryInformationFor(resource.getDomainType()).getRepositoryInterface()) : null;
 }
 

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

@@ -15,16 +15,20 @@
  */
 package org.springframework.data.rest.webmvc;
 
+import static java.util.Collections.*;
 import static org.hamcrest.MatcherAssert.*;
 import static org.hamcrest.Matchers.*;
+import static org.mockito.Mockito.*;
 
 import org.junit.Before;
 import org.junit.Test;
 import org.junit.runner.RunWith;
 import org.mockito.Mock;
 import org.mockito.runners.MockitoJUnitRunner;
 import org.springframework.data.repository.support.Repositories;
+import org.springframework.data.rest.core.Path;
 import org.springframework.data.rest.core.mapping.ResourceMappings;
+import org.springframework.data.rest.core.mapping.ResourceMetadata;
 import org.springframework.data.rest.webmvc.RepositoryRestHandlerMapping.NoOpStringValueResolver;
 import org.springframework.data.rest.webmvc.RepositoryRestHandlerMapping.RepositoryCorsConfigurationAccessor;
 import org.springframework.web.bind.annotation.CrossOrigin;
@@ -82,6 +86,22 @@ public void createConfigurationShouldConstructFullCorsConfiguration() {
 assertThat(configuration.getMaxAge(), is(1234L));
 }
 
+
+@Test // DATAREST-994
+public void returnsNullCorsConfigurationWithNullRepositories() {
+
+accessor = new RepositoryCorsConfigurationAccessor(mappings, null, NoOpStringValueResolver.INSTANCE);
+
+ResourceMetadata resourceMetadata = mock(ResourceMetadata.class);
+when(resourceMetadata.getPath()).thenReturn(new Path("/people"));
+when(resourceMetadata.isExported()).thenReturn(true);
+
+when(mappings.exportsTopLevelResourceFor("/people")).thenReturn(true);
+when(mappings.iterator()).thenReturn(singletonList(resourceMetadata).iterator());
+
+assertThat(accessor.findCorsConfiguration("/people"), is(nullValue()));
+}
+
 interface PlainRepository {}
 
 @CrossOrigin

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

@@ -42,7 +42,7 @@
 
 /**
  * Unit tests for {@link RepositoryRestHandlerMapping}.
- * 
+ *
  * @author Oliver Gierke
  * @author Greg Turnquist
  */
@@ -212,4 +212,9 @@ public void rejectsUnexpandedUriTemplateWithNotFound() throws Exception {
 
 assertThat(handlerMapping.getHandler(mockRequest), is(nullValue()));
 }
+
+@Test // DATAREST-994
+public void twoArgumentConstructorDoesNotThrowException() {
+new RepositoryRestHandlerMapping(mappings, configuration);
+}
 }