-
Notifications
You must be signed in to change notification settings - Fork 8
[16.0][MIG] sync_multi_company_bank_account: Migration to 16.0 #148
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[16.0][MIG] sync_multi_company_bank_account: Migration to 16.0 #148
Conversation
Codecov Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## 16.0 #148 +/- ##
==========================================
+ Coverage 80.06% 80.11% +0.04%
==========================================
Files 52 59 +7
Lines 577 674 +97
Branches 55 75 +20
==========================================
+ Hits 462 540 +78
- Misses 101 115 +14
- Partials 14 19 +5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
kaynnan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Precisa alterar algumas coisas nos tests, coloquei as informações no modelo do Test, seguindo o guia de Migration to Version 16.0
8594e12 to
0517851
Compare
kaynnan
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Deprecation of SavepointCase in your tests:
- The test file sync_multi_company_bank_account/tests/test_res_partner_bank.py uses SavepointCase:
class TestResPartnerBank(SavepointCase):
The logs show:
DeprecationWarning: Deprecated class SavepointCase has been merged into TransactionCase
This causes errors during test setup:
ERROR: setUpClass (odoo.addons.sync_multi_company_bank_account.tests.test_res_partner_bank.TestResPartnerBank)`
Unknown field parameter inverse_name:
Warnings in the logs:
Field account.bank.statement.line.payment_ids: unknown parameter 'inverse_name', if this is an actual parameter you may want to override the method _valid_field_parameter on the relevant model in ord
This indicates an invalid field argument in your model definition.
0517851 to
34dfdec
Compare
34dfdec to
5584a3f
Compare
Field account.bank.statement.line.payment_ids: unknown parameter 'inverse_name', if this is an actual parameter you may want to override the method _valid_field_parameter on the relevant model in ord Essa parte aqui tem a vê com esse campo aqui https://github.com/Escodoo/account-addons/blob/16.0/account_bank_statement_internal_transfer/models/account_bank_statement.py#L10 |
cc @marcelsavegnago @WesleyOliveira98 @kaynnan