Fix package and tests

Author: john0isaac

.devcontainer/devcontainer.json

@@ -54,7 +54,7 @@
 		}
 	},
 	// Use 'postCreateCommand' to run commands after the container is created.
-	"postCreateCommand": "pip install -r requirements-dev.txt && pip install -e src/backend && pip install -e src",
+	"postCreateCommand": "pip install -r requirements-dev.txt && pip install -e src/backend",
 	// Comment out to connect as root instead. More info: https://aka.ms/vscode-remote/containers/non-root.
 	"remoteUser": "vscode"
 }

.github/workflows/app-tests.yaml

@@ -56,7 +56,6 @@ jobs:
         - name: Install app as editable app
           run: |
             python -m pip install -e src/backend
-            python -m pip install -e src
         - name: Setup local database with seed data
           run: |
             cp .env.sample .env

README.md

@@ -129,7 +129,6 @@ Since the local app uses OpenAI models, you should first deploy it for the optim
 
     ```bash
     python3 -m pip install -e src/backend
-    python3 -m pip install -e src
     python ./src/backend/fastapi_app/setup_postgres_database.py
     python ./src/backend/fastapi_app/setup_postgres_seeddata.py
     ```

scripts/load_python_env.sh

@@ -5,5 +5,4 @@ python3 -m venv .venv
 
 echo 'Installing dependencies from "requirements.txt" into virtual environment (in quiet mode)...'
 .venv/bin/python -m pip --quiet --disable-pip-version-check install -e src/backend
-.venv/bin/python -m pip --quiet --disable-pip-version-check install -e src
 .venv/bin/python -m pip --quiet --disable-pip-version-check install -r requirements-dev.txt

src/backend/__init__.py

@@ -84,7 +84,7 @@ async def create_and_seed_db():
 @pytest_asyncio.fixture(scope="session")
 async def app(mock_session_env):
     """Create a FastAPI app."""
-    if not Path("src/static/").exists():
+    if not Path("src/backend/static/").exists():
         pytest.skip("Please generate frontend files first!")
     app = create_app(testing=True)
     await create_and_seed_db()

src/pyproject.toml

@@ -8,7 +8,7 @@ async def test_index(test_client):
     """test the index route"""
     response = test_client.get("/")
 
-    html_index_file_path = "src/static/index.html"
+    html_index_file_path = "src/backend/static/index.html"
     with open(html_index_file_path, "rb") as f:
         html_index_file = f.read()
 
@@ -23,7 +23,7 @@ async def test_favicon(test_client):
     """test the favicon route"""
     response = test_client.get("/favicon.ico")
 
-    favicon_file_path = "src/static/favicon.ico"
+    favicon_file_path = "src/backend/static/favicon.ico"
     with open(favicon_file_path, "rb") as f:
         favicon_file = f.read()
 
@@ -46,7 +46,7 @@ async def test_assets_non_existent_404(test_client):
 @pytest.mark.asyncio
 async def test_assets(test_client):
     """test the assets route with an existing file"""
-    assets_dir_path = "src/static/assets"
+    assets_dir_path = "src/backend/static/assets"
     assets_file_path = os.listdir(assets_dir_path)[0]
 
     with open(os.path.join(assets_dir_path, assets_file_path), "rb") as f: