|
47 | 47 | import static org.assertj.core.api.Assertions.assertThat;
|
48 | 48 | import static org.assertj.core.api.Assertions.assertThatThrownBy;
|
49 | 49 | import static org.mockito.ArgumentMatchers.any;
|
| 50 | +import static org.mockito.BDDMockito.doThrow; |
50 | 51 | import static org.mockito.BDDMockito.given;
|
51 |
| -import static org.mockito.Mockito.doThrow; |
52 |
| -import static org.mockito.Mockito.mock; |
53 |
| -import static org.mockito.Mockito.verify; |
| 52 | +import static org.mockito.BDDMockito.mock; |
| 53 | +import static org.mockito.BDDMockito.verify; |
54 | 54 |
|
55 | 55 | /**
|
56 | 56 | * Tests for {@link RestTemplateAutoConfiguration}
|
@@ -164,24 +164,21 @@ void whenReactiveWebApplicationRestTemplateBuilderIsNotConfigured() {
|
164 | 164 |
|
165 | 165 | @Test
|
166 | 166 | void customizerShouldCustomizeBuilder() {
|
167 |
| - contextRunner.withUserConfiguration(RestTemplateBuilderCustomizerConfig.class) |
168 |
| - .run((context) -> { |
169 |
| - assertThatThrownBy(() -> context.getBean(RestTemplateBuilder.class).build()) |
170 |
| - .isInstanceOf(IllegalStateException.class) |
171 |
| - .hasMessageContaining("customized builder"); |
172 |
| - }); |
| 167 | + this.contextRunner.withUserConfiguration(RestTemplateBuilderCustomizerConfig.class).run((context) -> |
| 168 | + assertThatThrownBy(() -> context.getBean(RestTemplateBuilder.class).build()) |
| 169 | + .isInstanceOf(IllegalStateException.class).hasMessageContaining("customized builder") |
| 170 | + ); |
173 | 171 | }
|
174 | 172 |
|
175 | 173 | @Test
|
176 | 174 | void customizerShouldReturnInstanceNotANull() {
|
177 |
| - contextRunner.withUserConfiguration(RestTemplateBuilderCustomizerReturnsNullConfig.class) |
178 |
| - .run((context) -> { |
179 |
| - assertThatThrownBy(() -> context.getBean(RestTemplateBuilder.class).build()) |
180 |
| - .isInstanceOf(BeanCreationException.class) |
181 |
| - .hasMessageContaining("Error creating bean with name 'restTemplateBuilder'") |
182 |
| - .hasRootCauseInstanceOf(IllegalArgumentException.class) |
183 |
| - .hasRootCauseMessage("RestTemplateBuilderCustomizer returned null builder"); |
184 |
| - }); |
| 175 | + this.contextRunner.withUserConfiguration(RestTemplateBuilderCustomizerReturnsNullConfig.class).run((context) -> |
| 176 | + assertThatThrownBy(() -> context.getBean(RestTemplateBuilder.class).build()) |
| 177 | + .isInstanceOf(BeanCreationException.class) |
| 178 | + .hasMessageContaining("Error creating bean with name 'restTemplateBuilder'") |
| 179 | + .hasRootCauseInstanceOf(IllegalArgumentException.class) |
| 180 | + .hasRootCauseMessage("RestTemplateBuilderCustomizer returned null builder") |
| 181 | + ); |
185 | 182 | }
|
186 | 183 |
|
187 | 184 | @Configuration(proxyBeanMethods = false)
|
@@ -261,21 +258,26 @@ static class CustomHttpMessageConverter extends StringHttpMessageConverter {
|
261 | 258 |
|
262 | 259 | @Configuration(proxyBeanMethods = false)
|
263 | 260 | static class RestTemplateBuilderCustomizerConfig {
|
| 261 | + |
264 | 262 | @Bean
|
265 | 263 | RestTemplateBuilderCustomizer restTemplateBuilderCustomizer() {
|
266 |
| - return oldBuilder -> { |
| 264 | + return (oldBuilder) -> { |
267 | 265 | final RestTemplateBuilder builder = Mockito.spy(oldBuilder);
|
268 | 266 | doThrow(new IllegalStateException("customized builder")).when(builder).build();
|
269 | 267 | return builder;
|
270 | 268 | };
|
271 | 269 | }
|
| 270 | + |
272 | 271 | }
|
273 | 272 |
|
274 | 273 | @Configuration(proxyBeanMethods = false)
|
275 | 274 | static class RestTemplateBuilderCustomizerReturnsNullConfig {
|
| 275 | + |
276 | 276 | @Bean
|
277 | 277 | RestTemplateBuilderCustomizer restTemplateBuilderCustomizer() {
|
278 |
| - return oldBuilder -> null; |
| 278 | + return (oldBuilder) -> null; |
279 | 279 | }
|
| 280 | + |
280 | 281 | }
|
| 282 | + |
281 | 283 | }
|
0 commit comments