[14.0][FIX] l10n_it_fatturapa_in: test improvement

[TheMule71]

Make one test independent from others (to be excuted in isolation)

[sergiocorato]

Forse si ottiene lo stesso risultato rinominando il test in modo che venga eseguito nell'ordine corretto? tipo test_54_duplicated_vat_on_partners

[TheMule71]

Forse si ottiene lo stesso risultato rinominando il test in modo che venga eseguito nell'ordine corretto? tipo test_54_duplicated_vat_on_partners

Sì ma il punto è quello di renderlo eseguibile separatamente. I test dovrebbero dipendere solo da oggetti creati a livello di setUp (o ancora meglio a livello di classe), in modo che l'ambiente sia ben documentato. Avere un test che dipende da un side-effect di un test eseguito precedentemente oltre che impedire l'esecuzione singola del test (molto utile in caso debug), crea anche un potenziale problema futuro se i test precedenti vengono modificati/migliorati.

[sergiocorato]

/ocabot rebase

[OCA-git-bot]

Congratulations, PR rebased to 14.0.

[SirAionTech]

Grazie della PR!
Ho proposto un'altra soluzione, secondo me più lineare, ma non vedo problemi con quella proposta

[SirAionTech]

Quindi per assurdo, questo test non è coperto dai test!
Non si potrebbe invece usare un'altra partita IVA che non è ancora stata usata per questo partner? Ho trovato https://strumentidev.it/generatore/partita-iva per generarne di valide.

Ho usato questa soluzione in 4d74656 (di #3810)

[sergiocorato]

Forse si ottiene lo stesso risultato rinominando il test in modo che venga eseguito nell'ordine corretto? tipo test_54_duplicated_vat_on_partners

Sì ma il punto è quello di renderlo eseguibile separatamente. I test dovrebbero dipendere solo da oggetti creati a livello di setUp (o ancora meglio a livello di classe), in modo che l'ambiente sia ben documentato. Avere un test che dipende da un side-effect di un test eseguito precedentemente oltre che impedire l'esecuzione singola del test (molto utile in caso debug), crea anche un potenziale problema futuro se i test precedenti vengono modificati/migliorati.

Ho riletto questa PR, questi test sono di tipo SingleTransactionCase per cui è normale che vengano eseguiti tutti di seguito con una dipendenza uno dall'altro.
Se si intende tenerli separati si può usare SavepointCase no?

[github-actions]

There hasn't been any activity on this pull request in the past 4 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 PR to never become stale, please ask a PSC member to apply the "no stale" label.