Skip to content

[Auto] Implement Delete Todo endpoint (DELETE /todos/:id)#16

Draft
landoyjx wants to merge 1 commit intomainfrom
auto/8-implement-delete-todo-endpoint--delete--todos--id
Draft

[Auto] Implement Delete Todo endpoint (DELETE /todos/:id)#16
landoyjx wants to merge 1 commit intomainfrom
auto/8-implement-delete-todo-endpoint--delete--todos--id

Conversation

@landoyjx
Copy link
Copy Markdown
Owner

Implement DELETE /todos/:id to remove a Todo item. Response should be 204 No Content on success, or 404 Not Found if the id does not exist.

Acceptance Criteria:

  • Returns 204 and deletes the Todo for an existing id
  • Returns 404 for a non-existent id
  • Subsequent GET /todos/:id for the deleted id returns 404

Test Cases:

  • Integration: DELETE /todos/:id with valid id returns 204
  • Integration: DELETE /todos/:id with non-existent id returns 404
  • Integration: Deleting a todo then calling GET /todos/:id returns 404
  • Edge: DELETE /todos/:id with malformed id returns 404 (or 400)

View issue: #8


Created by bass auto-programming

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant