Skip to content

Commit ff91a4f

Browse files
author
pietro convalle
committed
fixed rebase error
1 parent 9d745ee commit ff91a4f

3 files changed

Lines changed: 5 additions & 46 deletions

File tree

answerking_app/tests/test_categories.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -387,8 +387,7 @@ def test_put_not_existing_item_returns_not_found(self):
387387
"items": [item],
388388
}
389389
expected: DetailError = {
390-
"detail": "Object was not Found",
391-
"errors": ["Item matching query does not exist."],
390+
"detail": "Not Found",
392391
"status": 404,
393392
"title": "Resource not found",
394393
"type": "http://testserver/problems/error/",

answerking_app/tests/test_orderlines.py

Lines changed: 2 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,8 @@
11
from django.test import Client, TestCase
22

33
from answerking_app.models.models import Item, Order, Status
4-
from answerking_app.utils.model_types import (
5-
DetailError,
6-
OrderItemType,
7-
OrderType,
8-
)
4+
from answerking_app.utils.model_types import (DetailError, OrderItemType,
5+
OrderType)
96

107
client = Client()
118

@@ -329,40 +326,3 @@ def test_delete_nonexistant_id_returns_not_found(self):
329326
self.assertIsInstance(actual.pop("traceId"), str)
330327
self.assertEqual(expected, actual)
331328
self.assertEqual(response.status_code, 404)
332-
333-
def test_delete_invalid_id_returns_not_found(self):
334-
# Arrange
335-
expected: DetailError = {
336-
"detail": "Not found.",
337-
"status": 404,
338-
"title": "Not found.",
339-
"type": "http://testserver/problems/not_found/",
340-
}
341-
342-
# Act
343-
response = client.delete(
344-
f"/api/orders/{self.test_order_1.id}/orderline/100000"
345-
)
346-
actual = response.json()
347-
348-
# Assert
349-
self.assertEqual(expected, actual)
350-
self.assertEqual(response.status_code, 404)
351-
352-
def test_delete_item_when_in_order_returns_bad_request(self):
353-
# Arrange
354-
expected: DetailError = {
355-
"detail": "A server error occurred.",
356-
"status": 400,
357-
"title": "A server error occurred.",
358-
"type": "http://testserver/problems/error/",
359-
}
360-
361-
# Act
362-
response = client.delete(f"/api/items/{self.test_item_1.id}")
363-
actual = response.json()
364-
365-
# Assert
366-
self.assertIsInstance(actual.pop("traceId"), str)
367-
self.assertEqual(expected, actual)
368-
self.assertEqual(response.status_code, 400)

answerking_app/utils/mixins/SerializeErrorDetailRFCMixins.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,14 +11,14 @@
1111
def ValidationErrorDetailed(exc: ValidationError | ParseError):
1212
if isinstance(exc, ValidationError):
1313
return HttpErrorResponse(
14-
status=status.HTTP_404_NOT_FOUND,
14+
status=status.HTTP_400_BAD_REQUEST,
1515
detail="Validation Error",
1616
title="Invalid input.",
1717
extensions={"errors": exc.detail},
1818
)
1919
elif isinstance(exc, ParseError):
2020
return HttpErrorResponse(
21-
status=status.HTTP_404_NOT_FOUND,
21+
status=status.HTTP_400_BAD_REQUEST,
2222
detail="Parsing JSON Error",
2323
title="Invalid input json.",
2424
extensions={"errors": exc.detail},

0 commit comments

Comments
 (0)