write test bloominstituteoftechnology#4 and fix some bugs to make it pass

Author: aaschuster

api/auth/auth-router.js

@@ -6,7 +6,7 @@ const {
 } = require("../middleware/checkCreds");
 const Users = require("../users/user-model");
 
-router.post('/register', checkUserAndPass, usernameNotTaken, (req, res) => {
+router.post('/register', checkUserAndPass, usernameNotTaken, (req, res, next) => {
   Users.add(req.body)
     .then( newUser => res.status(201).json(newUser))
     .catch(next);

api/server.test.js

@@ -71,4 +71,9 @@ describe("[POST] /api/auth/register", () => {
     expect(await db("users")).toHaveLength(6);
   })
 
+  test('[4] responds with newly created user', async () => {
+    let res = await request(server).post("/api/auth/register").send({username: "george", password: "pass"});
+    expect(res.body).toMatchObject({username: "george"});
+  })
+
 })

api/users/user-model.js

@@ -1,10 +1,8 @@
 db = require("../../data/dbConfig");
 
 async function add(user) {
-    db("users").insert(user)
-        .then( ([newUserID]) => {
-            return getBy({id: newUserID});
-        });
+    return db("users").insert(user)
+        .then( async ([newUserID]) =>  getBy({id: newUserID}) )
 }
 
 function getBy(filter) {