[added] end point for delete infrastructure and its apidoc

_apidoc.js

@@ -18,11 +18,11 @@
 // History.
 // ------------------------------------------------------------------------------------------
 /**
- * @api {delete} /infrastructure/delete/:documentID Delete Infrastructure
+ * @api {delete} /infrastructure/delete/:infrastructureId Delete Infrastructure
  * @apiName DeleteInfrastructure
  * @apiGroup Infrastructure
  *
- * @apiParam {String} documentID The ID of the infrastructure document to delete.
+ * @apiParam {String} infrastructureId The ID of the infrastructure document to delete.
  *
  * @apiSuccess {String} res Success message indicating the deletion.
  *

controller/infrastructure.js

@@ -1,4 +1,4 @@
-import { createInfrastructure, deleteInfrastructure } from "#services/infrastructure";
+import { createInfrastructure, deleteInfrastructureById } from "#services/infrastructure";
 import { logger } from "#util";
 
 async function addinfrastructure(req, res) {
@@ -15,15 +15,15 @@ async function addinfrastructure(req, res) {
   }
 }
 
-async function deleteinfrastructure(req, res) {
-  const { documentID } = req.params;
+async function deleteInfrastructure(req, res) {
+  const { infrastructureId } = req.params;
   try {
-    await deleteInfrastructure(documentID);
+    await deleteInfrastructureById(infrastructureId);
 
-    res.json({ res: `Deleted infrastructure with ID ${documentID}` });
+    res.json({ res: `Deleted infrastructure with ID ${infrastructureId}` });
   } catch (error) {
     logger.error("Error while deleting", error);
     res.status(500).json({ error: "Error while deleting from DB" });
   }
 }
-export default { addinfrastructure, deleteinfrastructure };
+export default { addinfrastructure, deleteInfrastructure };

error/DataDeleteError.js

@@ -0,0 +1,6 @@
+export class DataDeleteError extends Error {
+    constructor(modelName) {
+      super(`Error while deleting document in ${modelName}`);
+      this.name = "DataDeleteError";
+    }
+  }

error/database.js

@@ -2,7 +2,8 @@ import { DataEntryError } from "#error/DataEntryError";
 import { DataNotFoundError } from "#error/DataNotFoundError";
 import { UpdateError } from "#error/UpdateError";
 import { UserDoesNotExistError } from "#error/UserDoesNotExist";
+import { DataDeleteError } from "#error/DataDeleteError";
 
 export default {
-  DataEntryError, DataNotFoundError, UpdateError, UserDoesNotExistError,
+  DataEntryError, DataNotFoundError, UpdateError, UserDoesNotExistError,DataDeleteError,
 };

routes/infrastructure.js

@@ -3,6 +3,6 @@ import infrastructureController from "#controller/infrastructure";
 
 const router = express.Router();
 router.post("/add", infrastructureController.addinfrastructure);
-router.post("/delete/:documentID", infrastructureController.deleteinfrastructure);
+router.post("/delete/:infrastructureId", infrastructureController.deleteInfrastructure);
 
 export default router;

services/infrastructure.js

@@ -11,11 +11,11 @@ export async function createInfrastructure(name, type, wing, floor, capacity) {
   throw new databaseError.DataEntryError("infrastructure");
 }
 
-export async function deleteInfrastructure(documentID) {
-  try {
-    await infrastructure.findOneAndDelete({ _id: documentID });
-  } catch (error) {
-    // Handle any errors that occur during the deletion process
-    throw new Error(`Error deleting infrastructure: ${error.message}`);
+export async function deleteInfrastructureById(infrastructureId) {
+  const deleted =  await infrastructure.remove({ _id: infrastructureId });
+  if (deleted){
+    return deleted;
   }
+  throw new databaseError.DataDeleteError("infrastructure");
+
 }