[17.0][MIG] l10n_us_gaap #132
Conversation
[IMP] adding first unit tests
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: l10n-usa-16.0/l10n-usa-16.0-l10n_us_gaap Translate-URL: https://translation.odoo-community.org/projects/l10n-usa-16-0/l10n-usa-16-0-l10n_us_gaap/
l10n_us_gaap/tests/test_base_us.py
Outdated
There was a problem hiding this comment.
Tests deleted during migration?
There was a problem hiding this comment.
Yes, because it will try to load the module again and will try to create a new account and can cause issue of duplicate account
There was a problem hiding this comment.
I think you need to specify the companies to apply and template while loading the coa, something like:
try_loading(template_code="us_gaap", company=self.company, install_demo=False)
l10n_us_gaap/tests/test_base_us.py
Outdated
| self.env.user.write( | ||
| {"company_ids": [(4, self.company.id)], "company_id": self.company.id} | ||
| ) | ||
| chart_us.try_loading() |
JasminSForgeFlow
force-pushed
the
17.0-mig-l10n_us_gaap
branch
from
f23b798
to
7e6d862
Compare
|
/ocabot migration l10n_us_gaap |
l10n_us_gaap/readme/USAGE.md
Outdated
|
|
||
| This module contains in the 'docs' folder a a sample Trial Balance | ||
| generated using the [OCA Account Financial Report | ||
| module](https://github.com/OCA/account-financial-reporting/blob/12.0/account_financial_report). |
There was a problem hiding this comment.
It would have been really good if this file had been generated recently.
JasminSForgeFlow
force-pushed
the
17.0-mig-l10n_us_gaap
branch
from
7e6d862
to
e0b8ce2
Compare
|
This PR has the |
Standard Migration
Adapt 17.0 chart of account related changes, and remove tests as it will try to load the module again and will try to create a new account and can cause issue of duplicate account
@ForgeFlow