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

Adaptar crear_lectura_data de la classe GiscedataPolissaCalculada als canvis de wizard crear lectura desde cch #723

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

aivan2022
Copy link
Contributor

Necessita

https://github.com/gisce/erp/pull/21582

Objectiu

La funció crear_lectura_data de la classe GiscedataPolissaCalculada ha de seguir funcionant com ho feia fins ara

Targeta on es demana o Incidència

Comportament antic

La funció crear_lectura_data de la classe GiscedataPolissaCalculada esperava que l'assistent wizard.measures.from.curve llances una excepció osv.orm.except_orm quan no es trobaven corbes per les dates proporcionades.

Comportament nou

Ara l'assistent wizard.measures.from.curve llança una excepció del tipus osv.except_osv i s'ha canviat la funció crear_lectura_data perquè la tingui en compte.

Comprovacions

  • Hi ha testos
  • Reiniciar serveis
  • Actualitzar mòdul
  • Script de migració
  • Modifica traduccions

Copy link

github-actions bot commented Sep 9, 2024

Test Results

73 tests   - 50   73 ✅  - 48   0s ⏱️ - 1m 52s
 2 suites ± 0    0 💤  -  1 
 2 files   ± 0    0 ❌ ± 0 

Results for commit 28158b7. ± Comparison against base commit 4025e64.

This pull request removes 53 and adds 3 tests. Note that renamed tests count towards both.
som_switching.tests.tests_activaciones.TestsAutoActiva ‑ test_import_r105_01_tancar_cac_al_finalitzar_r1
som_switching.tests.tests_activaciones.TestsAutoActiva ‑ test_se_cierra_caso_cac_activacion_automatica_009_open
som_switching.tests.tests_activaciones.TestsAutoActiva ‑ test_se_cierra_caso_cac_activacion_automatica_009_pending
som_switching.tests.tests_activacions_b1.TestActivacioB1 ‑ test_b1_05_baixa_mailchimp_error__active_contract
som_switching.tests.tests_activacions_b1.TestActivacioB1 ‑ test_b1_05_baixa_mailchimp_error__more_than_one_contract
som_switching.tests.tests_activacions_b1.TestActivacioB1 ‑ test_b1_05_baixa_mailchimp_ok
som_switching.tests.tests_activacions_b2.TestActivacioB2 ‑ test_b2_05_baixa_mailchimp_error__active_contract
som_switching.tests.tests_activacions_b2.TestActivacioB2 ‑ test_b2_05_baixa_mailchimp_error__more_than_one_contract
som_switching.tests.tests_activacions_b2.TestActivacioB2 ‑ test_b2_05_baixa_mailchimp_ok
som_switching.tests.tests_activacions_cn.TestActivacioCn ‑ test_c1_06_baixa_mailchimp_error__active_contract
…
som_facturacio_calculada.tests.test_polissa_lectures_calculades.PolissaLecturesCalculadesTest ‑ test_crear_lectures_calculades__data_fi_menor_inicial
som_facturacio_calculada.tests.test_polissa_lectures_calculades.PolissaLecturesCalculadesTest ‑ test_crear_lectures_calculades__sense_data_ultima_lectura
som_facturacio_calculada.tests.test_polissa_lectures_calculades.PolissaLecturesCalculadesTest ‑ test_crear_lectures_calculades__sense_lectures

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant