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

10n_it_fatturapa_out_triple_discount genera errore nei test automatici #1588

Closed
1 of 2 tasks
eLBati opened this issue Jan 14, 2020 · 9 comments
Closed
1 of 2 tasks

10n_it_fatturapa_out_triple_discount genera errore nei test automatici #1588

eLBati opened this issue Jan 14, 2020 · 9 comments
Labels
help wanted stale PR/Issue without recent activity, it'll be soon closed automatically. tracking

Comments

@eLBati
Copy link
Member

eLBati commented Jan 14, 2020

Versioni coinvolte:

Comportamento osservato:

Ad esempio

2020-01-13 13:00:26,096 5951 INFO openerp_test odoo.addons.l10n_it_fatturapa_out_triple_discount.tests.test_fatturapa_triple_discount: test_xml_export_triple_discount (odoo.addons.l10n_it_fatturapa_out_triple_discount.tests.test_fatturapa_triple_discount.TestInvoiceTripleDiscount)
2020-01-13 13:00:27,917 5951 ERROR openerp_test odoo.addons.l10n_it_fatturapa_out_triple_discount.tests.test_fatturapa_triple_discount: FAIL
2020-01-13 13:00:27,917 5951 INFO openerp_test odoo.addons.l10n_it_fatturapa_out_triple_discount.tests.test_fatturapa_triple_discount: ======================================================================
2020-01-13 13:00:27,917 5951 ERROR openerp_test odoo.addons.l10n_it_fatturapa_out_triple_discount.tests.test_fatturapa_triple_discount: FAIL: test_xml_export_triple_discount (odoo.addons.l10n_it_fatturapa_out_triple_discount.tests.test_fatturapa_triple_discount.TestInvoiceTripleDiscount)
2020-01-13 13:00:27,917 5951 ERROR openerp_test odoo.addons.l10n_it_fatturapa_out_triple_discount.tests.test_fatturapa_triple_discount: Traceback (most recent call last):
2020-01-13 13:00:27,917 5951 ERROR openerp_test odoo.addons.l10n_it_fatturapa_out_triple_discount.tests.test_fatturapa_triple_discount: `   File "/home/travis/build/OCA/l10n-italy/l10n_it_fatturapa_out_triple_discount/tests/test_fatturapa_triple_discount.py", line 50, in test_xml_export_triple_discount
2020-01-13 13:00:27,918 5951 ERROR openerp_test odoo.addons.l10n_it_fatturapa_out_triple_discount.tests.test_fatturapa_triple_discount: `     module_name='l10n_it_fatturapa_out_triple_discount')
2020-01-13 13:00:27,918 5951 ERROR openerp_test odoo.addons.l10n_it_fatturapa_out_triple_discount.tests.test_fatturapa_triple_discount: `   File "/home/travis/build/OCA/l10n-italy/l10n_it_fatturapa_out/tests/fatturapa_common.py", line 122, in check_content
2020-01-13 13:00:27,918 5951 ERROR openerp_test odoo.addons.l10n_it_fatturapa_out_triple_discount.tests.test_fatturapa_triple_discount: `     self.assertEqual(etree.tostring(test_fatt), etree.tostring(xml))
2020-01-13 13:00:27,918 5951 ERROR openerp_test odoo.addons.l10n_it_fatturapa_out_triple_discount.tests.test_fatturapa_triple_discount: ` AssertionError: b'<ns[2618 chars]ento>1.53</ImportoPagamento></DettaglioPagamen[65 chars]ica>' != b'<ns[2618 chars]ento>5.28</ImportoPagamento></DettaglioPagamen[65 chars]ica>'

(https://travis-ci.org/OCA/l10n-italy/jobs/636339731)

@eLBati
Copy link
Member Author

eLBati commented Jan 19, 2020

@tafaRU @SimoRubi disabilitiamo i test del modulo nel frattempo?

@tafaRU
Copy link
Member

tafaRU commented Jan 20, 2020

@eLBati per me è ok. @SimoRubi tu che dici?

@SimoRubi
Copy link
Member

Ciao! Per me disabilitare i test non è una buona idea: lasciando il problema aperto si spera che qualcuno prima o poi lo risolverà (se avessi il tempo di farlo proverei direttamente io), nasconderlo disabilitando i test lascia il problema latente e potrebbe rispuntare in futuro.

@eLBati
Copy link
Member Author

eLBati commented Jan 20, 2020

Il problema andrà risolto, però ci sono PR aperte (tipo #1482) su cui ho già lanciato 4 o 5 volte il comando di merge che è fallito per questo problema.

@OpenCode
Copy link
Contributor

@eLBati L'alternativa sarebbe mergiare a mano le PR facendo bump manuale della versione nel file di manifest

@eLBati
Copy link
Member Author

eLBati commented Jan 21, 2020

Non caricherei di lavoro i contributori a causa di un modulo che probabilmente non andava rilasciato, in quanto problematico.
A questo punto, visto che nessuno sembra interessato a sistemare il modulo (io non lo uso quindi non prevedo di aggiustarlo) piuttosto che rimuovere il modulo, disabilitiamo i test: #1603

@tafaRU
Copy link
Member

tafaRU commented Jan 22, 2020

Sul branch 10.0 si verifica lo stesso problema?

@eLBati
Copy link
Member Author

eLBati commented Jan 22, 2020

@tafaRU mi pare di sì

@github-actions
Copy link

github-actions bot commented Dec 4, 2022

There hasn't been any activity on this issue in the past 6 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this issue to never become stale, please ask a PSC member to apply the "no stale" label.

@github-actions github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Dec 4, 2022
@github-actions github-actions bot closed this as completed Jan 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted stale PR/Issue without recent activity, it'll be soon closed automatically. tracking
Projects
None yet
Development

No branches or pull requests

5 participants