Commit 5dc2339e authored by Andy Wilkinson's avatar Andy Wilkinson

Polish "Consider properties from @AutoConfigureTestDatabase"

See gh-16814
parent 92256c80
...@@ -30,7 +30,6 @@ import org.flywaydb.core.api.callback.Callback; ...@@ -30,7 +30,6 @@ import org.flywaydb.core.api.callback.Callback;
import org.flywaydb.core.api.callback.Context; import org.flywaydb.core.api.callback.Context;
import org.flywaydb.core.api.callback.Event; import org.flywaydb.core.api.callback.Event;
import org.flywaydb.core.api.callback.FlywayCallback; import org.flywaydb.core.api.callback.FlywayCallback;
import org.flywaydb.core.internal.jdbc.DriverDataSource;
import org.hibernate.engine.transaction.jta.platform.internal.NoJtaPlatform; import org.hibernate.engine.transaction.jta.platform.internal.NoJtaPlatform;
import org.junit.Test; import org.junit.Test;
import org.mockito.InOrder; import org.mockito.InOrder;
...@@ -105,11 +104,10 @@ public class FlywayAutoConfigurationTests { ...@@ -105,11 +104,10 @@ public class FlywayAutoConfigurationTests {
this.contextRunner.withUserConfiguration(EmbeddedDataSourceConfiguration.class) this.contextRunner.withUserConfiguration(EmbeddedDataSourceConfiguration.class)
.withPropertyValues("spring.flyway.url:jdbc:hsqldb:mem:flywaytest").run((context) -> { .withPropertyValues("spring.flyway.url:jdbc:hsqldb:mem:flywaytest").run((context) -> {
assertThat(context).hasSingleBean(Flyway.class); assertThat(context).hasSingleBean(Flyway.class);
assertThat(context.getBean(Flyway.class).getDataSource()).isNotNull(); DataSource dataSource = context.getBean(Flyway.class).getDataSource();
assertThat(((DriverDataSource) context.getBean(Flyway.class).getDataSource()).getUser()) assertThat(dataSource).isNotNull();
.isEqualTo("sa"); assertThat(dataSource).hasFieldOrPropertyWithValue("user", "sa");
assertThat(((DriverDataSource) context.getBean(Flyway.class).getDataSource()).getPassword()) assertThat(dataSource).hasFieldOrPropertyWithValue("password", "");
.isEqualTo("");
}); });
} }
...@@ -118,9 +116,9 @@ public class FlywayAutoConfigurationTests { ...@@ -118,9 +116,9 @@ public class FlywayAutoConfigurationTests {
this.contextRunner.withUserConfiguration(EmbeddedDataSourceConfiguration.class) this.contextRunner.withUserConfiguration(EmbeddedDataSourceConfiguration.class)
.withPropertyValues("spring.flyway.user:sa").run((context) -> { .withPropertyValues("spring.flyway.user:sa").run((context) -> {
assertThat(context).hasSingleBean(Flyway.class); assertThat(context).hasSingleBean(Flyway.class);
assertThat(context.getBean(Flyway.class).getDataSource()).isNotNull(); DataSource dataSource = context.getBean(Flyway.class).getDataSource();
assertThat(((DriverDataSource) context.getBean(Flyway.class).getDataSource()).getUrl()) assertThat(dataSource).isNotNull();
.startsWith("jdbc:h2:mem:"); assertThat(dataSource).extracting("url").hasSize(1).first().asString().startsWith("jdbc:h2:mem:");
}); });
} }
......
...@@ -35,5 +35,4 @@ ...@@ -35,5 +35,4 @@
<suppress files="LogbackMetricsAutoConfiguration\.java" checks="IllegalImport" /> <suppress files="LogbackMetricsAutoConfiguration\.java" checks="IllegalImport" />
<suppress files="RemoteUrlPropertyExtractorTests\.java" checks="IllegalImport" /> <suppress files="RemoteUrlPropertyExtractorTests\.java" checks="IllegalImport" />
<suppress files="SampleLogbackApplication\.java" checks="IllegalImport" /> <suppress files="SampleLogbackApplication\.java" checks="IllegalImport" />
<suppress files="FlywayAutoConfigurationTests\.java" checks="IllegalImport" />
</suppressions> </suppressions>
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