Merge remote-tracking branch 'origin/AlexTester' into gruppDM

backend/internal/database/db.go

@@ -41,6 +41,7 @@ type Database interface {
 	IsProjectManager(username string, projectname string) (bool, error)
 	GetProjectTimes(projectName string) (map[string]int, error)
 	UpdateWeeklyReport(projectName string, userName string, week int, developmentTime int, meetingTime int, adminTime int, ownWorkTime int, studyTime int, testingTime int) error
+	RemoveProject(projectname string) error
 }
 
 // This struct is a wrapper type that holds the database connection
@@ -549,3 +550,8 @@ func (d *Db) GetProjectTimes(projectName string) (map[string]int, error) {
 
 	return totalTime, nil
 }
+
+func (d *Db) RemoveProject(projectname string) error {
+	_, err := d.Exec("DELETE FROM projects WHERE name = ?", projectname)
+	return err
+}

backend/internal/database/db_test.go

@@ -894,3 +894,33 @@ func TestUpdateWeeklyReport(t *testing.T) {
 		t.Error("UpdateWeeklyReport failed: report not updated correctly")
 	}
 }
+
+func TestRemoveProject(t *testing.T) {
+	db, err := setupAdvancedState()
+	if err != nil {
+		t.Error("setupState failed:", err)
+	}
+
+	// Promote user to Admin
+	err = db.PromoteToAdmin("demouser")
+	if err != nil {
+		t.Error("PromoteToAdmin failed:", err)
+	}
+
+	// Remove project
+	err = db.RemoveProject("projecttest")
+	if err != nil {
+		t.Error("RemoveProject failed:", err)
+	}
+
+	// Check if the project was removed
+	projects, err := db.GetAllProjects()
+	if err != nil {
+		t.Error("GetAllProjects failed:", err)
+	}
+	if len(projects) != 0 {
+		t.Error("RemoveProject failed: expected 0, got", len(projects))
+	}
+	
+}
+	

backend/internal/handlers/global_state.go

@@ -29,6 +29,7 @@ type GlobalState interface {
 	ChangeUserName(c *fiber.Ctx) error      // WIP
 	GetAllUsersProject(c *fiber.Ctx) error  // WIP
 	UpdateWeeklyReport(c *fiber.Ctx) error
+	RemoveProject(c *fiber.Ctx) error
 }
 
 // "Constructor"

backend/internal/handlers/handlers_project_related.go

@@ -287,3 +287,29 @@ func (gs *GState) GetProjectTimesHandler(c *fiber.Ctx) error {
     log.Info("Returning project times for project:", projectName)
     return c.JSON(projectTimes)
 }
+
+func (gs *GState) RemoveProject(c *fiber.Ctx) error {
+	user := c.Locals("user").(*jwt.Token)
+    claims := user.Claims.(jwt.MapClaims)
+    username := claims["name"].(string)
+    
+	// Check if the user is a site admin
+	isAdmin, err := gs.Db.IsSiteAdmin(username)
+	if err != nil {
+		log.Info("Error checking admin status:", err)
+		return c.Status(500).SendString(err.Error())
+	}
+
+	if !isAdmin {
+		log.Info("User is not a site admin:", username)
+		return c.Status(403).SendString("User is not a site admin")
+	}
+
+	projectName := c.Params("projectName")
+
+	if err := gs.Db.RemoveProject(projectName); err != nil {
+		return c.Status(500).SendString((err.Error()))
+	}
+
+	return c.Status(200).SendString("Project deleted")
+}

backend/main.go

@@ -108,6 +108,7 @@ func main() {
 	server.Post("/api/ProjectRoleChange", gs.ProjectRoleChange)
 	server.Get("/api/getUsersProject/:projectName", gs.ListAllUsersProject)
 	server.Put("/api/updateWeeklyReport", gs.UpdateWeeklyReport)
+	server.Delete("/api/removeProject/:projectName", gs.RemoveProject)
 
 	// Announce the port we are listening on and start the server
 	err = server.Listen(fmt.Sprintf(":%d", conf.Port))

frontend/src/API/API.ts

@@ -47,7 +47,6 @@ interface API {
    * @returns {Promise<APIResponse<boolean>>} A promise containing the API response indicating if the user is a project manager.
    */
   checkIfProjectManager(
-    username: string,
     projectName: string,
     token: string,
   ): Promise<APIResponse<boolean>>;
@@ -133,6 +132,11 @@ interface API {
     projectName: string,
     token: string,
   ): Promise<APIResponse<UserProjectMember[]>>;
+
+  removeProject(
+    projectName: string,
+    token: string,
+  ): Promise<APIResponse<string>>;
 }
 
 /** An instance of the API */
@@ -190,19 +194,20 @@ export const api: API = {
   },
 
   async checkIfProjectManager(
-    username: string,
     projectName: string,
     token: string,
   ): Promise<APIResponse<boolean>> {
     try {
-      const response = await fetch("/api/checkIfProjectManager", {
+      const response = await fetch(
+        `/api/checkIfProjectManager?projectName=${projectName}`,
+        {
           method: "GET",
           headers: {
             "Content-Type": "application/json",
             Authorization: "Bearer " + token,
           },
-        body: JSON.stringify({ username, projectName }),
+        },
-      });
+      );
 
       if (!response.ok) {
         return {
@@ -214,7 +219,7 @@ export const api: API = {
         return { success: true, data };
       }
     } catch (e) {
-      return { success: false, message: "fuck" };
+      return { success: false, message: "Failed to check if project manager" };
     }
   },
 
@@ -484,4 +489,34 @@ export const api: API = {
       });
     }
   },
