Commit 2682b380 authored by Stephane Nicoll's avatar Stephane Nicoll

Merge pull request #25499 from dreis2211

* pr/25499:
  Fix some assertions

Closes gh-25499
parents ae630e03 89a44f3d
/* /*
* Copyright 2012-2020 the original author or authors. * Copyright 2012-2021 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
...@@ -284,7 +284,7 @@ class Neo4jAutoConfigurationTests { ...@@ -284,7 +284,7 @@ class Neo4jAutoConfigurationTests {
@Test @Test
void securityWithCustomCertificates(@TempDir File directory) throws IOException { void securityWithCustomCertificates(@TempDir File directory) throws IOException {
File certFile = new File(directory, "neo4j-driver.cert"); File certFile = new File(directory, "neo4j-driver.cert");
assertThat(certFile.createNewFile()); assertThat(certFile.createNewFile()).isTrue();
Neo4jProperties properties = new Neo4jProperties(); Neo4jProperties properties = new Neo4jProperties();
properties.getSecurity().setTrustStrategy(TrustStrategy.TRUST_CUSTOM_CA_SIGNED_CERTIFICATES); properties.getSecurity().setTrustStrategy(TrustStrategy.TRUST_CUSTOM_CA_SIGNED_CERTIFICATES);
......
/* /*
* Copyright 2012-2020 the original author or authors. * Copyright 2012-2021 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
...@@ -80,7 +80,7 @@ class BuildInfoIntegrationTests { ...@@ -80,7 +80,7 @@ class BuildInfoIntegrationTests {
Properties second = buildInfoProperties(); Properties second = buildInfoProperties();
String secondBuildTime = second.getProperty("build.time"); String secondBuildTime = second.getProperty("build.time");
assertThat(secondBuildTime).isNotNull(); assertThat(secondBuildTime).isNotNull();
assertThat(Instant.parse(firstBuildTime).isBefore(Instant.parse(secondBuildTime))); assertThat(Instant.parse(firstBuildTime)).isBefore(Instant.parse(secondBuildTime));
} }
@TestTemplate @TestTemplate
......
/* /*
* Copyright 2012-2020 the original author or authors. * Copyright 2012-2021 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
...@@ -61,7 +61,7 @@ public class BootBuildImageRegistryIntegrationTests { ...@@ -61,7 +61,7 @@ public class BootBuildImageRegistryIntegrationTests {
@BeforeEach @BeforeEach
void setUp() { void setUp() {
assertThat(registry.isRunning()); assertThat(registry.isRunning()).isTrue();
this.registryAddress = registry.getHost() + ":" + registry.getFirstMappedPort(); this.registryAddress = registry.getHost() + ":" + registry.getFirstMappedPort();
} }
......
...@@ -64,7 +64,7 @@ abstract class AbstractArchiveIntegrationTests { ...@@ -64,7 +64,7 @@ abstract class AbstractArchiveIntegrationTests {
private JarAssert(File actual) { private JarAssert(File actual) {
super(actual, JarAssert.class); super(actual, JarAssert.class);
assertThat(actual.exists()); assertThat(actual).exists();
} }
JarAssert doesNotHaveEntryWithName(String name) { JarAssert doesNotHaveEntryWithName(String name) {
......
/* /*
* Copyright 2012-2020 the original author or authors. * Copyright 2012-2021 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
...@@ -55,7 +55,7 @@ public class BuildImageRegistryIntegrationTests extends AbstractArchiveIntegrati ...@@ -55,7 +55,7 @@ public class BuildImageRegistryIntegrationTests extends AbstractArchiveIntegrati
@BeforeEach @BeforeEach
void setUp() { void setUp() {
assertThat(registry.isRunning()); assertThat(registry.isRunning()).isTrue();
this.dockerClient = registry.getDockerClient(); this.dockerClient = registry.getDockerClient();
this.registryAddress = registry.getHost() + ":" + registry.getFirstMappedPort(); this.registryAddress = registry.getHost() + ":" + registry.getFirstMappedPort();
} }
......
/* /*
* Copyright 2012-2020 the original author or authors. * Copyright 2012-2021 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
...@@ -1228,10 +1228,10 @@ class SpringApplicationTests { ...@@ -1228,10 +1228,10 @@ class SpringApplicationTests {
application.addListeners(listener); application.addListeners(listener);
application.run(); application.run();
ApplicationStartingEvent startingEvent = listener.getEvent(ApplicationStartingEvent.class); ApplicationStartingEvent startingEvent = listener.getEvent(ApplicationStartingEvent.class);
assertThat(startingEvent.getBootstrapContext().get(String.class)); assertThat(startingEvent.getBootstrapContext().get(String.class)).isEqualTo("boot");
ApplicationEnvironmentPreparedEvent environmentPreparedEvent = listener ApplicationEnvironmentPreparedEvent environmentPreparedEvent = listener
.getEvent(ApplicationEnvironmentPreparedEvent.class); .getEvent(ApplicationEnvironmentPreparedEvent.class);
assertThat(environmentPreparedEvent.getBootstrapContext().get(String.class)); assertThat(environmentPreparedEvent.getBootstrapContext().get(String.class)).isEqualTo("boot");
} }
@Test @Test
......
/* /*
* Copyright 2012-2020 the original author or authors. * Copyright 2012-2021 the original author or authors.
* *
* Licensed under the Apache License, Version 2.0 (the "License"); * Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License. * you may not use this file except in compliance with the License.
...@@ -54,7 +54,6 @@ import org.springframework.messaging.rsocket.RSocketStrategies; ...@@ -54,7 +54,6 @@ import org.springframework.messaging.rsocket.RSocketStrategies;
import org.springframework.util.SocketUtils; import org.springframework.util.SocketUtils;
import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThat;
import static org.assertj.core.api.Assertions.assertThatExceptionOfType;
import static org.mockito.ArgumentMatchers.any; import static org.mockito.ArgumentMatchers.any;
import static org.mockito.BDDMockito.will; import static org.mockito.BDDMockito.will;
import static org.mockito.Mockito.inOrder; import static org.mockito.Mockito.inOrder;
...@@ -203,7 +202,7 @@ class NettyRSocketServerFactoryTests { ...@@ -203,7 +202,7 @@ class NettyRSocketServerFactoryTests {
String payload = "test payload"; String payload = "test payload";
Mono<String> responseMono = this.requester.route("test").data(payload).retrieveMono(String.class); Mono<String> responseMono = this.requester.route("test").data(payload).retrieveMono(String.class);
StepVerifier.create(responseMono) StepVerifier.create(responseMono)
.verifyErrorSatisfies((ex) -> assertThatExceptionOfType(ClosedChannelException.class)); .verifyErrorSatisfies((ex) -> assertThat(ex).isInstanceOf(ClosedChannelException.class));
} }
private RSocketRequester createRSocketTcpClient() { private RSocketRequester createRSocketTcpClient() {
......
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