#1061 - Reenable default codecs for WebFlux configuration.

Spring WebFlux now correctly handles custom codecs. This means we can stop disabling them. This change includes more test cases, verifying Spring MVC configuration for hypermedia as well, verifying both Web MVC and WebFlux are properly and consistently configured.

Probably supercedes: #1047

Author: gregturn

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

@@ -124,21 +124,14 @@ static class HypermediaWebFluxConfigurer implements WebFluxConfigurer {
 		@Override
 		public void configureHttpMessageCodecs(ServerCodecConfigurer configurer) {
 
-			CodecConfigurer.CustomCodecs customCodecs = configurer.customCodecs();
-
 			this.hypermediaTypes.forEach(hypermedia -> {
 
-				MimeType[] mimeTypes = hypermedia.getMediaTypes().toArray(new MimeType[0]);
-
 				ObjectMapper objectMapper = hypermedia.configureObjectMapper(this.mapper.copy());
-				customCodecs.encoder(new Jackson2JsonEncoder(objectMapper, mimeTypes));
-				customCodecs.decoder(new Jackson2JsonDecoder(objectMapper, mimeTypes));
+				MimeType[] mimeTypes = hypermedia.getMediaTypes().toArray(new MimeType[0]);
+				
+				configurer.customCodecs().encoder(new Jackson2JsonEncoder(objectMapper, mimeTypes));
+				configurer.customCodecs().decoder(new Jackson2JsonDecoder(objectMapper, mimeTypes));
 			});
-
-			customCodecs.encoder(CharSequenceEncoder.allMimeTypes());
-			customCodecs.decoder(StringDecoder.allMimeTypes());
-
-			configurer.registerDefaults(false);
 		}
 	}
 }

src/main/java/org/springframework/hateoas/mediatype/collectionjson/Jackson2CollectionJsonModule.java

@@ -65,7 +65,7 @@
  * @author Greg Turnquist
  * @author Oliver Drotbohm
  */
-class Jackson2CollectionJsonModule extends SimpleModule {
+public class Jackson2CollectionJsonModule extends SimpleModule {
 
 	private static final long serialVersionUID = -6540574644565592709L;
 

src/main/java/org/springframework/hateoas/mediatype/hal/forms/Jackson2HalFormsModule.java

@@ -73,7 +73,7 @@
  * @author Greg Turnquist
  * @author Oliver Gierke
  */
-class Jackson2HalFormsModule extends SimpleModule {
+public class Jackson2HalFormsModule extends SimpleModule {
 
 	private static final long serialVersionUID = -4496351128468451196L;
 

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

@@ -50,6 +50,7 @@
 import org.springframework.web.bind.annotation.RequestBody;
 import org.springframework.web.bind.annotation.RestController;
 import org.springframework.web.reactive.config.EnableWebFlux;
+import org.springframework.web.servlet.config.annotation.EnableWebMvc;
 
 /**
  * @author Greg Turnquist
@@ -258,12 +259,17 @@ void registeringAllHypermediaTypesShouldAllowCreatingThroughAllFormats() {
 	 * @see #728
 	 */
 	@Test
-	void callingForUnregisteredMediaTypeShouldFail() {
+	void callingForUnregisteredMediaTypeShouldFallBackToJackson() {
 
 		setUp(HalWebFluxConfig.class);
 
-		this.testClient.get().uri("/").accept(MediaTypes.UBER_JSON).exchange().expectStatus().is4xxClientError()
-				.returnResult(String.class).getResponseBody().as(StepVerifier::create).verifyComplete();
+		this.testClient.get().uri("/").accept(MediaTypes.UBER_JSON) //
+				.exchange() //
+				.expectStatus().isOk() //
+				.returnResult(String.class).getResponseBody() //
+				.as(StepVerifier::create) //
+				.expectNext("{\"links\":[{\"rel\":\"self\",\"href\":\"/\"},{\"rel\":\"employees\",\"href\":\"/employees\"}]}")
+				.verifyComplete();
 	}
 
 	/**