From bcb661dc226acdf40380f52431422b0dddce87bd Mon Sep 17 00:00:00 2001 From: al8763be Date: Fri, 29 Mar 2024 13:02:31 +0100 Subject: [PATCH] bug fix getUnsignedReports --- backend/internal/handlers/handlers_report_related.go | 5 +++++ testing.py | 6 ++++-- 2 files changed, 9 insertions(+), 2 deletions(-) diff --git a/backend/internal/handlers/handlers_report_related.go b/backend/internal/handlers/handlers_report_related.go index 52e1564..9fa83cc 100644 --- a/backend/internal/handlers/handlers_report_related.go +++ b/backend/internal/handlers/handlers_report_related.go @@ -137,6 +137,11 @@ func (gs *GState) GetUnsignedReports(c *fiber.Ctx) error { log.Info("Failed to get project manager ID") return c.Status(500).SendString("Failed to get project manager ID") } + if !isProjectManager { + log.Info("User is not a project manager") + return c.Status(401).SendString("User is not a project manager") + } + log.Info("User is Project Manager: ", isProjectManager) // Call the database function to get the unsigned weekly reports diff --git a/testing.py b/testing.py index 5fb8ffe..cde8f5c 100644 --- a/testing.py +++ b/testing.py @@ -505,18 +505,20 @@ def test_remove_project(): def test_get_unsigned_reports(): # Log in as the user - token = login("user2", "123").json()["token"] + token = login("admin", "123").json()["token"] # Make a request to get all unsigned reports response = requests.get( - getUnsignedReportsPath + "/" + projectName, + getUnsignedReportsPath + "/" "projecttest", headers={"Authorization": "Bearer " + token}, ) + dprint(response.text) assert response.status_code == 200, "Get unsigned reports failed" gprint("test_get_unsigned_reports successful") if __name__ == "__main__": + test_get_unsigned_reports() test_remove_project() test_get_user_projects() test_create_user()