feat(finance/tests): tests for new rules #155

Merged
christian.merten merged 6 commits from MK/finance_tests into main 5 months ago
Collaborator

Also makes some checks safe.

Also makes some checks safe.
christian.merten reviewed 7 months ago
@ -69,0 +101,4 @@
activity = 'hi',
ljp_proposal=ljpproposal
)
int.save()

Intervention.objects.create speichert schon, kein weiteres int.save() notwendig.

`Intervention.objects.create` speichert schon, kein weiteres `int.save()` notwendig.
marius.klein marked this conversation as resolved
christian.merten reviewed 7 months ago
@ -69,0 +103,4 @@
)
int.save()
ljpproposal.save()

Selbes hier.

Selbes hier.
marius.klein marked this conversation as resolved
christian.merten reviewed 7 months ago
@ -1429,1 +1431,4 @@
def potential_ljp_contributions(self):
"""The maximal amount can be reduced if the actual costs are lower than the maximal amount
This calculation if intended for the LJP application, not for the payout."""
if not self.statement:

Das wird einen Fehler werfen, wenn die Ausfahrt keine Abrechnung zugeordnet hat. (Das OneToOneField ist im Statement model registriert, das heißt statement.excursion ist None falls keine Ausfahrt zugeordnet ist. Umgekehrt ist excursion.statement Django magic, die nicht funktioniert, wenn kein statement existiert mit statement.excursion = excursion. Ich teste daher immer mit hasattr(self, 'statement').

PS.: Es gibt noch mehr kaputte methoden, siehe der test FreizeitTestCase.test_no_statement, der aktuell geskipped wird, weil er einen Fehler wirft.

Das wird einen Fehler werfen, wenn die Ausfahrt keine Abrechnung zugeordnet hat. (Das `OneToOneField` ist im `Statement` model registriert, das heißt `statement.excursion` ist `None` falls keine Ausfahrt zugeordnet ist. Umgekehrt ist `excursion.statement` Django magic, die nicht funktioniert, wenn kein `statement` existiert mit `statement.excursion = excursion`. Ich teste daher immer mit `hasattr(self, 'statement')`. PS.: Es gibt noch mehr kaputte methoden, siehe der test `FreizeitTestCase.test_no_statement`, der aktuell geskipped wird, weil er einen Fehler wirft.
marius.klein marked this conversation as resolved
christian.merten reviewed 7 months ago
christian.merten left a comment
Owner

Danke!

Danke!
Poster
Collaborator

@christian.merten ich habe deine Hinweise eingearbeitet. Würdest du bei Gelegenheit einmal drauf schauen, ob das so für dich passt? Danke!

@christian.merten ich habe deine Hinweise eingearbeitet. Würdest du bei Gelegenheit einmal drauf schauen, ob das so für dich passt? Danke!
marius.klein requested review from christian.merten 5 months ago
christian.merten changed title from Tests für neue Berechnungslogik für Ausfahrten to feat(finance/tests): test for new rules 5 months ago
christian.merten changed title from feat(finance/tests): test for new rules to feat(finance/tests): tests for new rules 5 months ago
christian.merten approved these changes 5 months ago
christian.merten merged commit a8d4625719 into main 5 months ago
christian.merten deleted branch MK/finance_tests 5 months ago

Reviewers

christian.merten approved these changes 5 months ago
The pull request has been merged as a8d4625719.
Sign in to join this conversation.
No reviewers
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: digitales/kompass#155
Loading…
There is no content yet.