refactor out static function throwError to dry code

Author: Jrschellenberg

src/middleware/index.js

@@ -1,19 +1,16 @@
 const auth = require('basic-auth');
 const User = require('../models/user');
+import Utils from '../utils';
 
 
 export function authorizeUser(req, res, next) {
 	let user = auth(req);
 	if(!user || !user.name || !user.pass){
-		const err = new Error("Access Denied: Please supply login credentials!");
-		err.status = 401;
-		return next(err);
+		return Utils.throwError(401, "Access Denied: Please supply login credentials!", next);
 	}
 	User.authenticate(user.name, user.pass, (error, user) => {
 		if(error || !user){
-			const err = new Error("Access Denied: Wrong email or password");
-			err.status = 401;
-			return next(err);
+			return Utils.throwError(401, "Access Denied: Wrong email or password", next);
 		}
 		req.session.userId = user._id;
 		return next();

src/models/user.js

@@ -43,7 +43,6 @@ UserSchema.statics.authenticate = function (email, password, callback) {
 		});
 };
 
-
 /*
 May need some logic here later to check if email in correct format.
  */

src/routes/user.js

@@ -2,25 +2,25 @@ const express = require('express');
 const router = express.Router();
 const User = require('../models/user');
 import {authorizeUser} from "../middleware/index";
+import Utils from '../utils';
 
 /* GET home page. */
-router.get('/', authorizeUser, (req, res, next) => {
-	console.log(req.session.userId);
+router.get('/', authorizeUser, (req, res) => {
+	// if(!req.session || !req.session.userId){
+	// 	Utils.throwError()
+	// }
+	// User.findById(req.session.userId)
 	let status = 200;
 	res.status(status).json({success: true, message: "User Successfully retrieved", status: status});
 });
 
 
 router.post('/', (req, res, next) => {
 	if(!req.body.emailAddress || !req.body.fullName || !req.body.password){
-		const err = new Error("Missing Parameters");
-		err.status = 422;
-		return next(err);
+		return Utils.throwError(422, "Missing Parameters", next);
 	}
 	if(!User.validEmail(req.body.emailAddress)){
-		const err = new Error("Malformed Email Supplied");
-		err.status = 400;
-		return next(err);
+		return Utils.throwError(400, "Malformed Email Supplied", next);
 	}
 	User.userExist(req.body.emailAddress, (err) => {
 		if(err){

src/utils.js

@@ -0,0 +1,8 @@
+export default class Utils {
+	constructor(){}
+	static throwError(status, msg, next){
+		const err = new Error(msg);
+		err.status = status;
+		return next(err);
+	}
+}