fixup! [ADD] verdigado#57 allow higher value overtime for holidays

hbrunn · Jan 22, 2024 · 629cf13 · 629cf13
1 parent 609a146
commit 629cf13
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/verdigado_attendance/models/hr_attendance.py b/verdigado_attendance/models/hr_attendance.py
@@ -20,6 +20,8 @@ def _update_overtime(self, employee_attendance_dates=None):
         result = super()._update_overtime(
             employee_attendance_dates=employee_attendance_dates
         )
+        if not self.exists():
+            return result
         if employee_attendance_dates is None:
             employee_attendance_dates = self._get_attendances_dates()
 

diff --git a/verdigado_attendance/tests/test_overtime_calculation.py b/verdigado_attendance/tests/test_overtime_calculation.py
@@ -156,6 +156,9 @@ def test_calculation_employeeA(self):
         attendance.check_out += timedelta(hours=1)
         self.assertOvertime(employeeA, "2023-08-06", 34 * 60, 0)
         self.assertOvertime(employeeA, "2023-08-06", 1.5 * 34 * 60, 0, adjustment=True)
+        attendance.unlink()
+        self.assertOvertime(employeeA, "2023-08-06", 0, 0)
+        self.assertOvertime(employeeA, "2023-08-06", 0, 0, adjustment=True)
 
     def to_time(self, time_string):
         if isinstance(time_string, str):