fix: Correct static directory path for StaticFiles mount

Author: RyanL2004

src/main.py

@@ -156,7 +156,12 @@ async def rate_limit_exception_handler(request: Request, exc: Exception) -> Resp
 app.add_exception_handler(RateLimitExceeded, rate_limit_exception_handler)
 
 # Mount static files to serve CSS, JS, and other static assets
-app.mount("/static", StaticFiles(directory="src/static"), name="static")
+# Mount static files dynamically
+static_dir = Path(__file__).parent / "static"
+if static_dir.exists():
+    app.mount("/static", StaticFiles(directory=static_dir), name="static")
+else:
+    print(f"Warning: Static directory '{static_dir}' does not exist. Skipping static file mount.")
 
 # Set up API analytics middleware if an API key is provided
 if app_analytics_key := os.getenv("API_ANALYTICS_KEY"):
@@ -175,7 +180,7 @@ async def rate_limit_exception_handler(request: Request, exc: Exception) -> Resp
 app.add_middleware(TrustedHostMiddleware, allowed_hosts=allowed_hosts)
 
 # Set up template rendering
-templates = Jinja2Templates(directory="src/templates")
+templates = Jinja2Templates(directory="templates")
 
 
 @app.get("/health")

tests/test_flow_integration.py

@@ -11,7 +11,7 @@
 import pytest
 from fastapi.testclient import TestClient
 
-from main import app
+from src.main import app
 
 BASE_DIR = Path(__file__).resolve().parent.parent
 TEMPLATE_DIR = BASE_DIR / "src" / "templates"
@@ -20,9 +20,16 @@
 @pytest.fixture(scope="module")
 def test_client():
     """Create a test client fixture."""
-    with TestClient(app) as client:
-        client.headers.update({"Host": "localhost"})
-        yield client
+    with TestClient(app) as client_instance:
+        client_instance.headers.update({"Host": "localhost"})
+        yield client_instance
+
+
+@pytest.fixture(scope="module", autouse=True)
+def mock_static_files():
+    """Mock the static file mount to avoid directory errors."""
+    with patch("src.main.StaticFiles") as mock_static:
+        yield mock_static
 
 
 @pytest.fixture(scope="module", autouse=True)
@@ -50,53 +57,57 @@ def cleanup():
 
 
 @pytest.mark.asyncio
-async def test_remote_repository_analysis(test_client):  # pylint: disable=redefined-outer-name
+async def test_remote_repository_analysis(request):
     """Test the complete flow of analyzing a remote repository."""
+    client = request.getfixturevalue("test_client")
     form_data = {
         "input_text": "https://github.com/octocat/Hello-World",
         "max_file_size": "243",
         "pattern_type": "exclude",
         "pattern": "",
     }
 
-    response = test_client.post("/", data=form_data)
+    response = client.post("/", data=form_data)
     assert response.status_code == 200, f"Form submission failed: {response.text}"
     assert "Mocked Template Response" in response.text
 
 
 @pytest.mark.asyncio
-async def test_invalid_repository_url(test_client):  # pylint: disable=redefined-outer-name
+async def test_invalid_repository_url(request):
     """Test handling of an invalid repository URL."""
+    client = request.getfixturevalue("test_client")
     form_data = {
         "input_text": "https://github.com/nonexistent/repo",
         "max_file_size": "243",
         "pattern_type": "exclude",
         "pattern": "",
     }
 
-    response = test_client.post("/", data=form_data)
+    response = client.post("/", data=form_data)
     assert response.status_code == 200, f"Request failed: {response.text}"
     assert "Mocked Template Response" in response.text
 
 
 @pytest.mark.asyncio
-async def test_large_repository(test_client):  # pylint: disable=redefined-outer-name
+async def test_large_repository(request):
     """Simulate analysis of a large repository with nested folders."""
+    client = request.getfixturevalue("test_client")
     form_data = {
         "input_text": "https://github.com/large/repo-with-many-files",
         "max_file_size": "243",
         "pattern_type": "exclude",
         "pattern": "",
     }
 
-    response = test_client.post("/", data=form_data)
+    response = client.post("/", data=form_data)
     assert response.status_code == 200, f"Request failed: {response.text}"
     assert "Mocked Template Response" in response.text
 
 
 @pytest.mark.asyncio
-async def test_concurrent_requests(test_client):  # pylint: disable=redefined-outer-name
+async def test_concurrent_requests(request):
     """Test handling of multiple concurrent requests."""
+    client = request.getfixturevalue("test_client")
 
     def make_request():
         form_data = {
@@ -105,7 +116,7 @@ def make_request():
             "pattern_type": "exclude",
             "pattern": "",
         }
-        response = test_client.post("/", data=form_data)
+        response = client.post("/", data=form_data)
         assert response.status_code == 200, f"Request failed: {response.text}"
         assert "Mocked Template Response" in response.text
 
@@ -116,30 +127,32 @@ def make_request():
 
 
 @pytest.mark.asyncio
-async def test_large_file_handling(test_client):  # pylint: disable=redefined-outer-name
+async def test_large_file_handling(request):
     """Test handling of repositories with large files."""
+    client = request.getfixturevalue("test_client")
     form_data = {
         "input_text": "https://github.com/octocat/Hello-World",
         "max_file_size": "1",
         "pattern_type": "exclude",
         "pattern": "",
     }
 
-    response = test_client.post("/", data=form_data)
+    response = client.post("/", data=form_data)
     assert response.status_code == 200, f"Request failed: {response.text}"
     assert "Mocked Template Response" in response.text
 
 
 @pytest.mark.asyncio
-async def test_repository_with_patterns(test_client):  # pylint: disable=redefined-outer-name
+async def test_repository_with_patterns(request):
     """Test repository analysis with include/exclude patterns."""
+    client = request.getfixturevalue("test_client")
     form_data = {
         "input_text": "https://github.com/octocat/Hello-World",
         "max_file_size": "243",
         "pattern_type": "include",
         "pattern": "*.md",
     }
 
-    response = test_client.post("/", data=form_data)
+    response = client.post("/", data=form_data)
     assert response.status_code == 200, f"Request failed: {response.text}"
     assert "Mocked Template Response" in response.text