Fix coverage for auth

Server/controllers/authController.js

@@ -103,7 +103,6 @@ const registerUser = async (req, res, next) => {
 			data: { user: newUser, token: token },
 		});
 	} catch (error) {
-		console.log("ERROR", error);
 		next(handleError(error, SERVICE_NAME, "registerController"));
 	}
 };

Server/controllers/monitorController.js

@@ -77,7 +77,7 @@ const getMonitorStatsById = async (req, res, next) => {
 	}
 };
 
-const getMonitorCertificate = async (req, res, next) => {
+const getMonitorCertificate = async (req, res, next, fetchMonitorCertificate) => {
 	try {
 		await getCertificateParamValidation.validateAsync(req.params);
 	} catch (error) {

Server/routes/monitorRoute.js

@@ -1,25 +1,28 @@
 import { Router } from "express";
 import {
-  getAllMonitors,
-  getMonitorStatsById,
-  getMonitorCertificate,
-  getMonitorById,
-  getMonitorsAndSummaryByTeamId,
-  getMonitorsByTeamId,
-  createMonitor,
-  deleteMonitor,
-  deleteAllMonitors,
-  editMonitor,
-  pauseMonitor,
-  addDemoMonitors,
+	getAllMonitors,
+	getMonitorStatsById,
+	getMonitorCertificate,
+	getMonitorById,
+	getMonitorsAndSummaryByTeamId,
+	getMonitorsByTeamId,
+	createMonitor,
+	deleteMonitor,
+	deleteAllMonitors,
+	editMonitor,
+	pauseMonitor,
+	addDemoMonitors,
 } from "../controllers/monitorController.js";
 import { isAllowed } from "../middleware/isAllowed.js";
+import { fetchMonitorCertificate } from "../controllers/controllerUtils.js";
 
 const router = Router();
 
 router.get("/", getAllMonitors);
 router.get("/stats/:monitorId", getMonitorStatsById);
-router.get("/certificate/:monitorId", getMonitorCertificate);
+router.get("/certificate/:monitorId", (req, res, next) => {
+	getMonitorCertificate(req, res, next, fetchMonitorCertificate);
+});
 router.get("/:monitorId", getMonitorById);
 router.get("/team/summary/:teamId", getMonitorsAndSummaryByTeamId);
 router.get("/team/:teamId", getMonitorsByTeamId);
@@ -32,11 +35,7 @@ router.put("/:monitorId", isAllowed(["admin", "superadmin"]), editMonitor);
 
 router.delete("/", isAllowed(["superadmin"]), deleteAllMonitors);
 
-router.post(
-  "/pause/:monitorId",
-  isAllowed(["admin", "superadmin"]),
-  pauseMonitor
-);
+router.post("/pause/:monitorId", isAllowed(["admin", "superadmin"]), pauseMonitor);
 
 router.post("/demo", isAllowed(["admin", "superadmin"]), addDemoMonitors);
 

Server/tests/controllers/controllerUtils.test.js

@@ -204,7 +204,7 @@ describe("controllerUtils - fetchMonitorCertificate", () => {
 		try {
 			await fetchMonitorCertificate(tls, monitor);
 		} catch (error) {
-			console.log(error);
+			expect(error.message).to.equal("Certificate error");
 		}
 	});
 });