[18.0][MIG] l10n_es_vat_book_pos: Migration to 18.0

[polqubiq]

Migration of l10n_es_vat_book_pos from 17.0 to 18.0.

[polqubiq]

@etobella I migrated the module you mentioned #3754 (comment)
I think the tests got stuck.
cc: @pedrobaeza

[pedrobaeza]

Es cierto que hay un problema con la librería odoo_test_helper que emite unos logs molestos:

OCA/odoo-test-helper#39

Pero el error del CI es de este módulo:

2026-02-19 11:22:30,885 3232 ERROR odoo odoo.addons.l10n_es_vat_book_pos.tests.test_vat_book_pos: ERROR: TestVATBookPOS.test_vat_book_pos
Traceback (most recent call last):
  File "/opt/odoo/odoo/models.py", line 6274, in ensure_one
    _id, = self._ids
ValueError: not enough values to unpack (expected 1, got 0)

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/__w/l10n-spain/l10n-spain/l10n_es_vat_book_pos/tests/test_vat_book_pos.py", line 68, in test_vat_book_pos
    session = self._create_session()
  File "/__w/l10n-spain/l10n-spain/l10n_es_vat_book_pos/tests/test_vat_book_pos.py", line 21, in _create_session
    self.config.open_ui()
  File "/opt/odoo/addons/point_of_sale/models/pos_config.py", line 711, in open_ui
    self.ensure_one()
  File "/opt/odoo/odoo/models.py", line 6277, in ensure_one
    raise ValueError("Expected singleton: %s" % self)
ValueError: Expected singleton: pos.config()

[HaraldPanten]

/ocabot migration l10n_es_vat_book_pos

[polqubiq]

@pedrobaeza ahora los tests están en estado cancelado.
De todas formas, he hecho modificaciones en los tests. Ahora deberían estar bien (como mínimo, han pasado correctamente en mi local).
Gracias.

[pedrobaeza]

@polqubiq haz por favor rebase sobre lo último de la rama, que el otro error está ya corregido.

[polqubiq]

@polqubiq haz por favor rebase sobre lo último de la rama, que el otro error está ya corregido.

@pedrobaeza ¡hecho! Sale otro error en el test pero no es de código. Es de Remote end closed connection without response.

[pedrobaeza]

/ocabot merge nobump

[OCA-git-bot]

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-4841-by-pedrobaeza-bump-nobump, awaiting test results.

[OCA-git-bot]

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-4841-by-pedrobaeza-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

[pedrobaeza]

/ocabot merge nobump

[OCA-git-bot]

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-4841-by-pedrobaeza-bump-nobump, awaiting test results.

[OCA-git-bot]

Congratulations, your PR was merged at 601f602. Thanks a lot for contributing to OCA. ❤️