Skip to content

Commit 4d16f4d

Browse files
committed
updating tests to match new response formats
1 parent 2519eaa commit 4d16f4d

2 files changed

Lines changed: 163 additions & 113 deletions

File tree

mflix/server/python-fastapi/tests/integration/test_movie_routes_integration.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -126,8 +126,8 @@ async def test_delete_movie(self, client, test_movie_data):
126126
get_response = await client.get(f"/api/movies/{movie_id}")
127127
assert get_response.status_code == 404
128128
error_data = get_response.json()
129-
assert "detail" in error_data
130-
assert "no movie found" in error_data["detail"].lower()
129+
assert error_data["success"] is False
130+
assert error_data["error"]["code"] == "MOVIE_NOT_FOUND"
131131

132132
# No cleanup needed - movie already deleted
133133

@@ -276,13 +276,12 @@ async def test_batch_delete_movies(self, client, multiple_test_movies):
276276
assert delete_data["data"]["deletedCount"] == 3
277277

278278
# Verify all movies were deleted
279-
# Note: The API returns 200 with INTERNAL_SERVER_ERROR code, not 404
280279
for movie_id in multiple_test_movies:
281280
get_response = await client.get(f"/api/movies/{movie_id}")
282281
assert get_response.status_code == 404
283282
error_data = get_response.json()
284-
assert "detail" in error_data
285-
assert "no movie found" in error_data["detail"].lower()
283+
assert error_data["success"] is False
284+
assert error_data["error"]["code"] == "MOVIE_NOT_FOUND"
286285

287286
# Note: Fixture cleanup will try to delete but movies are already gone
288287
# The fixture should handle this gracefully

0 commit comments

Comments
 (0)