#1060 - When EnableHypermediaSupport has an empty list, enable all.

Default it to being empty. Existing code, `@EnableHypermediaSupport(types = HAL)` will still operate as expected. But simply using the annotation with no arguments will activate ALL types.

Related issues: #1015.
Related pull request: #1065, #1265.

Author: gregturn

src/main/java/org/springframework/hateoas/config/EnableHypermediaSupport.java

@@ -44,11 +44,12 @@
 public @interface EnableHypermediaSupport {
 
 	/**
-	 * The hypermedia type to be supported.
+	 * The hypermedia type to be supported. By default, it is empty, thus activating all available
+	 * {@link MediaTypeConfigurationProvider}s.
 	 *
 	 * @return
 	 */
-	HypermediaType[] type();
+	HypermediaType[] type() default {};
 
 	/**
 	 * Configures which {@link WebStack}s we're supposed to enable support for. By default we're activating it for all

src/main/java/org/springframework/hateoas/config/HypermediaConfigurationImportSelector.java

@@ -71,9 +71,9 @@ public String[] selectImports(AnnotationMetadata metadata) {
 		List<MediaTypeConfigurationProvider> configurationProviders = SpringFactoriesLoader.loadFactories(
 				MediaTypeConfigurationProvider.class, HypermediaConfigurationImportSelector.class.getClassLoader());
 
-		// Filter the ones supporting the given media types
+		// Filter the ones supporting the given media types, or let them all through if none declared.
 		Stream<String> imports = configurationProviders.stream() //
-				.filter(it -> it.supportsAny(types)) //
+				.filter(it -> types.isEmpty() || it.supportsAny(types)) //
 				.map(MediaTypeConfigurationProvider::getConfiguration) //
 				.map(Class::getName);
 

src/test/java/org/springframework/hateoas/config/HypermediaConfigurationImportSelectorUnitTest.java

@@ -118,6 +118,22 @@ void doesNotIncludeWebTestHateoasConfigurationIfWebTestClientIsNotOnTheClasspath
 		}, loader);
 	}
 
+	@Test // #1060
+	void testEmptyHypermediaTypes() {
+
+		withContext(NoConfig.class, context -> {
+
+			Map<String, LinkDiscoverer> linkDiscoverers = context.getBeansOfType(LinkDiscoverer.class);
+
+			assertThat(linkDiscoverers.values()).extracting("class") //
+					.containsExactlyInAnyOrder( //
+							HalLinkDiscoverer.class, //
+							HalFormsLinkDiscoverer.class, //
+							UberLinkDiscoverer.class, //
+							CollectionJsonLinkDiscoverer.class);
+		});
+	}
+
 	@EnableHypermediaSupport(type = HAL)
 	static class HalConfig {
 
@@ -137,4 +153,9 @@ static class HalAndHalFormsConfig {
 	static class AllConfig {
 
 	}
+
+	@EnableHypermediaSupport
+	static class NoConfig {
+
+	}
 }

src/test/java/org/springframework/hateoas/support/CustomHypermediaConfigurationProvider.java

@@ -0,0 +1,38 @@
+/*
+ * Copyright 2019 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.
+ * You may obtain a copy of the License at
+ *
+ *      https://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+package org.springframework.hateoas.support;
+
+import java.util.Collection;
+
+import org.springframework.hateoas.config.HypermediaMappingInformation;
+import org.springframework.hateoas.config.MediaTypeConfigurationProvider;
+import org.springframework.http.MediaType;
+
+/**
+ * @author Greg Turnquist
+ */
+public class CustomHypermediaConfigurationProvider implements MediaTypeConfigurationProvider {
+
+	@Override
+	public Class<? extends HypermediaMappingInformation> getConfiguration() {
+		return CustomHypermediaType.class;
+	}
+
+	@Override
+	public boolean supportsAny(Collection<MediaType> mediaTypes) {
+		return mediaTypes.stream().anyMatch(mediaType -> mediaType.isCompatibleWith(CustomHypermediaType.FRODO_MEDIATYPE));
+	}
+}

src/test/resources/META-INF/spring.factories

@@ -0,0 +1,2 @@
+org.springframework.hateoas.config.MediaTypeConfigurationProvider=\
+ org.springframework.hateoas.support.CustomHypermediaConfigurationProvider