From 6739ca82ce616c3790cc580b7eabe1dedb43837c Mon Sep 17 00:00:00 2001 From: Johnny Lim Date: Fri, 17 Feb 2023 23:14:31 +0900 Subject: [PATCH] Polishing See gh-23846 See gh-29955 Closes gh-29992 --- ...bstractCoroutinesTransactionAspectTests.kt | 34 +++++++++---------- .../springframework/http/CacheControl.java | 1 + .../bind/MethodArgumentNotValidException.java | 1 + ...sourceRegionHttpMessageConverterTests.java | 8 ++--- ...odelAttributeMethodProcessorKotlinTests.kt | 16 ++++----- .../web/server/CoWebFilterTests.kt | 4 +-- 6 files changed, 33 insertions(+), 31 deletions(-) diff --git a/spring-tx/src/test/kotlin/org/springframework/transaction/interceptor/AbstractCoroutinesTransactionAspectTests.kt b/spring-tx/src/test/kotlin/org/springframework/transaction/interceptor/AbstractCoroutinesTransactionAspectTests.kt index 1f5c936a8cb..f4c4ba096f4 100644 --- a/spring-tx/src/test/kotlin/org/springframework/transaction/interceptor/AbstractCoroutinesTransactionAspectTests.kt +++ b/spring-tx/src/test/kotlin/org/springframework/transaction/interceptor/AbstractCoroutinesTransactionAspectTests.kt @@ -1,5 +1,5 @@ /* - * Copyright 2002-2022 the original author or authors. + * Copyright 2002-2023 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. @@ -22,7 +22,7 @@ import org.assertj.core.api.Assertions.assertThat import org.assertj.core.api.Fail import org.junit.jupiter.api.BeforeEach import org.junit.jupiter.api.Test -import org.mockito.BDDMockito +import org.mockito.BDDMockito.* import org.mockito.Mockito import org.springframework.transaction.* import org.springframework.transaction.reactive.TransactionContext @@ -83,8 +83,8 @@ abstract class AbstractCoroutinesTransactionAspectTests { val status = Mockito.mock(ReactiveTransaction::class.java) val rtm = Mockito.mock(ReactiveTransactionManager::class.java) // expect a transaction - BDDMockito.given(rtm.getReactiveTransaction(txatt)).willReturn(Mono.just(status)) - BDDMockito.given(rtm.commit(status)).willReturn(Mono.empty()) + given(rtm.getReactiveTransaction(txatt)).willReturn(Mono.just(status)) + given(rtm.commit(status)).willReturn(Mono.empty()) val tb = DefaultTestBean() val itb = advised(tb, rtm, tas) as TestBean runBlocking { @@ -106,8 +106,8 @@ abstract class AbstractCoroutinesTransactionAspectTests { val status = Mockito.mock(ReactiveTransaction::class.java) val rtm = Mockito.mock(ReactiveTransactionManager::class.java) // expect a transaction - BDDMockito.given(rtm.getReactiveTransaction(txatt)).willReturn(Mono.just(status)) - BDDMockito.given(rtm.commit(status)).willReturn(Mono.empty()) + given(rtm.getReactiveTransaction(txatt)).willReturn(Mono.just(status)) + given(rtm.commit(status)).willReturn(Mono.empty()) val tb = DefaultTestBean() val itb = advised(tb, rtm, arrayOf(tas1, tas2)) as TestBean runBlocking { @@ -128,8 +128,8 @@ abstract class AbstractCoroutinesTransactionAspectTests { val status = Mockito.mock(ReactiveTransaction::class.java) val rtm = Mockito.mock(ReactiveTransactionManager::class.java) // expect a transaction - BDDMockito.given(rtm.getReactiveTransaction(txatt)).willReturn(Mono.just(status)) - BDDMockito.given(rtm.commit(status)).willReturn(Mono.empty()) + given(rtm.getReactiveTransaction(txatt)).willReturn(Mono.just(status)) + given(rtm.commit(status)).willReturn(Mono.empty()) val tb = DefaultTestBean() val itb = advised(tb, rtm, tas) as TestBean runBlocking { @@ -198,17 +198,17 @@ abstract class AbstractCoroutinesTransactionAspectTests { val status = Mockito.mock(ReactiveTransaction::class.java) val rtm = Mockito.mock(ReactiveTransactionManager::class.java) // Gets additional call(s) from TransactionControl - BDDMockito.given(rtm.getReactiveTransaction(txatt)).willReturn(Mono.just(status)) + given(rtm.getReactiveTransaction(txatt)).willReturn(Mono.just(status)) val tex = TransactionSystemException("system exception") if (rollbackException) { if (shouldRollback) { - BDDMockito.given(rtm.rollback(status)).willReturn(Mono.error(tex)) + given(rtm.rollback(status)).willReturn(Mono.error(tex)) } else { - BDDMockito.given(rtm.commit(status)).willReturn(Mono.error(tex)) + given(rtm.commit(status)).willReturn(Mono.error(tex)) } } else { - BDDMockito.given(rtm.commit(status)).willReturn(Mono.empty()) - BDDMockito.given(rtm.rollback(status)).willReturn(Mono.empty()) + given(rtm.commit(status)).willReturn(Mono.empty()) + given(rtm.rollback(status)).willReturn(Mono.empty()) } val tb = DefaultTestBean() val itb = advised(tb, rtm, tas) as TestBean @@ -246,7 +246,7 @@ abstract class AbstractCoroutinesTransactionAspectTests { val rtm = Mockito.mock(ReactiveTransactionManager::class.java) // Expect a transaction val ex = CannotCreateTransactionException("foobar") - BDDMockito.given(rtm.getReactiveTransaction(txatt)).willThrow(ex) + given(rtm.getReactiveTransaction(txatt)).willThrow(ex) val tb: DefaultTestBean = object : DefaultTestBean() { override suspend fun getName(): String? { throw UnsupportedOperationException( @@ -279,10 +279,10 @@ abstract class AbstractCoroutinesTransactionAspectTests { // No attributes for m2 val rtm = Mockito.mock(ReactiveTransactionManager::class.java) val status = Mockito.mock(ReactiveTransaction::class.java) - BDDMockito.given(rtm.getReactiveTransaction(txatt)).willReturn(Mono.just(status)) + given(rtm.getReactiveTransaction(txatt)).willReturn(Mono.just(status)) val ex = UnexpectedRollbackException("foobar") - BDDMockito.given(rtm.commit(status)).willReturn(Mono.error(ex)) - BDDMockito.given(rtm.rollback(status)).willReturn(Mono.empty()) + given(rtm.commit(status)).willReturn(Mono.error(ex)) + given(rtm.rollback(status)).willReturn(Mono.empty()) val tb = DefaultTestBean() val itb = advised(tb, rtm, tas) as TestBean val name = "new name" diff --git a/spring-web/src/main/java/org/springframework/http/CacheControl.java b/spring-web/src/main/java/org/springframework/http/CacheControl.java index a7b29acda41..4cb693cf8f9 100644 --- a/spring-web/src/main/java/org/springframework/http/CacheControl.java +++ b/spring-web/src/main/java/org/springframework/http/CacheControl.java @@ -329,6 +329,7 @@ public class CacheControl { * Adding a {@link #maxAge(Duration) max-age} directive is strongly advised * to enforce the actual freshness lifetime. * @return {@code this}, to facilitate method chaining + * @since 6.0.5 * @see rfc8246 */ public CacheControl immutable() { diff --git a/spring-web/src/main/java/org/springframework/web/bind/MethodArgumentNotValidException.java b/spring-web/src/main/java/org/springframework/web/bind/MethodArgumentNotValidException.java index 46290aa206b..ce1c20e7dd5 100644 --- a/spring-web/src/main/java/org/springframework/web/bind/MethodArgumentNotValidException.java +++ b/spring-web/src/main/java/org/springframework/web/bind/MethodArgumentNotValidException.java @@ -74,6 +74,7 @@ public class MethodArgumentNotValidException extends BindException implements Er * Constructor for {@link MethodArgumentNotValidException}. * @param executable the executable that failed validation * @param bindingResult the results of the validation + * @since 6.0.5 */ public MethodArgumentNotValidException(Executable executable, BindingResult bindingResult) { super(bindingResult); diff --git a/spring-web/src/test/java/org/springframework/http/converter/ResourceRegionHttpMessageConverterTests.java b/spring-web/src/test/java/org/springframework/http/converter/ResourceRegionHttpMessageConverterTests.java index 356904e3f25..407fd758c75 100644 --- a/spring-web/src/test/java/org/springframework/http/converter/ResourceRegionHttpMessageConverterTests.java +++ b/spring-web/src/test/java/org/springframework/http/converter/ResourceRegionHttpMessageConverterTests.java @@ -24,7 +24,6 @@ import java.util.Collections; import java.util.List; import org.junit.jupiter.api.Test; -import org.mockito.BDDMockito; import org.springframework.core.ParameterizedTypeReference; import org.springframework.core.io.ClassPathResource; @@ -37,6 +36,7 @@ import org.springframework.util.StringUtils; import org.springframework.web.testfixture.http.MockHttpOutputMessage; import static org.assertj.core.api.Assertions.assertThat; +import static org.mockito.BDDMockito.given; import static org.mockito.Mockito.mock; /** @@ -187,9 +187,9 @@ public class ResourceRegionHttpMessageConverterTests { public void applicationOctetStreamDefaultContentType() throws Exception { MockHttpOutputMessage outputMessage = new MockHttpOutputMessage(); ClassPathResource body = mock(); - BDDMockito.given(body.getFilename()).willReturn("spring.dat"); - BDDMockito.given(body.contentLength()).willReturn(12L); - BDDMockito.given(body.getInputStream()).willReturn(new ByteArrayInputStream("Spring Framework".getBytes())); + given(body.getFilename()).willReturn("spring.dat"); + given(body.contentLength()).willReturn(12L); + given(body.getInputStream()).willReturn(new ByteArrayInputStream("Spring Framework".getBytes())); HttpRange range = HttpRange.createByteRange(0, 5); ResourceRegion resourceRegion = range.toResourceRegion(body); diff --git a/spring-web/src/test/kotlin/org/springframework/web/method/annotation/ModelAttributeMethodProcessorKotlinTests.kt b/spring-web/src/test/kotlin/org/springframework/web/method/annotation/ModelAttributeMethodProcessorKotlinTests.kt index b088360ee60..62047b74a9a 100644 --- a/spring-web/src/test/kotlin/org/springframework/web/method/annotation/ModelAttributeMethodProcessorKotlinTests.kt +++ b/spring-web/src/test/kotlin/org/springframework/web/method/annotation/ModelAttributeMethodProcessorKotlinTests.kt @@ -21,8 +21,8 @@ import org.assertj.core.api.Assertions.assertThatThrownBy import org.junit.jupiter.api.BeforeEach import org.junit.jupiter.api.Test import org.mockito.ArgumentMatchers.* -import org.mockito.BDDMockito -import org.mockito.Mockito +import org.mockito.BDDMockito.given +import org.mockito.Mockito.mock import org.springframework.core.MethodParameter import org.springframework.core.annotation.SynthesizingMethodParameter import org.springframework.web.bind.MethodArgumentNotValidException @@ -51,7 +51,7 @@ class ModelAttributeMethodProcessorKotlinTests { fun setup() { container = ModelAndViewContainer() processor = ModelAttributeMethodProcessor(false) - var method = ModelAttributeHandler::class.java.getDeclaredMethod("test",Param::class.java) + val method = ModelAttributeHandler::class.java.getDeclaredMethod("test", Param::class.java) param = SynthesizingMethodParameter(method, 0) } @@ -59,8 +59,8 @@ class ModelAttributeMethodProcessorKotlinTests { fun resolveArgumentWithValue() { val mockRequest = MockHttpServletRequest().apply { addParameter("a", "b") } val requestWithParam = ServletWebRequest(mockRequest) - val factory = Mockito.mock() - BDDMockito.given(factory.createBinder(any(), any(), eq("param"))) + val factory = mock() + given(factory.createBinder(any(), any(), eq("param"))) .willAnswer { WebRequestDataBinder(it.getArgument(1)) } assertThat(processor.resolveArgument(this.param, container, requestWithParam, factory)).isEqualTo(Param("b")) } @@ -69,8 +69,8 @@ class ModelAttributeMethodProcessorKotlinTests { fun throwMethodArgumentNotValidExceptionWithNull() { val mockRequest = MockHttpServletRequest().apply { addParameter("a", null) } val requestWithParam = ServletWebRequest(mockRequest) - val factory = Mockito.mock() - BDDMockito.given(factory.createBinder(any(), any(), eq("param"))) + val factory = mock() + given(factory.createBinder(any(), any(), eq("param"))) .willAnswer { WebRequestDataBinder(it.getArgument(1)) } assertThatThrownBy { processor.resolveArgument(this.param, container, requestWithParam, factory) @@ -80,8 +80,8 @@ class ModelAttributeMethodProcessorKotlinTests { private data class Param(val a: String) - @Suppress("UNUSED_PARAMETER") private class ModelAttributeHandler { + @Suppress("UNUSED_PARAMETER") fun test(param: Param) { } } diff --git a/spring-web/src/test/kotlin/org/springframework/web/server/CoWebFilterTests.kt b/spring-web/src/test/kotlin/org/springframework/web/server/CoWebFilterTests.kt index af6b7dac81f..85c3b0c5d04 100644 --- a/spring-web/src/test/kotlin/org/springframework/web/server/CoWebFilterTests.kt +++ b/spring-web/src/test/kotlin/org/springframework/web/server/CoWebFilterTests.kt @@ -18,7 +18,7 @@ package org.springframework.web.server import org.assertj.core.api.Assertions.assertThat import org.junit.jupiter.api.Test -import org.mockito.BDDMockito +import org.mockito.BDDMockito.given import org.mockito.Mockito import org.springframework.web.testfixture.http.server.reactive.MockServerHttpRequest import org.springframework.web.testfixture.server.MockServerWebExchange @@ -35,7 +35,7 @@ class CoWebFilterTests { val exchange = MockServerWebExchange.from(MockServerHttpRequest.get("https://example.com")) val chain = Mockito.mock(WebFilterChain::class.java) - BDDMockito.given(chain.filter(exchange)).willReturn(Mono.empty()) + given(chain.filter(exchange)).willReturn(Mono.empty()) val filter = MyCoWebFilter() val result = filter.filter(exchange, chain)