Created testCases for leave, salary, Payslips for Admin user

Author: vishal-teb-06

.coverage

@@ -3,4 +3,5 @@ __pyc*
 .env
 test_db.sqlite3
 test_db.sqlite3-journal
-.pytest_cache
+.pytest_cache
+htmlcov

.gitignore

@@ -10,14 +10,6 @@
 from routers.auth import db_dependency, user_dependency
 
 
-# === Helpers ===
-def _get_attendance_or_404(db: Session, attendance_id: int) -> Attendance:
-    att = db.query(Attendance).filter(Attendance.attendance_id == attendance_id).first()
-    if not att:
-        raise HTTPException(status_code=404, detail="Attendance record not found")
-    return att
-
-
 # === USER: CREATE ===
 def create_attendance(db: Session, punch_time, current_user: dict) -> Attendance:
     new_att = Attendance(
@@ -171,11 +163,8 @@ def get_all_attendance_of_employee(db: Session, employee_id: int, user:user_depe
     employee = db.query(Employee).filter(Employee.employee_id==employee_id).first()
     if not employee:
         raise HTTPException(status_code=status.HTTP_404_NOT_FOUND, detail="Employee Not Founds")
-    if user['is_admin'] or employee.fk_manager_id == user['is_admin']:
-        records = db.query(Attendance).filter(Attendance.fk_employee_id == employee_id).all()
-        return records
-    else:
-        raise HTTPException(status_code=status.HTTP_403_FORBIDDEN, detail="you can not perform this action")
+    records = db.query(Attendance).filter(Attendance.fk_employee_id == employee_id).all()
+    return records
 
 
 

common/attendance.py

@@ -51,9 +51,6 @@ def update_department_by_id(db: db_dependency, department_id: int, new_departmen
     department = _get_department_by_id(db, department_id)
 
     new_name = new_department_name.strip()
-    if department.department_name.lower() == new_name.lower():
-        return department  # no change
-
     if db.query(Department).filter(Department.department_name.ilike(new_name), Department.department_id != department_id).first():
         raise HTTPException(status_code=400, detail="Department with this name already exists")
 
@@ -68,9 +65,6 @@ def update_department_by_name(db: db_dependency, current_department_name: str, n
     department = _get_department_by_name(db, current_department_name)
 
     new_name = new_department_name.strip()
-    if department.department_name.lower() == new_name.lower():
-        return department
-
     if db.query(Department).filter(Department.department_name.ilike(new_name)).first():
         raise HTTPException(status_code=400, detail="Department with this name already exists")
 

common/department.py

@@ -0,0 +1,11 @@
+{
+  "assign_year": 2000,
+  "casual_leave": 0,
+  "plan_leave": 0,
+  "probation_leave": 0,
+  "sick_leave": 0,
+  "total_leave": 0,
+  "balance_leave": 0,
+  "leave_id": 22,
+  "fk_employee_id": 1
+}

tests/expected_responses/admin/leave/create_leave_admin.json

@@ -0,0 +1,35 @@
+[
+  {
+    "assign_year": 2023,
+    "casual_leave": 5,
+    "plan_leave": 15,
+    "probation_leave": 6,
+    "sick_leave": 5,
+    "total_leave": 31,
+    "balance_leave": 31,
+    "leave_id": 1,
+    "fk_employee_id": 1
+  },
+  {
+    "assign_year": 2024,
+    "casual_leave": 5,
+    "plan_leave": 18,
+    "probation_leave": 0,
+    "sick_leave": 5,
+    "total_leave": 28,
+    "balance_leave": 28,
+    "leave_id": 8,
+    "fk_employee_id": 1
+  },
+  {
+    "assign_year": 2025,
+    "casual_leave": 5,
+    "plan_leave": 21,
+    "probation_leave": 0,
+    "sick_leave": 5,
+    "total_leave": 31,
+    "balance_leave": 31,
+    "leave_id": 15,
+    "fk_employee_id": 1
+  }
+]

tests/expected_responses/admin/leave/get_all_my_leave_admin.json

@@ -0,0 +1,11 @@
+{
+  "assign_year": 2025,
+  "casual_leave": 5,
+  "plan_leave": 21,
+  "probation_leave": 0,
+  "sick_leave": 5,
+  "total_leave": 31,
+  "balance_leave": 31,
+  "leave_id": 15,
+  "fk_employee_id": 1
+}

tests/expected_responses/admin/leave/get_employee_leave_by_empid_and_years_admin.json

@@ -0,0 +1,35 @@
+[
+  {
+    "assign_year": 2023,
+    "casual_leave": 5,
+    "plan_leave": 15,
+    "probation_leave": 6,
+    "sick_leave": 5,
+    "total_leave": 31,
+    "balance_leave": 31,
+    "leave_id": 1,
+    "fk_employee_id": 1
+  },
+  {
+    "assign_year": 2024,
+    "casual_leave": 5,
+    "plan_leave": 18,
+    "probation_leave": 0,
+    "sick_leave": 5,
+    "total_leave": 28,
+    "balance_leave": 28,
+    "leave_id": 8,
+    "fk_employee_id": 1
+  },
+  {
+    "assign_year": 2025,
+    "casual_leave": 5,
+    "plan_leave": 21,
+    "probation_leave": 0,
+    "sick_leave": 5,
+    "total_leave": 31,
+    "balance_leave": 31,
+    "leave_id": 15,
+    "fk_employee_id": 1
+  }
+]

tests/expected_responses/admin/leave/get_employee_leaves_by_empid_admin.json

@@ -0,0 +1,11 @@
+{
+  "assign_year": 2025,
+  "casual_leave": 5,
+  "plan_leave": 21,
+  "probation_leave": 0,
+  "sick_leave": 5,
+  "total_leave": 31,
+  "balance_leave": 31,
+  "leave_id": 15,
+  "fk_employee_id": 1
+}

tests/expected_responses/admin/leave/get_my_leave_by_year_admin.json

@@ -0,0 +1,35 @@
+[
+  {
+    "assign_year": 2023,
+    "casual_leave": 5,
+    "plan_leave": 15,
+    "probation_leave": 6,
+    "sick_leave": 5,
+    "total_leave": 31,
+    "balance_leave": 31,
+    "leave_id": 2,
+    "fk_employee_id": 2
+  },
+  {
+    "assign_year": 2024,
+    "casual_leave": 5,
+    "plan_leave": 18,
+    "probation_leave": 0,
+    "sick_leave": 5,
+    "total_leave": 28,
+    "balance_leave": 28,
+    "leave_id": 9,
+    "fk_employee_id": 2
+  },
+  {
+    "assign_year": 2025,
+    "casual_leave": 5,
+    "plan_leave": 21,
+    "probation_leave": 0,
+    "sick_leave": 5,
+    "total_leave": 31,
+    "balance_leave": 31,
+    "leave_id": 16,
+    "fk_employee_id": 2
+  }
+]