#3 remove the restriction that flyway schema must always be set

Author: tomix26

README.md

@@ -4,8 +4,10 @@ The primary goal of this project is to make it easier to write Spring-powered in
 
 ## Features
 
-* Integration with Spring TestContext Framework
-* Both `Spring` and `Spring Boot` Frameworks are supported
+* Integration with Spring TestContext framework
+    * Context caching is fully supported
+* Both `Spring` and `Spring Boot` frameworks are supported
+    * Supported versions are Spring 4.3.0+ and Spring Boot 1.4.0+
 * Lightweight bundle of PostgreSQL database with reduced size
 * Integration with Flyway database migration tool
 * Optimized migration and cleaning of embedded database

embedded-database-spring-test-core/src/main/java/io/zonky/test/db/flyway/DefaultFlywayDataSourceContext.java

@@ -21,7 +21,6 @@
 import com.google.common.cache.LoadingCache;
 import com.opentable.db.postgres.embedded.DatabasePreparer;
 import com.opentable.db.postgres.embedded.PreparedDbProvider;
-import org.apache.commons.lang3.ArrayUtils;
 import org.apache.commons.lang3.exception.ExceptionUtils;
 import org.flywaydb.core.Flyway;
 import org.postgresql.ds.PGSimpleDataSource;
@@ -159,10 +158,7 @@ public void setBootstrapExecutor(TaskExecutor bootstrapExecutor) {
     }
 
     protected FlywayConfigSnapshot createConfigSnapshot(Flyway flyway) {
-        FlywayConfigSnapshot configSnapshot = new FlywayConfigSnapshot(flyway);
-        checkState(ArrayUtils.isNotEmpty(configSnapshot.getSchemas()),
-                "org.flywaydb.core.Flyway#schemaNames must be specified");
-        return configSnapshot;
+        return new FlywayConfigSnapshot(flyway);
     }
 
     protected class FlywayDatabasePreparer implements DatabasePreparer {

embedded-database-spring-test-core/src/main/java/io/zonky/test/db/flyway/OptimizedFlywayTestExecutionListener.java

@@ -20,6 +20,7 @@
 import com.google.common.collect.ObjectArrays;
 import org.apache.commons.lang3.ArrayUtils;
 import org.flywaydb.core.Flyway;
+import org.flywaydb.core.api.FlywayException;
 import org.flywaydb.core.api.MigrationVersion;
 import org.flywaydb.core.api.resolver.MigrationResolver;
 import org.flywaydb.core.api.resolver.ResolvedMigration;
@@ -40,6 +41,7 @@
 import java.lang.annotation.Annotation;
 import java.lang.reflect.AnnotatedElement;
 import java.lang.reflect.Method;
+import java.sql.SQLException;
 import java.util.Collection;
 import java.util.Map;
 
@@ -145,7 +147,17 @@ protected synchronized void optimizedDbReset(TestContext testContext, FlywayTest
             }
         }
 
-        ReflectionTestUtils.invokeMethod(this, dbResetMethodName, testContext, annotation);
+        try {
+            ReflectionTestUtils.invokeMethod(this, dbResetMethodName, testContext, annotation);
+        } catch (FlywayException e) {
+            if (e.getCause() instanceof SQLException) {
+                String errorCode = ((SQLException) e.getCause()).getSQLState();
+                if (errorCode != null && errorCode.matches("(42723|42P06|42P07|42712|42710)")) {
+                    logger.error("HINT: Check that you have correctly set org.flywaydb.core.Flyway#schemaNames property!!!");
+                }
+            }
+            throw e;
+        }
     }
 
     protected static void prepareDataSourceContext(FlywayDataSourceContext dataSourceContext, Flyway flywayBean, FlywayTest annotation) throws Exception {