fix resolvers so that pre remove hook runs

Author: rwieruch

src/resolvers/message.js

@@ -66,7 +66,14 @@ export default {
  isAuthenticated,
  isMessageOwner,
  async (parent, { id }, { models }) => {
- return await models.Message.findOneAndDelete({ _id: id });
+ const message = await models.Message.findById(id);
+
+ if (message) {
+ await message.remove();
+ return true;
+ } else {
+ return false;
+ }
  },
  ),
  },

src/resolvers/user.js

@@ -79,9 +79,14 @@ export default {
  deleteUser: combineResolvers(
  isAdmin,
  async (parent, { id }, { models }) => {
- return await models.User.findOneAndDelete({ _id: id }).then(
- () => true,
- );
+ const user = await models.User.findById(id);
+
+ if (user) {
+ await user.remove();
+ return true;
+ } else {
+ return false;
+ }
  },
  ),
  },