1
0
Fork 0
mirror of https://github.com/maybe-finance/maybe.git synced 2025-07-19 13:19:39 +02:00

fix migration and add smoke test for migrations (#438)

* adds a smoke test in the CI for the migration

* add back money rails gem

* really remove money-rails

* update the migration to remove add_monetize
This commit is contained in:
Adrien Poly 2024-02-11 19:34:18 +01:00 committed by GitHub
parent c483153721
commit d3151be9ae
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 8 additions and 2 deletions

View file

@ -3,7 +3,7 @@ name: CI
on: on:
pull_request: pull_request:
push: push:
branches: [ main ] branches: [main]
jobs: jobs:
scan_ruby: scan_ruby:
@ -92,6 +92,12 @@ jobs:
# REDIS_URL: redis://localhost:6379/0 # REDIS_URL: redis://localhost:6379/0
run: bin/rails db:setup test test:system run: bin/rails db:setup test test:system
- name: Smoke test database seeds
env:
RAILS_ENV: test
DATABASE_URL: postgres://postgres:postgres@localhost:5432
run: bin/rails db:reset
- name: Keep screenshots from failed system tests - name: Keep screenshots from failed system tests
uses: actions/upload-artifact@v4 uses: actions/upload-artifact@v4
if: failure() if: failure()

View file

@ -1,6 +1,6 @@
class ReplaceMoneyField < ActiveRecord::Migration[7.2] class ReplaceMoneyField < ActiveRecord::Migration[7.2]
def change def change
add_monetize :accounts, :balance add_column :accounts, :balance_cents
change_column :accounts, :balance_cents, :integer, limit: 8 change_column :accounts, :balance_cents, :integer, limit: 8
Account.reset_column_information Account.reset_column_information