From 3f1aaa1f7e4d67969f6c95922c94ab5d5dd461db Mon Sep 17 00:00:00 2001 From: Stephane Nicoll Date: Fri, 26 Jan 2018 09:54:13 +0100 Subject: [PATCH] Polish contribution Closes gh-11783 --- .../data/ldap/DataLdapTestIntegrationTests.java | 2 +- .../boot/web/servlet/server/StaticResourceJars.java | 2 +- ...idConfigurationPropertyValueFailureAnalyzerTests.java | 9 ++++++--- 3 files changed, 8 insertions(+), 5 deletions(-) diff --git a/spring-boot-project/spring-boot-test-autoconfigure/src/test/java/org/springframework/boot/test/autoconfigure/data/ldap/DataLdapTestIntegrationTests.java b/spring-boot-project/spring-boot-test-autoconfigure/src/test/java/org/springframework/boot/test/autoconfigure/data/ldap/DataLdapTestIntegrationTests.java index 674e209132..458525bf6f 100644 --- a/spring-boot-project/spring-boot-test-autoconfigure/src/test/java/org/springframework/boot/test/autoconfigure/data/ldap/DataLdapTestIntegrationTests.java +++ b/spring-boot-project/spring-boot-test-autoconfigure/src/test/java/org/springframework/boot/test/autoconfigure/data/ldap/DataLdapTestIntegrationTests.java @@ -1,5 +1,5 @@ /* - * Copyright 2012-2017 the original author or authors. + * Copyright 2012-2018 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. diff --git a/spring-boot-project/spring-boot/src/main/java/org/springframework/boot/web/servlet/server/StaticResourceJars.java b/spring-boot-project/spring-boot/src/main/java/org/springframework/boot/web/servlet/server/StaticResourceJars.java index c95940764a..74ca4d7214 100644 --- a/spring-boot-project/spring-boot/src/main/java/org/springframework/boot/web/servlet/server/StaticResourceJars.java +++ b/spring-boot-project/spring-boot/src/main/java/org/springframework/boot/web/servlet/server/StaticResourceJars.java @@ -1,5 +1,5 @@ /* - * Copyright 2012-2017 the original author or authors. + * Copyright 2012-2018 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. diff --git a/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/diagnostics/analyzer/InvalidConfigurationPropertyValueFailureAnalyzerTests.java b/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/diagnostics/analyzer/InvalidConfigurationPropertyValueFailureAnalyzerTests.java index 984904f60f..563fe7d675 100644 --- a/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/diagnostics/analyzer/InvalidConfigurationPropertyValueFailureAnalyzerTests.java +++ b/spring-boot-project/spring-boot/src/test/java/org/springframework/boot/diagnostics/analyzer/InvalidConfigurationPropertyValueFailureAnalyzerTests.java @@ -58,6 +58,8 @@ public class InvalidConfigurationPropertyValueFailureAnalyzerTests { "test.property", "invalid", "This is not valid."); FailureAnalysis analysis = performAnalysis(failure); assertCommonParts(failure, analysis); + assertThat(analysis.getAction()) + .contains("Review the value of the property with the provided reason."); assertThat(analysis.getDescription()) .contains("Validation failed for the following reason") .contains("This is not valid.") @@ -73,7 +75,8 @@ public class InvalidConfigurationPropertyValueFailureAnalyzerTests { InvalidConfigurationPropertyValueException failure = new InvalidConfigurationPropertyValueException( "test.property", "invalid", null); FailureAnalysis analysis = performAnalysis(failure); - assertCommonParts(failure, analysis); + assertThat(analysis.getAction()) + .contains("Review the value of the property."); assertThat(analysis.getDescription()).contains("No reason was provided.") .doesNotContain("Additionally, this property is also set"); } @@ -95,6 +98,8 @@ public class InvalidConfigurationPropertyValueFailureAnalyzerTests { "test.property", "invalid", "This is not valid."); FailureAnalysis analysis = performAnalysis(failure); assertCommonParts(failure, analysis); + assertThat(analysis.getAction()) + .contains("Review the value of the property with the provided reason."); assertThat(analysis.getDescription()) .contains("Additionally, this property is also set in the following " + "property sources:") @@ -113,8 +118,6 @@ public class InvalidConfigurationPropertyValueFailureAnalyzerTests { FailureAnalysis analysis) { assertThat(analysis.getDescription()).contains("test.property") .contains("invalid").contains("TestOrigin test.property"); - assertThat(analysis.getAction()) - .contains("Review the value of the property with the provided reason."); assertThat(analysis.getCause()).isSameAs(failure); }