Fix Allow ignoreMigrationPatterns to be empty

Removes the `.whenNot(List::isEmpty)` condition to allow clearing the
ignore-migration-patterns property via an empty string.

To avoid affecting existing users, the property's default is now
aligned with Flyway's default of `*:future`

Signed-off-by: Chanwon-Seo <[email protected]>

Author: Chanwon-Seo

module/spring-boot-flyway/src/main/java/org/springframework/boot/flyway/autoconfigure/FlywayProperties.java

@@ -323,7 +323,7 @@ public class FlywayProperties {
 	/**
 	 * List of patterns that identify migrations to ignore when performing validation.
 	 */
-	private @Nullable List<String> ignoreMigrationPatterns;
+	private List<String> ignoreMigrationPatterns = new ArrayList<>(Collections.singletonList("*:future"));
 
 	/**
 	 * Whether to attempt to automatically detect SQL migration file encoding.
@@ -757,11 +757,11 @@ public void setSkipExecutingMigrations(@Nullable Boolean skipExecutingMigrations
 		this.skipExecutingMigrations = skipExecutingMigrations;
 	}
 
-	public @Nullable List<String> getIgnoreMigrationPatterns() {
+	public List<String> getIgnoreMigrationPatterns() {
 		return this.ignoreMigrationPatterns;
 	}
 
-	public void setIgnoreMigrationPatterns(@Nullable List<String> ignoreMigrationPatterns) {
+	public void setIgnoreMigrationPatterns(List<String> ignoreMigrationPatterns) {
 		this.ignoreMigrationPatterns = ignoreMigrationPatterns;
 	}
 

module/spring-boot-flyway/src/test/java/org/springframework/boot/flyway/autoconfigure/FlywayAutoConfigurationTests.java

@@ -898,11 +898,10 @@ void ignoreMigrationPatternsCorrectlyMapped() {
 	}
 
 	@Test
-	void ignoreMigrationPatternsIsEmpty() {
+	void ignoreMigrationPatternsIsFutureByDefault() {
 		this.contextRunner.withUserConfiguration(EmbeddedDataSourceConfiguration.class)
-			.withPropertyValues("spring.flyway.ignore-migration-patterns=")
 			.run((context) -> assertThat(context.getBean(Flyway.class).getConfiguration().getIgnoreMigrationPatterns())
-				.isEmpty());
+				.containsExactly(ValidatePattern.fromPattern("*:future")));
 	}
 
 	private ContextConsumer<AssertableApplicationContext> validateFlywayTeamsPropertyOnly(String propertyName) {