Skip to content

Introduzione di nuovi unit test con mocking - #1646

Merged
Pek5892 merged 3 commits into
devcode-it:masterfrom
Dasc3er:master
Jul 17, 2026
Merged

Introduzione di nuovi unit test con mocking#1646
Pek5892 merged 3 commits into
devcode-it:masterfrom
Dasc3er:master

Conversation

@Dasc3er

@Dasc3er Dasc3er commented Mar 27, 2025

Copy link
Copy Markdown
Contributor

Introduzione di nuovi unit test a livello di modulo, come proposto in #1567

L'unit test introdotto fa riferimento alla gestione delle scadenze da fattura, con verifica delle rate generate. Maggiori test dovrebbero essere aggiunto per verificare il corretto funzionamento di questo calcolo.

Nota: si raccomanda test intesivi prima di fare il merge, per verificare che il refactoring introdotto (richiesto per mocking degli oggetti) non abbia rotto qualche funzionalità particolare.

Idealmente, prima del merge si dovrebbero aggiungere test per tutti i casi rilevanti per la classe, dunque:

  • Lettura da fattura elettronica
  • Ritenuta acconto
  • Assicurazione Crediti
  • Pagamento personalizzato in fattura/da anagrafica principale (potrebbe richiedere ulteriori mock)
  • Con scadenze già inserite
  • Con scadenze già inserite e parzialmente pagate
  • Diverse logiche previste per il calcolo della data, incluso controllo con anni bisestili e altri casi particolari rilevanti
  • Totali con valori particolari che provocano arrotondamenti nelle rate, per verificare che il totale finale di ogni rata sommato corrisponda al totale effettivo in fattura al centesimo

Si può considerare questo come un esempio semplice del test, e sviluppare il tutto separatamente in un'altra PR.

@Dasc3er
Dasc3er force-pushed the master branch 2 times, most recently from a1d5715 to 60a1dfa Compare March 27, 2025 19:51
@Dasc3er
Dasc3er requested a review from loviuz March 27, 2025 20:02
@Dasc3er
Dasc3er marked this pull request as ready for review March 27, 2025 20:03
@Dasc3er
Dasc3er force-pushed the master branch 2 times, most recently from 61d9cd0 to 0d0dac1 Compare May 10, 2025 09:11
@Dasc3er
Dasc3er force-pushed the master branch 2 times, most recently from ec828db to 80c02d0 Compare July 7, 2026 17:57
@Pek5892
Pek5892 merged commit b0d60f6 into devcode-it:master Jul 17, 2026
2 of 3 checks passed
@Pek5892

Pek5892 commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Ho allineato le modifiche al repository aggiornato e introdotto i nuovi tests in modules/fatture/tests

@Dasc3er

Dasc3er commented Jul 18, 2026

Copy link
Copy Markdown
Contributor Author

Grazie! Una nota, per favore verificate bene che le modifiche sulla classe per le scadenze non abbiano introdotto bug
Ho testato al mio meglio ma è possibile qualcosa mi sia sfuggito, e gli unit test dovrebbero aiutare in futuro.

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