Commit c05a2995 authored by Johnny Lim's avatar Johnny Lim Committed by Stephane Nicoll

Polish

Closes gh-10921
parent c9e18e21
...@@ -23,9 +23,7 @@ import okhttp3.mockwebserver.MockWebServer; ...@@ -23,9 +23,7 @@ import okhttp3.mockwebserver.MockWebServer;
import okhttp3.mockwebserver.RecordedRequest; import okhttp3.mockwebserver.RecordedRequest;
import org.junit.After; import org.junit.After;
import org.junit.Before; import org.junit.Before;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import reactor.test.StepVerifier; import reactor.test.StepVerifier;
import org.springframework.boot.actuate.autoconfigure.cloudfoundry.AccessLevel; import org.springframework.boot.actuate.autoconfigure.cloudfoundry.AccessLevel;
...@@ -43,9 +41,6 @@ import static org.assertj.core.api.Assertions.assertThat; ...@@ -43,9 +41,6 @@ import static org.assertj.core.api.Assertions.assertThat;
*/ */
public class ReactiveCloudFoundrySecurityServiceTests { public class ReactiveCloudFoundrySecurityServiceTests {
@Rule
public ExpectedException thrown = ExpectedException.none();
private static final String CLOUD_CONTROLLER = "/my-cloud-controller.com"; private static final String CLOUD_CONTROLLER = "/my-cloud-controller.com";
private static final String CLOUD_CONTROLLER_PERMISSIONS = CLOUD_CONTROLLER private static final String CLOUD_CONTROLLER_PERMISSIONS = CLOUD_CONTROLLER
......
...@@ -30,9 +30,7 @@ import java.util.Map; ...@@ -30,9 +30,7 @@ import java.util.Map;
import org.apache.commons.codec.binary.Base64; import org.apache.commons.codec.binary.Base64;
import org.junit.Before; import org.junit.Before;
import org.junit.Rule;
import org.junit.Test; import org.junit.Test;
import org.junit.rules.ExpectedException;
import org.mockito.Mock; import org.mockito.Mock;
import org.mockito.MockitoAnnotations; import org.mockito.MockitoAnnotations;
import reactor.core.publisher.Mono; import reactor.core.publisher.Mono;
...@@ -58,9 +56,6 @@ public class ReactiveTokenValidatorTests { ...@@ -58,9 +56,6 @@ public class ReactiveTokenValidatorTests {
private static final Charset UTF_8 = Charset.forName("UTF-8"); private static final Charset UTF_8 = Charset.forName("UTF-8");
@Rule
public ExpectedException thrown = ExpectedException.none();
@Mock @Mock
private ReactiveCloudFoundrySecurityService securityService; private ReactiveCloudFoundrySecurityService securityService;
......
...@@ -82,7 +82,7 @@ class GsonHttpMessageConvertersConfiguration { ...@@ -82,7 +82,7 @@ class GsonHttpMessageConvertersConfiguration {
} }
@ConditionalOnBean(MappingJackson2HttpMessageConverter.class) @ConditionalOnBean(MappingJackson2HttpMessageConverter.class)
static class JacksonMissing { static class JacksonAvailable {
} }
......
...@@ -241,6 +241,8 @@ public class HttpMessageConvertersAutoConfigurationTests { ...@@ -241,6 +241,8 @@ public class HttpMessageConvertersAutoConfigurationTests {
.run((context) -> { .run((context) -> {
assertConverterBeanExists(context, GsonHttpMessageConverter.class, assertConverterBeanExists(context, GsonHttpMessageConverter.class,
"gsonHttpMessageConverter"); "gsonHttpMessageConverter");
assertConverterBeanRegisteredWithHttpMessageConverters(context,
GsonHttpMessageConverter.class);
assertThat(context).doesNotHaveBean(JsonbHttpMessageConverter.class); assertThat(context).doesNotHaveBean(JsonbHttpMessageConverter.class);
}); });
} }
......
...@@ -822,7 +822,7 @@ Consider the following class: ...@@ -822,7 +822,7 @@ Consider the following class:
} }
---- ----
The preceding example produces metdata information for `server.name`, `server.host.ip`, The preceding example produces metadata information for `server.name`, `server.host.ip`,
and `server.host.port` properties. You can use the `@NestedConfigurationProperty` and `server.host.port` properties. You can use the `@NestedConfigurationProperty`
annotation on a field to indicate that a regular (non-inner) class should be treated as annotation on a field to indicate that a regular (non-inner) class should be treated as
if it were nested. if it were nested.
......
...@@ -25,7 +25,7 @@ The buildpack wraps your deployed code in whatever is needed to _start_ your app ...@@ -25,7 +25,7 @@ The buildpack wraps your deployed code in whatever is needed to _start_ your app
it might be a JDK and a call to `java`, it might be an embedded web server, or it might be it might be a JDK and a call to `java`, it might be an embedded web server, or it might be
a full-fledged application server. A buildpack is pluggable, but ideally you should be a full-fledged application server. A buildpack is pluggable, but ideally you should be
able to get by with as few customizations to it as possible. This reduces the footprint of able to get by with as few customizations to it as possible. This reduces the footprint of
unctionality that is not under your control. It minimizes divergence between development functionality that is not under your control. It minimizes divergence between development
and production environments. and production environments.
Ideally, your application, like a Spring Boot executable jar, has everything that it needs Ideally, your application, like a Spring Boot executable jar, has everything that it needs
......
...@@ -206,7 +206,7 @@ the `<build/>` element of your `pom.xml`: ...@@ -206,7 +206,7 @@ the `<build/>` element of your `pom.xml`:
</resources> </resources>
---- ----
You also need to incude the following element inside `<plugins/>`: You also need to include the following element inside `<plugins/>`:
[source,xml,indent=0] [source,xml,indent=0]
---- ----
...@@ -710,7 +710,7 @@ processed early (before the value is actually available). ...@@ -710,7 +710,7 @@ processed early (before the value is actually available).
=== Configure SSL === Configure SSL
SSL can be configured declaratively by setting the various `+server.ssl.*+` properties, SSL can be configured declaratively by setting the various `+server.ssl.*+` properties,
typically in `application.properties` or `application.yml`. The following example shows typically in `application.properties` or `application.yml`. The following example shows
setting SSL properties in `applicaiton.properties`: setting SSL properties in `application.properties`:
[source,properties,indent=0,subs="verbatim,quotes,attributes"] [source,properties,indent=0,subs="verbatim,quotes,attributes"]
---- ----
...@@ -1591,7 +1591,7 @@ To run an app with that `DataSource`, all you need is the connection ...@@ -1591,7 +1591,7 @@ To run an app with that `DataSource`, all you need is the connection
information. Pool-specific settings can also be provided. Check the implementation that information. Pool-specific settings can also be provided. Check the implementation that
is going to be used at runtime for more details. is going to be used at runtime for more details.
The following example shows how to deine a JDBC data source by setting properties: The following example shows how to define a JDBC data source by setting properties:
[source,properties,indent=0] [source,properties,indent=0]
---- ----
...@@ -1947,7 +1947,7 @@ If you are configuring a component yourself, you can use an ...@@ -1947,7 +1947,7 @@ If you are configuring a component yourself, you can use an
`EntityManagerFactoryDependsOnPostProcessor` subclass as a convenient way of setting up `EntityManagerFactoryDependsOnPostProcessor` subclass as a convenient way of setting up
the necessary dependencies. For example, if you use Hibernate Search with the necessary dependencies. For example, if you use Hibernate Search with
Elasticsearch as its index manager, any `EntityManagerFactory` beans must be Elasticsearch as its index manager, any `EntityManagerFactory` beans must be
configured to depend on the `elasticsearchClient` bean, as shown in the follwing example: configured to depend on the `elasticsearchClient` bean, as shown in the following example:
[source,java,indent=0] [source,java,indent=0]
---- ----
......
...@@ -938,7 +938,7 @@ information: ...@@ -938,7 +938,7 @@ information:
- The request's method - The request's method
- The request's URI (templated if possible). - The request's URI (templated if possible).
- The esponse's status. - The response's status.
To customize the tags, use the `defaultTags` method on your `RouterFunctionMetrics` To customize the tags, use the `defaultTags` method on your `RouterFunctionMetrics`
instance. instance.
......
...@@ -2137,7 +2137,7 @@ used instead of the `ErrorAttributes` representation. ...@@ -2137,7 +2137,7 @@ used instead of the `ErrorAttributes` representation.
[[boot-features-error-handling-custom-error-pages]] [[boot-features-error-handling-custom-error-pages]]
===== Custom Error Eages ===== Custom Error Pages
If you want to display a custom HTML error page for a given status code, you can add a If you want to display a custom HTML error page for a given status code, you can add a
file to an `/error` folder. Error pages can either be static HTML (that is, added under file to an `/error` folder. Error pages can either be static HTML (that is, added under
any of the static resource folders) or built by using templates. The name of the file any of the static resource folders) or built by using templates. The name of the file
...@@ -2777,7 +2777,7 @@ register a Spring bean that implements the `WebServerFactoryCustomizer` interfac ...@@ -2777,7 +2777,7 @@ register a Spring bean that implements the `WebServerFactoryCustomizer` interfac
`WebServerFactoryCustomizer` provides access to the `WebServerFactoryCustomizer` provides access to the
`ConfigurableServletWebServerFactory`, which includes numerous customization setter `ConfigurableServletWebServerFactory`, which includes numerous customization setter
methods. Dedicated variants exist for Tomcat, Jetty, and Undertow. The following example methods. Dedicated variants exist for Tomcat, Jetty, and Undertow. The following example
shows programatically setting the port: shows programmatically setting the port:
[source,java,indent=0] [source,java,indent=0]
---- ----
...@@ -3070,7 +3070,7 @@ Production database connections can also be auto-configured by using a pooling ...@@ -3070,7 +3070,7 @@ Production database connections can also be auto-configured by using a pooling
`DataSource`. Spring Boot uses the following algorithm for choosing a specific `DataSource`. Spring Boot uses the following algorithm for choosing a specific
implementation: implementation:
. We prefer HikariCP for its performance and concurrency. If HkiariCP is available, we . We prefer HikariCP for its performance and concurrency. If HikariCP is available, we
always choose it. always choose it.
. Otherwise, if the Tomcat pooling `DataSource` is available, we use it. . Otherwise, if the Tomcat pooling `DataSource` is available, we use it.
. If neither HikariCP nor the Tomcat pooling datasource are available and if Commons . If neither HikariCP nor the Tomcat pooling datasource are available and if Commons
...@@ -6173,7 +6173,7 @@ Spring Data JPA repositories. Regular `@Component` beans are not loaded into the ...@@ -6173,7 +6173,7 @@ Spring Data JPA repositories. Regular `@Component` beans are not loaded into the
By default, data JPA tests are transactional and roll back at the end of each test. See By default, data JPA tests are transactional and roll back at the end of each test. See
the {spring-reference}testing.html#testcontext-tx-enabling-transactions[relevant section] the {spring-reference}testing.html#testcontext-tx-enabling-transactions[relevant section]
in the Spring Framwork Reference Documentation for more details. If that is not what you in the Spring Framework Reference Documentation for more details. If that is not what you
want, you can disable transaction management for a test or for the whole class as want, you can disable transaction management for a test or for the whole class as
follows: follows:
...@@ -6438,7 +6438,7 @@ A list of the auto-configuration settings that are enabled by `@DataNeo4jTest` c ...@@ -6438,7 +6438,7 @@ A list of the auto-configuration settings that are enabled by `@DataNeo4jTest` c
==== Auto-configured Data Redis Tests ==== Auto-configured Data Redis Tests
You can use `@DataRedisTest` to test Redis applications. By default, it scans for You can use `@DataRedisTest` to test Redis applications. By default, it scans for
`@RedisHash` classes and configures Spring Data Redis repositories. Regular `@Component` `@RedisHash` classes and configures Spring Data Redis repositories. Regular `@Component`
beans are not loaded into the `ApplicationContext`. The follwing example shows the beans are not loaded into the `ApplicationContext`. The following example shows the
`@DataRedisTest` annotation in use: `@DataRedisTest` annotation in use:
[source,java,indent=0] [source,java,indent=0]
...@@ -6510,7 +6510,7 @@ following example: ...@@ -6510,7 +6510,7 @@ following example:
} }
---- ----
A list of the auto-configuration settiongs that are enabled by `@DataLdapTest` can be A list of the auto-configuration settings that are enabled by `@DataLdapTest` can be
<<appendix-test-auto-configuration#test-auto-configuration,found in the appendix>>. <<appendix-test-auto-configuration#test-auto-configuration,found in the appendix>>.
......
...@@ -47,7 +47,7 @@ public @interface AutoConfigureJsonTesters { ...@@ -47,7 +47,7 @@ public @interface AutoConfigureJsonTesters {
/** /**
* If {@link BasicJsonTester}, {@link JacksonTester}, {@link JsonbTester} and * If {@link BasicJsonTester}, {@link JacksonTester}, {@link JsonbTester} and
* {@link GsonTester} beans should be registered. Defaults to {@code true} * {@link GsonTester} beans should be registered. Defaults to {@code true}.
* @return if tester support is enabled * @return if tester support is enabled
*/ */
boolean enabled() default true; boolean enabled() default true;
......
...@@ -41,7 +41,7 @@ import org.springframework.test.context.junit4.SpringRunner; ...@@ -41,7 +41,7 @@ import org.springframework.test.context.junit4.SpringRunner;
* existing bean is defined a new one will be added. Dependencies that are known to the * existing bean is defined a new one will be added. Dependencies that are known to the
* application context but are not beans (such as those * application context but are not beans (such as those
* {@link org.springframework.beans.factory.config.ConfigurableListableBeanFactory#registerResolvableDependency(Class, Object) * {@link org.springframework.beans.factory.config.ConfigurableListableBeanFactory#registerResolvableDependency(Class, Object)
* registered directly} ) will not be found and a mocked bean will be added to the context * registered directly}) will not be found and a mocked bean will be added to the context
* alongside the existing dependency. * alongside the existing dependency.
* <p> * <p>
* When {@code @MockBean} is used on a field, as well as being registered in the * When {@code @MockBean} is used on a field, as well as being registered in the
......
...@@ -41,7 +41,7 @@ import org.springframework.test.context.junit4.SpringRunner; ...@@ -41,7 +41,7 @@ import org.springframework.test.context.junit4.SpringRunner;
* will be added. Dependencies that are known to the application context but are not beans * will be added. Dependencies that are known to the application context but are not beans
* (such as those * (such as those
* {@link org.springframework.beans.factory.config.ConfigurableListableBeanFactory#registerResolvableDependency(Class, Object) * {@link org.springframework.beans.factory.config.ConfigurableListableBeanFactory#registerResolvableDependency(Class, Object)
* registered directly} ) will not be found and a spied bean will be added to the context * registered directly}) will not be found and a spied bean will be added to the context
* alongside the existing dependency. * alongside the existing dependency.
* <p> * <p>
* When {@code @SpyBean} is used on a field, as well as being registered in the * When {@code @SpyBean} is used on a field, as well as being registered in the
......
...@@ -44,7 +44,7 @@ public class SpyBeanWithNameOnTestFieldForMultipleExistingBeansTests { ...@@ -44,7 +44,7 @@ public class SpyBeanWithNameOnTestFieldForMultipleExistingBeansTests {
@Test @Test
public void testSpying() throws Exception { public void testSpying() throws Exception {
MockingDetails mockingDetails = Mockito.mockingDetails(this.spy); MockingDetails mockingDetails = Mockito.mockingDetails(this.spy);
assertThat(mockingDetails.isMock()).isTrue(); assertThat(mockingDetails.isSpy()).isTrue();
assertThat(mockingDetails.getMockCreationSettings().getMockName().toString()) assertThat(mockingDetails.getMockCreationSettings().getMockName().toString())
.isEqualTo("two"); .isEqualTo("two");
} }
......
...@@ -104,7 +104,7 @@ public class DefaultLaunchScript implements LaunchScript { ...@@ -104,7 +104,7 @@ public class DefaultLaunchScript implements LaunchScript {
if (properties != null && properties.containsKey(name)) { if (properties != null && properties.containsKey(name)) {
Object propertyValue = properties.get(name); Object propertyValue = properties.get(name);
if (FILE_PATH_KEYS.contains(name)) { if (FILE_PATH_KEYS.contains(name)) {
value = parseFilePropertyValue(properties.get(name)); value = parseFilePropertyValue(propertyValue);
} }
else { else {
value = propertyValue.toString(); value = propertyValue.toString();
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment