Skip to content

Commit 1eaf297

Browse files
author
inikiforov
committed
feat: add PUT /users/:id to update a user
Adds an update-user endpoint following the existing POST /users pattern: validates name and email are present (400 if not), returns 404 for an unknown id, and otherwise updates the user via a new db/store.js helper.
1 parent 19e97af commit 1eaf297

2 files changed

Lines changed: 31 additions & 1 deletion

File tree

db/store.js

Lines changed: 13 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,4 +24,16 @@ function createUser({ name, email }) {
2424
return user;
2525
}
2626

27-
module.exports = { getAllUsers, getUserById, createUser };
27+
function updateUser(id, { name, email }) {
28+
const user = getUserById(id);
29+
30+
if (!user) {
31+
return undefined;
32+
}
33+
34+
user.name = name;
35+
user.email = email;
36+
return user;
37+
}
38+
39+
module.exports = { getAllUsers, getUserById, createUser, updateUser };

routes/users.js

Lines changed: 18 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,4 +32,22 @@ router.post("/", (req, res) => {
3232
res.status(201).json(user);
3333
});
3434

35+
// PUT /users/:id — update an existing user; name and email are required
36+
router.put("/:id", (req, res) => {
37+
const { name, email } = req.body;
38+
39+
if (!name || !email) {
40+
return res.status(400).json({ error: "name and email are required" });
41+
}
42+
43+
const id = Number(req.params.id);
44+
const user = store.updateUser(id, { name, email });
45+
46+
if (!user) {
47+
return res.status(404).json({ error: "User not found" });
48+
}
49+
50+
res.json(user);
51+
});
52+
3553
module.exports = router;

0 commit comments

Comments
 (0)