Merge pull request #531 from bluewave-labs/fix/delete-user

Added empty query to request object

Server/controllers/authController.js

@@ -439,7 +439,9 @@ const deleteUserController = async (req, res, next) => {
     const decodedToken = jwt.decode(token);
     const { _id, email } = decodedToken;
 
+    // TODO fix this hack
     const decodedTokenCastedAsRequest = {
+      query: {},
       params: {
         userId: _id,
       },
@@ -480,6 +482,7 @@ const deleteUserController = async (req, res, next) => {
           await req.jobQueue.deleteJob(monitor);
           await req.db.deleteChecks(monitor._id);
           await req.db.deleteAlertByMonitorId(monitor._id);
+          await req.db.deletePageSpeedChecksByMonitorId(monitor._id);
           await req.db.deleteNotificationsByMonitorId(monitor._id);
         })
       );

Server/controllers/monitorController.js

@@ -182,6 +182,7 @@ const deleteMonitor = async (req, res, next) => {
     await req.jobQueue.deleteJob(monitor);
     await req.db.deleteChecks(monitor._id);
     await req.db.deleteAlertByMonitorId(monitor._id);
+    await req.db.deletePageSpeedChecksByMonitorId(monitor._id);
     await req.db.deleteNotificationsByMonitorId(monitor._id);
 
     /**

Server/db/mongo/MongoDB.js

@@ -79,7 +79,7 @@ const {
 const {
   createPageSpeedCheck,
   getPageSpeedChecks,
-  deletePageSpeedChecks,
+  deletePageSpeedChecksByMonitorId,
 } = require("./modules/pageSpeedCheckModule");
 
 //****************************************
@@ -157,7 +157,7 @@ module.exports = {
   deleteMonitorsByUserId,
   createPageSpeedCheck,
   getPageSpeedChecks,
-  deletePageSpeedChecks,
+  deletePageSpeedChecksByMonitorId,
   createMaintenanceWindow,
   getMaintenanceWindowsByUserId,
   getMaintenanceWindowsByMonitorId,

Server/db/mongo/modules/pageSpeedCheckModule.js

@@ -48,7 +48,7 @@ const getPageSpeedChecks = async (monitorId) => {
  * @throws {Error}
  */
 
-const deletePageSpeedChecks = async (monitorId) => {
+const deletePageSpeedChecksByMonitorId = async (monitorId) => {
   try {
     const result = await PageSpeedCheck.deleteMany({ monitorId });
     return result.deletedCount;
@@ -60,5 +60,5 @@ const deletePageSpeedChecks = async (monitorId) => {
 module.exports = {
   createPageSpeedCheck,
   getPageSpeedChecks,
-  deletePageSpeedChecks,
+  deletePageSpeedChecksByMonitorId,
 };