Skip to content

[14.0][FIX] l10n_it_split_payment#11

Merged
Borruso merged 3 commits into
Borruso:14.0-fix-unbalanced-linesfrom
tafaRU:14.0-fix-unbalanced-lines-sp
Jul 22, 2022
Merged

[14.0][FIX] l10n_it_split_payment#11
Borruso merged 3 commits into
Borruso:14.0-fix-unbalanced-linesfrom
tafaRU:14.0-fix-unbalanced-lines-sp

Conversation

@tafaRU

@tafaRU tafaRU commented Jul 22, 2022

Copy link
Copy Markdown

No description provided.

tafaRU added 3 commits July 22, 2022 13:24
Steps to reproduce:

* choose partner with fiscal position (Split Payment = True)
* create an invoice with a line
* edit the price unit

Current behavior:

See https://user-images.githubusercontent.com/3512779/179964568-955cbe00-5346-4532-a53b-efa1b1eb663e.gif for further info
2022-07-18 10:49:29,798 268 ERROR odoo odoo.addons.l10n_it_fatturapa_out_sp.tests.test_fatturapa_xml_validation: FAIL: TestFatturaPAXMLValidation.test_4_xml_export
Traceback (most recent call last):
  File "/__w/l10n-italy/l10n-italy/l10n_it_fatturapa_out_sp/tests/test_fatturapa_xml_validation.py", line 91, in test_4_xml_export
    self.check_content(xml_content, "IT06363391001_00004.xml")
  File "/__w/l10n-italy/l10n-italy/l10n_it_fatturapa_out/tests/fatturapa_common.py", line 209, in check_content
    self.assertEqual(etree.tostring(test_fatt), etree.tostring(xml))
AssertionError: b'<ns[1596 chars]016/0016</Numero><ImportoTotaleDocumento>17.08[1123 chars]ica>' != b'<ns[1596 chars]016/06/0001</Numero><ImportoTotaleDocumento>17[1126 chars]ica>'

See https://github.com/OCA/l10n-italy/runs/7387629100?check_suite_focus=true#step:8:909 for reference
@tafaRU tafaRU force-pushed the 14.0-fix-unbalanced-lines-sp branch from 89bc3f3 to 2261ece Compare July 22, 2022 11:25
@Borruso Borruso merged commit 1e4fe17 into Borruso:14.0-fix-unbalanced-lines Jul 22, 2022
@tafaRU tafaRU deleted the 14.0-fix-unbalanced-lines-sp branch July 22, 2022 13:04
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.

2 participants