From cd8684499d410f357f3834f070562db25d1fb991 Mon Sep 17 00:00:00 2001 From: Stephane Nicoll Date: Wed, 15 Mar 2017 08:57:24 +0100 Subject: [PATCH] Polish See gh-8621 --- .../NoSpringSecurityHealthMvcEndpointIntegrationTests.java | 4 ++-- .../NoSpringSecurityMvcEndpointSecurityInterceptorTests.java | 5 ----- .../src/main/asciidoc/appendix-executable-jar-format.adoc | 2 +- 3 files changed, 3 insertions(+), 8 deletions(-) diff --git a/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/mvc/NoSpringSecurityHealthMvcEndpointIntegrationTests.java b/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/mvc/NoSpringSecurityHealthMvcEndpointIntegrationTests.java index 1885a14839..ae0865daab 100644 --- a/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/mvc/NoSpringSecurityHealthMvcEndpointIntegrationTests.java +++ b/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/mvc/NoSpringSecurityHealthMvcEndpointIntegrationTests.java @@ -68,7 +68,7 @@ public class NoSpringSecurityHealthMvcEndpointIntegrationTests { } @Test - public void healthWhenRightRoleNotPresentShouldExposeHealthDetails() + public void healthWhenRightRoleNotPresentShouldNotExposeHealthDetails() throws Exception { this.context = new AnnotationConfigWebApplicationContext(); this.context.setServletContext(new MockServletContext()); @@ -77,7 +77,7 @@ public class NoSpringSecurityHealthMvcEndpointIntegrationTests { MockMvc mockMvc = MockMvcBuilders.webAppContextSetup(this.context).build(); mockMvc.perform(get("/health").with(getRequestPostProcessor())) .andExpect(status().isOk()) - .andExpect(content().string(containsString("\"status\":\"UP\""))); + .andExpect(content().string("{\"status\":\"UP\"}")); } @Test diff --git a/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/mvc/NoSpringSecurityMvcEndpointSecurityInterceptorTests.java b/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/mvc/NoSpringSecurityMvcEndpointSecurityInterceptorTests.java index 6515265831..070d430da8 100644 --- a/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/mvc/NoSpringSecurityMvcEndpointSecurityInterceptorTests.java +++ b/spring-boot-actuator/src/test/java/org/springframework/boot/actuate/endpoint/mvc/NoSpringSecurityMvcEndpointSecurityInterceptorTests.java @@ -23,14 +23,12 @@ import java.util.List; import javax.servlet.http.HttpServletResponse; import org.junit.Before; -import org.junit.Rule; import org.junit.Test; import org.junit.runner.RunWith; import org.springframework.boot.actuate.endpoint.AbstractEndpoint; import org.springframework.boot.junit.runner.classpath.ClassPathExclusions; import org.springframework.boot.junit.runner.classpath.ModifiedClassPathRunner; -import org.springframework.boot.test.rule.OutputCapture; import org.springframework.mock.web.MockHttpServletRequest; import org.springframework.mock.web.MockServletContext; import org.springframework.web.method.HandlerMethod; @@ -47,9 +45,6 @@ import static org.mockito.Mockito.mock; @ClassPathExclusions("spring-security-*.jar") public class NoSpringSecurityMvcEndpointSecurityInterceptorTests { - @Rule - public OutputCapture output = new OutputCapture(); - private MvcEndpointSecurityInterceptor securityInterceptor; private TestMvcEndpoint mvcEndpoint; diff --git a/spring-boot-docs/src/main/asciidoc/appendix-executable-jar-format.adoc b/spring-boot-docs/src/main/asciidoc/appendix-executable-jar-format.adoc index 1a0732358d..45fc7ba418 100644 --- a/spring-boot-docs/src/main/asciidoc/appendix-executable-jar-format.adoc +++ b/spring-boot-docs/src/main/asciidoc/appendix-executable-jar-format.adoc @@ -217,7 +217,7 @@ NOTE: `PropertiesLauncher` supports loading properties from |`loader.home` |Used to resolve relative paths in `loader.path`. E.g. `loader.path=lib` then `${loader.home}/lib` is a classpath location (along with all jar files in that - directory). Also used to locate a `loader.properties file`. Example `file:///opt/app` + directory). Also used to locate a `loader.properties` file. Example `file:///opt/app` (defaults to `${user.dir}`). |`loader.args`