mirror of
https://github.com/spring-projects/spring-boot.git
synced 2024-07-05 00:56:58 +08:00
Merge branch '3.1.x'
Closes gh-37662
This commit is contained in:
commit
7d40a45b50
@ -30,7 +30,7 @@ import org.springframework.context.annotation.Configuration;
|
||||
import org.springframework.context.annotation.Import;
|
||||
|
||||
import static org.assertj.core.api.Assertions.assertThat;
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
|
||||
/**
|
||||
* Tests for {@link ValidationFailureAnalyzer}.
|
||||
|
@ -43,7 +43,7 @@ import org.springframework.jmx.export.MBeanExporter;
|
||||
|
||||
import static org.assertj.core.api.Assertions.assertThat;
|
||||
import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
|
||||
/**
|
||||
* Tests for {@link SpringApplicationAdminJmxAutoConfiguration}.
|
||||
|
@ -37,7 +37,7 @@ import org.springframework.context.annotation.Configuration;
|
||||
import org.springframework.context.annotation.EnableMBeanExport;
|
||||
|
||||
import static org.assertj.core.api.Assertions.assertThat;
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
|
||||
/**
|
||||
* Tests for {@link TomcatDataSourceConfiguration}.
|
||||
|
@ -69,7 +69,7 @@ import org.springframework.web.socket.sockjs.client.Transport;
|
||||
import org.springframework.web.socket.sockjs.client.WebSocketTransport;
|
||||
|
||||
import static org.assertj.core.api.Assertions.assertThat;
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
|
||||
/**
|
||||
* Tests for {@link WebSocketMessagingAutoConfiguration}.
|
||||
|
@ -21,7 +21,7 @@ import java.util.concurrent.CountDownLatch;
|
||||
import org.junit.jupiter.api.Test;
|
||||
|
||||
import static org.assertj.core.api.Assertions.assertThat;
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
|
||||
/**
|
||||
* Tests for {@link SilentExitExceptionHandler}.
|
||||
|
@ -38,7 +38,7 @@ import org.springframework.core.io.Resource;
|
||||
import org.springframework.util.FileCopyUtils;
|
||||
import org.springframework.util.function.ThrowingSupplier;
|
||||
|
||||
import static org.junit.Assert.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
|
||||
/**
|
||||
* Abstract base class for integration tests.
|
||||
|
@ -27,7 +27,7 @@ import org.springframework.core.test.tools.SourceFile;
|
||||
import org.springframework.core.test.tools.TestCompiler;
|
||||
|
||||
import static org.assertj.core.api.Assertions.assertThat;
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
|
||||
/**
|
||||
* Tests for {@link AutoConfigureAnnotationProcessor}.
|
||||
|
@ -34,7 +34,7 @@ import org.mockito.junit.jupiter.MockitoExtension;
|
||||
import org.springframework.boot.cli.command.status.ExitStatus;
|
||||
|
||||
import static org.assertj.core.api.Assertions.assertThat;
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
import static org.mockito.ArgumentMatchers.any;
|
||||
import static org.mockito.ArgumentMatchers.assertArg;
|
||||
import static org.mockito.ArgumentMatchers.isNull;
|
||||
@ -192,7 +192,7 @@ class InitCommandTests extends AbstractHttpClientMockTests {
|
||||
}
|
||||
|
||||
@Test
|
||||
void generateProjectAndExtractUnknownContentType(@TempDir File tempDir) throws Exception {
|
||||
void generateProjectAndExtractUnknownContentType(@TempDir File tempDir) {
|
||||
String fileName = UUID.randomUUID().toString() + ".zip";
|
||||
File file = new File(fileName);
|
||||
assertThat(file).as("file should not exist").doesNotExist();
|
||||
@ -204,7 +204,7 @@ class InitCommandTests extends AbstractHttpClientMockTests {
|
||||
assertThat(file).as("file should have been saved instead").exists();
|
||||
}
|
||||
catch (Exception ex) {
|
||||
fail(ex);
|
||||
fail(null, ex);
|
||||
}
|
||||
finally {
|
||||
assertThat(file.delete()).as("failed to delete test file").isTrue();
|
||||
|
@ -61,7 +61,7 @@ import org.springframework.util.FileCopyUtils;
|
||||
import org.springframework.util.FileSystemUtils;
|
||||
|
||||
import static org.assertj.core.api.Assertions.assertThat;
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
|
||||
/**
|
||||
* A {@code GradleBuild} is used to run a Gradle build using {@link GradleRunner}.
|
||||
|
@ -18,7 +18,7 @@ package org.springframework.boot.testsupport.process;
|
||||
|
||||
import org.junit.jupiter.api.Test;
|
||||
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
|
||||
/**
|
||||
* Tests for {@link DisabledIfProcessUnavailable}.
|
||||
|
@ -18,11 +18,11 @@ package org.springframework.boot.context.properties;
|
||||
|
||||
import java.util.function.Supplier;
|
||||
|
||||
import org.junit.jupiter.api.Assertions;
|
||||
import org.junit.jupiter.api.Test;
|
||||
|
||||
import static org.assertj.core.api.Assertions.assertThat;
|
||||
import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
|
||||
/**
|
||||
* Tests for {@link PropertyMapper}.
|
||||
@ -57,7 +57,7 @@ class PropertyMapperTests {
|
||||
|
||||
@Test
|
||||
void fromValueAlwaysApplyingWhenNonNullShouldAlwaysApplyNonNullToSource() {
|
||||
this.map.alwaysApplyingWhenNonNull().from((String) null).toCall(Assertions::fail);
|
||||
this.map.alwaysApplyingWhenNonNull().from((String) null).toCall(() -> fail(null));
|
||||
}
|
||||
|
||||
@Test
|
||||
@ -101,14 +101,14 @@ class PropertyMapperTests {
|
||||
|
||||
@Test
|
||||
void whenNonNullWhenSuppliedNullShouldNotMap() {
|
||||
this.map.from(() -> null).whenNonNull().as(String::valueOf).toCall(Assertions::fail);
|
||||
this.map.from(() -> null).whenNonNull().as(String::valueOf).toCall(() -> fail(null));
|
||||
}
|
||||
|
||||
@Test
|
||||
void whenNonNullWhenSuppliedThrowsNullPointerExceptionShouldNotMap() {
|
||||
this.map.from(() -> {
|
||||
throw new NullPointerException();
|
||||
}).whenNonNull().as(String::valueOf).toCall(Assertions::fail);
|
||||
}).whenNonNull().as(String::valueOf).toCall(() -> fail(null));
|
||||
}
|
||||
|
||||
@Test
|
||||
@ -119,7 +119,7 @@ class PropertyMapperTests {
|
||||
|
||||
@Test
|
||||
void whenTrueWhenValueIsFalseShouldNotMap() {
|
||||
this.map.from(false).whenTrue().toCall(Assertions::fail);
|
||||
this.map.from(false).whenTrue().toCall(() -> fail(null));
|
||||
}
|
||||
|
||||
@Test
|
||||
@ -130,17 +130,17 @@ class PropertyMapperTests {
|
||||
|
||||
@Test
|
||||
void whenFalseWhenValueIsTrueShouldNotMap() {
|
||||
this.map.from(true).whenFalse().toCall(Assertions::fail);
|
||||
this.map.from(true).whenFalse().toCall(() -> fail(null));
|
||||
}
|
||||
|
||||
@Test
|
||||
void whenHasTextWhenValueIsNullShouldNotMap() {
|
||||
this.map.from(() -> null).whenHasText().toCall(Assertions::fail);
|
||||
this.map.from(() -> null).whenHasText().toCall(() -> fail(null));
|
||||
}
|
||||
|
||||
@Test
|
||||
void whenHasTextWhenValueIsEmptyShouldNotMap() {
|
||||
this.map.from("").whenHasText().toCall(Assertions::fail);
|
||||
this.map.from("").whenHasText().toCall(() -> fail(null));
|
||||
}
|
||||
|
||||
@Test
|
||||
@ -157,7 +157,7 @@ class PropertyMapperTests {
|
||||
|
||||
@Test
|
||||
void whenEqualToWhenValueIsNotEqualShouldNotMatch() {
|
||||
this.map.from("123").whenEqualTo("321").toCall(Assertions::fail);
|
||||
this.map.from("123").whenEqualTo("321").toCall(() -> fail(null));
|
||||
}
|
||||
|
||||
@Test
|
||||
@ -169,7 +169,7 @@ class PropertyMapperTests {
|
||||
@Test
|
||||
void whenInstanceOfWhenValueIsNotTargetTypeShouldNotMatch() {
|
||||
Supplier<Number> supplier = () -> 123L;
|
||||
this.map.from(supplier).whenInstanceOf(Double.class).toCall(Assertions::fail);
|
||||
this.map.from(supplier).whenInstanceOf(Double.class).toCall(() -> fail(null));
|
||||
}
|
||||
|
||||
@Test
|
||||
@ -180,7 +180,7 @@ class PropertyMapperTests {
|
||||
|
||||
@Test
|
||||
void whenWhenValueDoesNotMatchShouldNotMap() {
|
||||
this.map.from("123").when("321"::equals).toCall(Assertions::fail);
|
||||
this.map.from("123").when("321"::equals).toCall(() -> fail(null));
|
||||
}
|
||||
|
||||
@Test
|
||||
@ -198,12 +198,12 @@ class PropertyMapperTests {
|
||||
|
||||
@Test
|
||||
void alwaysApplyingWhenNonNullShouldAlwaysApplyNonNullToSource() {
|
||||
this.map.alwaysApplyingWhenNonNull().from(() -> null).toCall(Assertions::fail);
|
||||
this.map.alwaysApplyingWhenNonNull().from(() -> null).toCall(() -> fail(null));
|
||||
}
|
||||
|
||||
@Test
|
||||
void whenWhenValueNotMatchesShouldSupportChainedCalls() {
|
||||
this.map.from("123").when("456"::equals).when("123"::equals).toCall(Assertions::fail);
|
||||
this.map.from("123").when("456"::equals).when("123"::equals).toCall(() -> fail(null));
|
||||
}
|
||||
|
||||
@Test
|
||||
|
@ -40,7 +40,7 @@ import org.springframework.util.Assert;
|
||||
|
||||
import static org.assertj.core.api.Assertions.assertThat;
|
||||
import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
|
||||
/**
|
||||
* Tests for {@link ValueObjectBinder}.
|
||||
|
@ -36,7 +36,7 @@ import org.springframework.context.annotation.Bean;
|
||||
import org.springframework.context.annotation.Configuration;
|
||||
|
||||
import static org.assertj.core.api.Assertions.assertThat;
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
|
||||
/**
|
||||
* Tests for {@link BeanCurrentlyInCreationFailureAnalyzer}.
|
||||
|
@ -29,7 +29,7 @@ import org.springframework.scheduling.annotation.Async;
|
||||
import org.springframework.scheduling.annotation.EnableAsync;
|
||||
|
||||
import static org.assertj.core.api.Assertions.assertThat;
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
|
||||
/**
|
||||
* Tests for {@link BeanNotOfRequiredTypeFailureAnalyzer}.
|
||||
|
@ -29,7 +29,7 @@ import org.springframework.boot.util.LambdaSafe.InvocationResult;
|
||||
|
||||
import static org.assertj.core.api.Assertions.assertThat;
|
||||
import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
import static org.mockito.ArgumentMatchers.any;
|
||||
import static org.mockito.ArgumentMatchers.contains;
|
||||
import static org.mockito.BDDMockito.given;
|
||||
|
@ -160,7 +160,7 @@ import static org.assertj.core.api.Assertions.assertThatIOException;
|
||||
import static org.assertj.core.api.Assertions.assertThatIllegalArgumentException;
|
||||
import static org.assertj.core.api.Assertions.assertThatIllegalStateException;
|
||||
import static org.assertj.core.api.Assertions.assertThatThrownBy;
|
||||
import static org.junit.jupiter.api.Assertions.fail;
|
||||
import static org.assertj.core.api.Assertions.fail;
|
||||
import static org.mockito.ArgumentMatchers.any;
|
||||
import static org.mockito.BDDMockito.given;
|
||||
import static org.mockito.BDDMockito.then;
|
||||
|
@ -33,7 +33,7 @@
|
||||
</module>
|
||||
<module name="com.puppycrawl.tools.checkstyle.checks.regexp.RegexpSinglelineJavaCheck">
|
||||
<property name="maximum" value="0"/>
|
||||
<property name="format" value="org\.junit\.Assert\.assert" />
|
||||
<property name="format" value="org\.junit\.Assert|org\.junit\.jupiter\.api\.Assertions" />
|
||||
<property name="message"
|
||||
value="Please use AssertJ imports." />
|
||||
<property name="ignoreComments" value="true" />
|
||||
|
Loading…
Reference in New Issue
Block a user