search

OCA / l10n-brazil

Localização brasileira oficial do Odoo.
https://odoo-community.org/psc-teams/brazil-66
GNU Affero General Public License v3.0
236 stars 244 forks source link

[16.0][REF][l10n_br_nfe_spec] monkey patch is now unused #3240

Open rvalyi opened 1 month ago

rvalyi commented 1 month ago

olhando pro coverage, parece que os monkey patches que eu tive que fazer para migrar nao estao mais sendo usados agora que eu simplifiquei o modelo de herança dos mixins spec: https://app.codecov.io/gh/OCA/l10n-brazil/blob/16.0/l10n_br_nfe_spec%2Ftests%2Ftest_nfe_import.py https://github.com/OCA/l10n-brazil/pull/2880

rvalyi commented 1 month ago

hum vou olhar melhor, parece que ainda usa sim...

   File "/__w/l10n-brazil/l10n-brazil/l10n_br_nfe_spec/tests/test_nfe_import.py", line 111, in match_or_create_m2o_fake
    return comodel.new(new_value)._ids[0]
  File "/opt/odoo/odoo/models.py", line 5757, in new
    record._update_cache(values, validate=False)
  File "/opt/odoo/odoo/models.py", line 5331, in _update_cache
    value = field.convert_to_cache(value, self, validate)
  File "/opt/odoo/odoo/fields.py", line 4132, in convert_to_cache
    if record and not record.id:
AttributeError: 'nfe.40.ide' object has no attribute 'id'