Add comments to unit tests with fixed flyway migrations

Signed-off-by: Michael Collado <[email protected]>
MarquezProject · Jul 18, 2022 · 2eb2bf1 · 2eb2bf1
1 parent 8f30da9
commit 2eb2bf1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/api/src/test/java/marquez/db/migrations/V44_2__BackfillAirflowParentRunsTest.java b/api/src/test/java/marquez/db/migrations/V44_2__BackfillAirflowParentRunsTest.java
@@ -31,6 +31,8 @@
 import org.junit.jupiter.api.extension.ExtendWith;
 
 @ExtendWith(MarquezJdbiExternalPostgresExtension.class)
+// fix the flyway migration up to v44 since we depend on the database structure as it exists at this
+// point in time. The migration will only ever be applied on a database at this version.
 @FlywayTarget("44")
 class V44_2__BackfillAirflowParentRunsTest {
 

diff --git a/api/src/test/java/marquez/db/migrations/V44_3_BackfillJobsWithParentsTest.java b/api/src/test/java/marquez/db/migrations/V44_3_BackfillJobsWithParentsTest.java
@@ -29,7 +29,9 @@
 import org.junit.jupiter.api.extension.ExtendWith;
 
 @ExtendWith(MarquezJdbiExternalPostgresExtension.class)
-@FlywayTarget("44") // apply migrations up to and including v44 but not beyond
+// fix the flyway migration up to v44 since we depend on the database structure as it exists at this
+// point in time. The migration will only ever be applied on a database at this version.
+@FlywayTarget("44")
 class V44_3_BackfillJobsWithParentsTest {
 
   static Jdbi jdbi;