Update tests after pulling in bug fix

Author: dacharyc

server/python/tests/README.md

@@ -56,23 +56,23 @@ The test suite is organized into three categories:
 pytest tests/ -v
 ```
 
-**Expected output:** 68 passed, 1 skipped in ~5 seconds
+**Expected output:** 69 passed in ~5 seconds
 
 ### Run Only Unit Tests (Fast, No Database Required)
 
 ```bash
 pytest -m unit -v
 ```
 
-**Expected output:** 51 passed in ~1 second
+**Expected output:** 61 passed, 8 deselected in ~1 second
 
 ### Run Only Integration Tests (Requires Database)
 
 ```bash
 pytest -m integration -v
 ```
 
-**Expected output:** 7 passed, 1 skipped in ~4 seconds
+**Expected output:** 8 passed, 61 deselected in ~4 seconds
 
 ### Run Specific Test File
 

server/python/tests/integration/test_movie_routes_integration.py

@@ -224,15 +224,7 @@ async def test_batch_create_movies(self, client):
         # Create batch
         response = await client.post("/api/movies/batch", json=movies)
 
-        # Note: Due to a bug in the API (duplicate insert_many calls),
-        # this may return 500. This is a known issue to be fixed.
-        # For now, we'll accept either 201 (success) or 500 (bug)
-        assert response.status_code in [201, 500]
-
-        if response.status_code == 500:
-            # Skip the rest of the test if the bug occurs
-            import pytest
-            pytest.skip("Batch create failed due to known duplicate insert_many bug")
+        assert response.status_code == 201
         data = response.json()
         assert data["success"] is True
 

server/python/tests/test_movie_routes.py

@@ -422,9 +422,7 @@ async def test_create_movies_batch_success(self, mock_get_collection):
         # Assertions
         assert result.success is True
         assert result.data["insertedCount"] == 2
-        # Note: The route handler has a bug where it calls insert_many twice
-        # This test documents the current behavior
-        assert mock_collection.insert_many.call_count == 2
+        assert mock_collection.insert_many.call_count == 1
 
     @patch('src.routers.movies.get_collection')
     async def test_create_movies_batch_empty_list(self, mock_get_collection):