Closed lluchez closed 6 years ago
Merging #36 into master will decrease coverage by
1.9%
. The diff coverage is100%
.
@@ Coverage Diff @@
## master #36 +/- ##
==========================================
- Coverage 100% 98.09% -1.91%
==========================================
Files 28 31 +3
Lines 661 681 +20
==========================================
+ Hits 661 668 +7
- Misses 0 13 +13
Impacted Files | Coverage Δ | |
---|---|---|
app/models/bank_account.rb | 100% <100%> (ø) |
:arrow_up: |
app/models/user.rb | 100% <100%> (ø) |
:arrow_up: |
app/models/bank_statement.rb | 100% <100%> (ø) |
:arrow_up: |
app/models/statement_record.rb | 100% <100%> (ø) |
:arrow_up: |
app/models/statement_record_category.rb | 100% <100%> (ø) |
:arrow_up: |
...tement_record_category_rules/category_rule_base.rb | 100% <100%> (ø) |
:arrow_up: |
app/mailers/application_mailer.rb | 0% <0%> (ø) |
|
app/controllers/errors_controller.rb | 100% <0%> (ø) |
|
app/controllers/registrations_controller.rb | 0% <0%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 82f00b3...5d695d1. Read the comment docs.
Closes https://github.com/lluchez/my-ledger/issues/34
Changes due to the version change:
be_success?
deprecated. Replaced bybe_successful?
_was
andchanged?
aren't properly working in theafter_save
anymore. Needs to use ActiveModel::Dirty model insteadid: :integer
added to thecreate_table
commands to avoid this issue to happen.Resources used: