Update database migrations and strengthen db smoke tests (maybe-finan…

…ce#462)
johnny-rice · Feb 15, 2024 · 093ddba · 093ddba
1 parent 3ec9c9b
commit 093ddba
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 3 deletions.
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -96,7 +96,7 @@ jobs:
         env:
           RAILS_ENV: test
           DATABASE_URL: postgres://postgres:postgres@localhost:5432
-        run: bin/rails db:reset
+        run: bin/rails db:drop db:create db:migrate db:seed
 
       - name: Keep screenshots from failed system tests
         uses: actions/upload-artifact@v4

diff --git a/db/migrate/20240206031739_replace_money_field.rb b/db/migrate/20240206031739_replace_money_field.rb
@@ -1,6 +1,6 @@
 class ReplaceMoneyField < ActiveRecord::Migration[7.2]
   def change
-    add_column :accounts, :balance_cents
+    add_column :accounts, :balance_cents, :integer
     change_column :accounts, :balance_cents, :integer, limit: 8
 
     Account.reset_column_information

diff --git a/db/migrate/20240209174912_redo_money_storage.rb b/db/migrate/20240209174912_redo_money_storage.rb
@@ -6,7 +6,6 @@ def change
     add_column :accounts, :converted_currency, :string, default: "USD"
 
     remove_column :accounts, :balance_cents
-    remove_column :accounts, :balance_currency
     remove_column :accounts, :currency
   end
 end