+
+  async removeProject(
+    projectName: string,
+    token: string,
+  ): Promise<APIResponse<string>> {
+    try {
+      const response = await fetch(`/api/projectdelete/${projectName}`, {
+        method: "DELETE",
+        headers: {
+          "Content-Type": "application/json",
+          Authorization: "Bearer " + token,
+        },
+      });
+
+      if (!response.ok) {
+        return Promise.resolve({
+          success: false,
+          message: "Failed to remove project",
+        });
+      } else {
+        const data = await response.text();
+        return Promise.resolve({ success: true, message: data });
+      }
+    } catch (e) {
+      return Promise.resolve({
+        success: false,
+        message: "Failed to remove project",
+      });
+    }
+  },
 };

frontend/src/Pages/AdminPages/AdminProjectPage.tsx

@@ -1,17 +1,26 @@
+import { useParams } from "react-router-dom";
+import { api } from "../../API/API";
 import BackButton from "../../Components/BackButton";
 import BasicWindow from "../../Components/BasicWindow";
 import Button from "../../Components/Button";
 
+async function handleDeleteProject(
+  projectName: string,
+  token: string,
+): Promise<void> {
+  await api.removeProject(projectName, token);
+}
+
 function AdminProjectPage(): JSX.Element {
   const content = <></>;
+  const { projectName } = useParams();
+  const token = localStorage.getItem("accessToken");
 
   const buttons = (
     <>
       <Button
         text="Delete"
-        onClick={(): void => {
+        onClick={() => handleDeleteProject(projectName, token)}
-          return;
-        }}
         type="button"
       />
       <BackButton />
@@ -20,4 +29,5 @@ function AdminProjectPage(): JSX.Element {
 
   return <BasicWindow content={content} buttons={buttons} />;
 }
+
 export default AdminProjectPage;

testing.py

@@ -43,6 +43,7 @@ ProjectRoleChangePath = base_url + "/api/ProjectRoleChange"
 getUsersProjectPath = base_url + "/api/getUsersProject"
 getChangeUserNamePath = base_url + "/api/changeUserName"
 getUpdateWeeklyReportPath = base_url + "/api/updateWeeklyReport"
+removeProjectPath = base_url + "/api/removeProject"
 
 #ta bort auth i handlern för att få testet att gå igenom
 def test_ProjectRoleChange():
@@ -462,7 +463,47 @@ def test_update_weekly_report():
     assert response.status_code == 200, "Update weekly report failed"
     gprint("test_update_weekly_report successful")
 
+
+def test_remove_project():
+    admin_username = randomString()
+    admin_password = "admin_password2"
+    dprint(
+        "Registering with username: ", admin_username, " and password: ", admin_password
+    )
+    response = requests.post(
+        registerPath, json={"username": admin_username, "password": admin_password}
+    )
+    dprint(response.text)
+
+    # Log in as the admin
+    admin_token = login(admin_username, admin_password).json()["token"]
+    response = requests.post(
+        promoteToAdminPath,
+        json={"username": admin_username},
+        headers={"Authorization": "Bearer " + admin_token},
+    )
+
+    # Create a new project
+    new_project = randomString()
+    response = requests.post(
+        addProjectPath,
+        json={"name": new_project, "description": "This is a project"},
+        headers={"Authorization": "Bearer " + admin_token},
+    )
+    assert response.status_code == 200, "Add project failed"
+
+    # Remove the project
+    response = requests.delete(
+        removeProjectPath + "/" + new_project,
+        headers={"Authorization": "Bearer " + admin_token},
+    )
+    assert response.status_code == 200, "Remove project failed"
+    gprint("test_remove_project successful")
+
+
+
 if __name__ == "__main__":
+    test_remove_project()
     test_get_user_projects()
     test_create_user()
     test_login()
@@ -479,3 +520,4 @@ if __name__ == "__main__":
     test_list_all_users_project()
     test_change_user_name()
     test_update_weekly_report()
+