diff --git a/controllers/users.js b/controllers/users.js new file mode 100644 index 0000000..ecd9e4a --- /dev/null +++ b/controllers/users.js @@ -0,0 +1,57 @@ +const users = require('../data/index'); +const sampleUser = require('../data/sampleUser'); + +// Should retrieve the entire array from _data/index_ + +const listUsers = (req, res) => { + res.json(users) +} + +// Should retrieve just the user that matches the passed-in id +const showUser = (req, res) => { + let id = req.params.id -1 + if (users[id]){ + return res.json(users[id]) + } else { + res.status(404).send({ msg: `User with the ID of ${req.params.id} not found!` }) + } +} + +// Should add a user to the array +const createUser = (req, res) => { + let counter = users.length + let newUser = sampleUser + newUser.id = counter + 1 + users.push(newUser) + return res.json(users[counter]) +} + +// Should update one user in the array based on its id +const updateUser = (req, res) => { + let targetId = req.params.id -1 + if (users[targetId]){ + users[targetId].name = sampleUser.name + return res.json(users[targetId]) + } else { + res.status(400).send({ msg: `User with the ID of ${req.params.id} not found!` }) + } +} + +// Should delete one user from the array based on its id +const deleteUser = (req, res) => { + let targetId = req.params.id -1 + if (users[targetId]){ + users[targetId].isActive = false + res.send("Deleted") + } else { + res.status(400).send({ msg: `User with the ID of ${req.params.id} not found!` }) + } +} + +module.exports = { + listUsers, + showUser, + createUser, + updateUser, + deleteUser +}; \ No newline at end of file diff --git a/index.js b/index.js index e9537d2..83990dc 100644 --- a/index.js +++ b/index.js @@ -1,9 +1,14 @@ const express = require('express') +const bodyParser = require('body-parser') const app = express() const port = process.env.PORT || 4000 +const users = require('./routes/users') app.get('/', (req, res) => res.send('default route')) +app.use(bodyParser.json()) +app.use(users) + app.listen(port, () => { console.log('app is listening on:', port) }) \ No newline at end of file diff --git a/package-lock.json b/package-lock.json index e572a2b..4ef13b4 100644 --- a/package-lock.json +++ b/package-lock.json @@ -756,7 +756,8 @@ }, "ansi-regex": { "version": "2.1.1", - "bundled": true + "bundled": true, + "optional": true }, "aproba": { "version": "1.2.0", @@ -774,11 +775,13 @@ }, "balanced-match": { "version": "1.0.0", - "bundled": true + "bundled": true, + "optional": true }, "brace-expansion": { "version": "1.1.11", "bundled": true, + "optional": true, "requires": { "balanced-match": "^1.0.0", "concat-map": "0.0.1" @@ -791,15 +794,18 @@ }, "code-point-at": { "version": "1.1.0", - "bundled": true + "bundled": true, + "optional": true }, "concat-map": { "version": "0.0.1", - "bundled": true + "bundled": true, + "optional": true }, "console-control-strings": { "version": "1.1.0", - "bundled": true + "bundled": true, + "optional": true }, "core-util-is": { "version": "1.0.2", @@ -902,7 +908,8 @@ }, "inherits": { "version": "2.0.3", - "bundled": true + "bundled": true, + "optional": true }, "ini": { "version": "1.3.5", @@ -912,6 +919,7 @@ "is-fullwidth-code-point": { "version": "1.0.0", "bundled": true, + "optional": true, "requires": { "number-is-nan": "^1.0.0" } @@ -924,17 +932,20 @@ "minimatch": { "version": "3.0.4", "bundled": true, + "optional": true, "requires": { "brace-expansion": "^1.1.7" } }, "minimist": { "version": "0.0.8", - "bundled": true + "bundled": true, + "optional": true }, "minipass": { "version": "2.3.5", "bundled": true, + "optional": true, "requires": { "safe-buffer": "^5.1.2", "yallist": "^3.0.0" @@ -951,6 +962,7 @@ "mkdirp": { "version": "0.5.1", "bundled": true, + "optional": true, "requires": { "minimist": "0.0.8" } @@ -1023,7 +1035,8 @@ }, "number-is-nan": { "version": "1.0.1", - "bundled": true + "bundled": true, + "optional": true }, "object-assign": { "version": "4.1.1", @@ -1033,6 +1046,7 @@ "once": { "version": "1.4.0", "bundled": true, + "optional": true, "requires": { "wrappy": "1" } @@ -1108,7 +1122,8 @@ }, "safe-buffer": { "version": "5.1.2", - "bundled": true + "bundled": true, + "optional": true }, "safer-buffer": { "version": "2.1.2", @@ -1138,6 +1153,7 @@ "string-width": { "version": "1.0.2", "bundled": true, + "optional": true, "requires": { "code-point-at": "^1.0.0", "is-fullwidth-code-point": "^1.0.0", @@ -1155,6 +1171,7 @@ "strip-ansi": { "version": "3.0.1", "bundled": true, + "optional": true, "requires": { "ansi-regex": "^2.0.0" } @@ -1193,11 +1210,13 @@ }, "wrappy": { "version": "1.0.2", - "bundled": true + "bundled": true, + "optional": true }, "yallist": { "version": "3.0.3", - "bundled": true + "bundled": true, + "optional": true } } }, diff --git a/routes/users.js b/routes/users.js new file mode 100644 index 0000000..5a87e39 --- /dev/null +++ b/routes/users.js @@ -0,0 +1,11 @@ +const express = require('express'); +const controller = require('../controllers/users'); +const router = express.Router(); + +router.get('/users', controller.listUsers) +router.get('/users/:id', controller.showUser) +router.post('/users', controller.createUser) +router.put('/users/:id', controller.updateUser) +router.delete('/users/:id', controller.deleteUser) + +module.exports = router; \ No newline at end of file