Fix direction choice while migration (#75)

abogoyavlensky · Jan 11, 2024 · 9a14843 · 9a14843
1 parent bc08db9
commit 9a14843
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/src/automigrate/migrations.clj b/src/automigrate/migrations.clj
@@ -941,9 +941,9 @@
           (get-detailed-migrations-to-migrate all-migrations-detailed migrated number)]
       (if (seq to-migrate)
         (doseq [{:keys [migration-name file-name migration-type number-int]} to-migrate]
-          (if (= direction FORWARD-DIRECTION)
-            (println (str "Applying " migration-name "..."))
-            (println (str "Reverting " migration-name "...")))
+          (condp = direction
+            FORWARD-DIRECTION (println (str "Applying " migration-name "..."))
+            BACKWARD-DIRECTION (println (str "Reverting " migration-name "...")))
           (jdbc/with-transaction [tx db]
             (let [actions (migration->actions {:file-name file-name
                                                :migrations-dir migrations-dir

diff --git a/test/automigrate/testing_util.clj b/test/automigrate/testing_util.clj
@@ -161,7 +161,8 @@
 (defn perform-make-and-migrate!
   [{:keys [jdbc-url existing-actions existing-models direction]
     :or {existing-actions []
-         existing-models {}}}]
+         existing-models {}
+         direction :forward}}]
   (bond/with-spy [migrations/make-next-migration]
     ; Generate new actions
     (make-migration! {:existing-models existing-models