/*
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 * http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package com.opentable.db.postgres.embedded;

import java.sql.SQLException;
import java.util.Arrays;
import java.util.List;
import java.util.Objects;

import javax.sql.DataSource;

import org.flywaydb.core.Flyway;

// TODO: Detect missing migration files.
// cf. https://github.com/flyway/flyway/issues/1496
// There is also a related @Ignored test in otj-sql.

public final class FlywayPreparer implements DatabasePreparer {

    private final List<String> locations;

    public static FlywayPreparer forClasspathLocation(String... locations) {
        return new FlywayPreparer(Arrays.asList(locations));
    }

    private FlywayPreparer(List<String> locations) {
        this.locations = locations;
    }

    @Override
    public void prepare(DataSource ds) throws SQLException {
        Flyway.configure()
                .locations(locations.toArray(new String[0]))
                .dataSource(ds)
                .load()
                .migrate();
    }

    @Override
    public boolean equals(Object obj) {
        if (! (obj instanceof FlywayPreparer)) {
            return false;
        }
        return Objects.equals(locations, ((FlywayPreparer) obj).locations);
    }

    @Override
    public int hashCode() {
        return Objects.hashCode(locations);
    }
}

@@ -911,16 +911,141 @@
 tj-sql.%0A
+// MJB: This is finally fixed in Flyway 8.41  onwards%0A// failOnMissingLocations = true, not willing to force that update yet.
 %0Apublic 

Add note about flyway 8.41