Resolve failing tests

Author: tanzilahmed0

backend/api/projects.py

@@ -185,6 +185,9 @@ async def get_project(
 
     except ValueError as e:
         raise HTTPException(status_code=400, detail=f"Invalid project ID: {str(e)}")
+    except HTTPException:
+        # Re-raise HTTPExceptions without wrapping them
+        raise
     except Exception as e:
         raise HTTPException(
             status_code=500, detail=f"Failed to fetch project: {str(e)}"
@@ -217,6 +220,9 @@ async def delete_project(
 
     except ValueError as e:
         raise HTTPException(status_code=400, detail=f"Invalid project ID: {str(e)}")
+    except HTTPException:
+        # Re-raise HTTPExceptions without wrapping them
+        raise
     except Exception as e:
         raise HTTPException(
             status_code=500, detail=f"Failed to delete project: {str(e)}"
@@ -255,6 +261,9 @@ async def get_upload_url(
 
     except ValueError as e:
         raise HTTPException(status_code=400, detail=f"Invalid project ID: {str(e)}")
+    except HTTPException:
+        # Re-raise HTTPExceptions without wrapping them
+        raise
     except Exception as e:
         raise HTTPException(
             status_code=500, detail=f"Failed to generate upload URL: {str(e)}"
@@ -309,6 +318,9 @@ async def get_project_status(
 
     except ValueError as e:
         raise HTTPException(status_code=400, detail=f"Invalid project ID: {str(e)}")
+    except HTTPException:
+        # Re-raise HTTPExceptions without wrapping them
+        raise
     except Exception as e:
         raise HTTPException(
             status_code=500, detail=f"Failed to fetch project status: {str(e)}"

backend/tests/conftest.py

@@ -1,4 +1,5 @@
 import os
+from unittest.mock import Mock, patch
 
 import pytest
 from fastapi.testclient import TestClient
@@ -8,8 +9,22 @@
 os.environ["JWT_SECRET"] = "test_secret"
 os.environ["TESTING"] = "true"
 
-# Now that the environment is configured, we can import the application
-from main import app
+# Mock the storage service BEFORE importing the app to prevent MinIO connection attempts
+mock_storage = Mock()
+mock_storage.connect.return_value = True
+mock_storage.generate_presigned_url.return_value = (
+    "https://test-minio/test-bucket/test-file?presigned=true"
+)
+mock_storage.health_check.return_value = {
+    "status": "healthy",
+    "message": "MinIO storage is operational (mocked)",
+    "bucket": "test-bucket",
+}
+
+# Apply the mock before importing the app
+with patch("services.storage_service.storage_service", mock_storage):
+    # Now that the environment is configured, we can import the application
+    from main import app
 from models.base import Base
 from models.project import ProjectTable  # Import to register with Base
 from models.user import UserTable  # Import to register with Base
@@ -44,7 +59,16 @@ def test_db_setup():
 
 
 @pytest.fixture(scope="function")
-def test_client(test_db_setup):
+def mock_storage_service():
+    """Mock the storage service to avoid MinIO connection in tests"""
+    # The storage service is already mocked at import time, just return the mock
+    from services.storage_service import storage_service
+
+    return storage_service
+
+
+@pytest.fixture(scope="function")
+def test_client(test_db_setup, mock_storage_service):
     """
     A TestClient that uses the in-memory SQLite database.
     Each test function gets a clean database.

backend/tests/test_mock_endpoints.py

@@ -141,7 +141,9 @@ def test_get_projects(
         app.dependency_overrides.clear()
 
 
-def test_create_project(test_client, test_access_token, test_user_in_db):
+def test_create_project(
+    test_client, test_access_token, test_user_in_db, mock_storage_service
+):
     """Test create project endpoint"""
     app.dependency_overrides[verify_token] = mock_verify_token
     try:
@@ -303,7 +305,11 @@ def test_project_status(
 
 
 def test_get_upload_url(
-    test_client, test_access_token, test_user_in_db, test_project_in_db
+    test_client,
+    test_access_token,
+    test_user_in_db,
+    test_project_in_db,
+    mock_storage_service,
 ):
     """Test get upload URL endpoint"""
     app.dependency_overrides[verify_token] = mock_verify_token
@@ -352,7 +358,9 @@ def test_invalid_google_token(test_client):
         assert response.status_code == 401
 
 
-def test_project_not_found(test_client, test_access_token, test_user_in_db):
+def test_project_not_found(
+    test_client, test_access_token, test_user_in_db, mock_storage_service
+):
     """Test project not found error"""
     app.dependency_overrides[verify_token] = mock_verify_token
     try: