Skip to content
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

[FIX] tests data translation source strings #678

Conversation

primes2h
Copy link
Contributor

Some strings depend on l10n_it_fatturapa/demo/account_invoice_fatturapa.xml fields

Some strings depend on l10n_it_fatturapa/demo/account_invoice_fatturapa.xml fields
Copy link
Member

@eLBati eLBati left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@primes2h Questa fallisce perchè nei moduli rilasciati è ancora presente Pubblica Amministrazione.
Dovresti includere questo commit nella #677

Se vuoi provare i test in locale devi creare un DB demo e aggiornare il modulo da testare passando al server --test-enable.
Puoi guardare come fa travis
https://travis-ci.org/OCA/l10n-italy/jobs/460172919#L1396
anche se in questo caso ti è sufficiente eseguire i test solo di l10n_it_fatturapa_out

@primes2h
Copy link
Contributor Author

@primes2h Questa fallisce perchè nei moduli rilasciati è ancora presente Pubblica Amministrazione.
Dovresti includere questo commit nella #677

In effetti non avevo considerato il fatto che la dipendenza tra i due moduli è reciproca, quindi è necessario inserire tutte le modifiche in un'unica PR.
Metterò nella #677 anche le modifiche per l10n_it_fatturapa_in.

Questa può essere chiusa direi.

@eLBati
Copy link
Member

eLBati commented Nov 27, 2018

Grazie

@eLBati eLBati closed this Nov 27, 2018
@primes2h
Copy link
Contributor Author

Nei test in locale dopo alcuni fix non mi compaiono più gli errori di travis ma una manciata di questi, non credo siano legati alle modifiche fatte però.

2018-11-28 10:53:39,428 15615 ERROR fatturapa-testenable odoo.addons.l10n_it_fatturapa_out.tests.test_fatturapa_xml_validation: ERROR: test_3_xml_export (odoo.addons.l10n_it_fatturapa_out.tests.test_fatturapa_xml_validation.TestFatturaPAXMLValidation)
2018-11-28 10:53:39,428 15615 ERROR fatturapa-testenable odoo.addons.l10n_it_fatturapa_out.tests.test_fatturapa_xml_validation: Traceback (most recent call last):
2018-11-28 10:53:39,428 15615 ERROR fatturapa-testenable odoo.addons.l10n_it_fatturapa_out.tests.test_fatturapa_xml_validation: File "/-----------------/l10n-italy/l10n_it_fatturapa_out/tests/test_fatturapa_xml_validation.py", line 107, in test_3_xml_export 2018-11-28 10:53:39,428 15615 ERROR fatturapa-testenable odoo.addons.l10n_it_fatturapa_out.tests.test_fatturapa_xml_validation: self.set_sequences(3, 15, '2016-06-15')
2018-11-28 10:53:39,428 15615 ERROR fatturapa-testenable odoo.addons.l10n_it_fatturapa_out.tests.test_fatturapa_xml_validation: File "/--------------------------/l10n-italy/l10n_it_fatturapa_out/tests/fatturapa_common.py", line 91, in set_sequences 2018-11-28 10:53:39,428 15615 ERROR fatturapa-testenable odoo.addons.l10n_it_fatturapa_out.tests.test_fatturapa_xml_validation: inv_seq = seq_pool.search([('name', '=', 'Customer Invoices')])[0]
2018-11-28 10:53:39,428 15615 ERROR fatturapa-testenable odoo.addons.l10n_it_fatturapa_out.tests.test_fatturapa_xml_validation: File "/-------------------/odoo10/OCA/OCB/odoo/models.py", line 5240, in __getitem__ 2018-11-28 10:53:39,428 15615 ERROR fatturapa-testenable odoo.addons.l10n_it_fatturapa_out.tests.test_fatturapa_xml_validation: return self._browse((self._ids[key],), self.env)
2018-11-28 10:53:39,429 15615 ERROR fatturapa-testenable odoo.addons.l10n_it_fatturapa_out.tests.test_fatturapa_xml_validation: ` IndexError: tuple index out of range

@primes2h primes2h deleted the 10.0-fix-l10n_it_fatturapa_out-translation_source_strings branch February 3, 2019 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants