Feat: adds lesson 26 , amiyah jones' MediaItemsController #626
Conversation
Accidentally removed my work the 1st time by merging
only failed test is simply for addItems where status expects 200 but is actually 400
| return ResponseEntity.ok(response); | ||
| } | ||
|
|
||
| @PostMapping("/items") |
There was a problem hiding this comment.
My only error is that it expects the status 200 , but gets 400. Fixing this has been an issue
chatgpt expects me to manually include a book if statement , or a bindingResult <- both still gives the same 400 error.
There was a problem hiding this comment.
So you had two problems. For one, you were using the wrong @RequestBody annotation so the request message was not being read correctly. Secondly, you should have preferred to accept the CreateMediaItemRequest type which is the type of request the test anticipates sending.
anthonydmays
left a comment
anthonydmays
left a comment
There was a problem hiding this comment.
Got very close. Feel free to re-open and make changes if you wish, but closing for now.
| return ResponseEntity.ok(response); | ||
| } | ||
|
|
||
| @PostMapping("/items") |
There was a problem hiding this comment.
So you had two problems. For one, you were using the wrong @RequestBody annotation so the request message was not being read correctly. Secondly, you should have preferred to accept the CreateMediaItemRequest type which is the type of request the test anticipates sending.
2 participants
Implements the following:
GET /itemsPOST /itemsGET /items/:idDELETE /items/:id