update apiCreateProject to async/await

Author: raclim

server/controllers/project.controller/createProject.js

@@ -31,33 +31,33 @@ export default function createProject(req, res) {
 }
 
 // TODO: What happens if you don't supply any files?
-export function apiCreateProject(req, res) {
+export async function apiCreateProject(req, res) {
   const params = Object.assign({ user: req.user._id }, req.body);
 
-  function sendValidationErrors(err, type, code = 422) {
+  const sendValidationErrors = (err, type, code = 422) => {
     res.status(code).json({
       message: `${type} Validation Failed`,
       detail: err.message,
       errors: err.files
     });
-  }
+  };
 
   // TODO: Error handling to match spec
-  function sendFailure(err) {
+  const sendFailure = (err) => {
     res.status(500).end();
-  }
+  };
 
-  function handleErrors(err) {
+  const handleErrors = (err) => {
     if (err instanceof FileValidationError) {
       sendValidationErrors(err, 'File', err.code);
     } else if (err instanceof ProjectValidationError) {
       sendValidationErrors(err, 'Sketch', err.code);
     } else {
       sendFailure();
     }
-  }
+  };
 
-  function checkUserHasPermission() {
+  const checkUserHasPermission = () => {
     if (req.user.username !== req.params.username) {
       console.log('no permission');
       const error = new ProjectValidationError(
@@ -67,35 +67,32 @@ export function apiCreateProject(req, res) {
 
       throw error;
     }
-  }
+  };
 
   try {
     checkUserHasPermission();
 
+    if (!params.files || typeof params.files !== 'object') {
+      const error = new FileValidationError("'files' must be an object");
+      throw error;
+    }
+
     const model = toModel(params);
 
-    return model
-      .isSlugUnique()
-      .then(({ isUnique, conflictingIds }) => {
-        if (isUnique) {
-          return model.save().then((newProject) => {
-            res.status(201).json({ id: newProject.id });
-          });
-        }
-
-        const error = new ProjectValidationError(
-          `Slug "${model.slug}" is not unique. Check ${conflictingIds.join(
-            ', '
-          )}`
-        );
-        error.code = 409;
-
-        throw error;
-      })
-      .then(checkUserHasPermission)
-      .catch(handleErrors);
+    const { isUnique, conflictingIds } = await model.isSlugUnique();
+
+    if (!isUnique) {
+      const error = new ProjectValidationError(
+        `Slug "${model.slug}" is not unique. Check ${conflictingIds.join(', ')}`
+      );
+      error.code = 409;
+
+      throw error;
+    }
+
+    const newProject = await model.save();
+    res.status(201).json({ id: newProject.id });
   } catch (err) {
     handleErrors(err);
-    return Promise.reject(err);
   }
 }