public_git/bambuddy
1
0
Fork 0
mirror of https://github.com/maziggy/bambuddy.git synced 2026-10-21 01:57:41 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge branch '0.2.1b' into feature/filament-fix

Browse Source
This commit is contained in:
MartinNYHC
2026-02-18 12:23:21 +01:00
committed by GitHub
parent a649a246b2 2fd90e9dce
commit 72f5bd445f
7 changed files with 22 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files
backend/app/api/routes/filaments.py
+1 -1
View File
@@ -14,7 +14,7 @@ from backend.app.schemas.filament import (
FilamentUpdate,
)
router = APIRouter(prefix="/filaments", tags=["filaments"])
router = APIRouter(prefix="/filament-catalog", tags=["filament-catalog"])
@router.get("/", response_model=list[FilamentResponse])
backend/tests/integration/test_endpoint_auth.py
+3 -3
View File
@@ -70,7 +70,7 @@ class TestEndpointAuthenticationEnforcement:
async def test_filaments_list_accessible_without_auth_when_disabled(self, async_client: AsyncClient):
"""Verify filaments list is accessible when auth is disabled."""
with patch("backend.app.core.auth.is_auth_enabled", return_value=False):
response = await async_client.get("/api/v1/filaments/")
response = await async_client.get("/api/v1/filament-catalog/")
assert response.status_code == 200
@pytest.mark.asyncio
@@ -153,7 +153,7 @@ class TestAuthenticationPatterns:
"""Verify require_permission_if_auth_enabled allows access when auth disabled."""
with patch("backend.app.core.auth.is_auth_enabled", return_value=False):
# Test a protected endpoint
response = await async_client.get("/api/v1/filaments/")
response = await async_client.get("/api/v1/filament-catalog/")
assert response.status_code == 200
@pytest.mark.asyncio
@@ -162,7 +162,7 @@ class TestAuthenticationPatterns:
"""Verify multiple protected endpoints are accessible when auth is disabled."""
with patch("backend.app.core.auth.is_auth_enabled", return_value=False):
endpoints = [
"/api/v1/filaments/",
"/api/v1/filament-catalog/",
"/api/v1/external-links/",
"/api/v1/notifications/",
"/api/v1/maintenance/types",
backend/tests/integration/test_filaments_api.py
+9 -9
View File
@@ -5,7 +5,7 @@ from httpx import AsyncClient
class TestFilamentsAPI:
"""Integration tests for /api/v1/filaments/ endpoints."""
"""Integration tests for /api/v1/filament-catalog/ (material types) endpoints."""
@pytest.fixture
async def filament_factory(self, db_session):
@@ -36,7 +36,7 @@ class TestFilamentsAPI:
@pytest.mark.integration
async def test_list_filaments_empty(self, async_client: AsyncClient):
"""Verify empty list when no filaments exist."""
response = await async_client.get("/api/v1/filaments/")
response = await async_client.get("/api/v1/filament-catalog/")
assert response.status_code == 200
assert isinstance(response.json(), list)
@@ -45,7 +45,7 @@ class TestFilamentsAPI:
async def test_list_filaments_with_data(self, async_client: AsyncClient, filament_factory, db_session):
"""Verify list returns existing filaments."""
await filament_factory(name="Test Filament")
response = await async_client.get("/api/v1/filaments/")
response = await async_client.get("/api/v1/filament-catalog/")
assert response.status_code == 200
data = response.json()
assert any(f["name"] == "Test Filament" for f in data)
@@ -62,7 +62,7 @@ class TestFilamentsAPI:
"brand": "Bambu",
"cost_per_kg": 30.0,
}
response = await async_client.post("/api/v1/filaments/", json=data)
response = await async_client.post("/api/v1/filament-catalog/", json=data)
assert response.status_code == 200
result = response.json()
assert result["name"] == "New PETG"
@@ -73,7 +73,7 @@ class TestFilamentsAPI:
async def test_get_filament(self, async_client: AsyncClient, filament_factory, db_session):
"""Verify single filament can be retrieved."""
filament = await filament_factory(name="Get Test")
response = await async_client.get(f"/api/v1/filaments/{filament.id}")
response = await async_client.get(f"/api/v1/filament-catalog/{filament.id}")
assert response.status_code == 200
assert response.json()["name"] == "Get Test"
@@ -81,7 +81,7 @@ class TestFilamentsAPI:
@pytest.mark.integration
async def test_get_filament_not_found(self, async_client: AsyncClient):
"""Verify 404 for non-existent filament."""
response = await async_client.get("/api/v1/filaments/9999")
response = await async_client.get("/api/v1/filament-catalog/9999")
assert response.status_code == 404
@pytest.mark.asyncio
@@ -90,7 +90,7 @@ class TestFilamentsAPI:
"""Verify filament can be updated."""
filament = await filament_factory(name="Original")
response = await async_client.patch(
f"/api/v1/filaments/{filament.id}", json={"name": "Updated", "cost_per_kg": 35.0}
f"/api/v1/filament-catalog/{filament.id}", json={"name": "Updated", "cost_per_kg": 35.0}
)
assert response.status_code == 200
result = response.json()
@@ -102,8 +102,8 @@ class TestFilamentsAPI:
async def test_delete_filament(self, async_client: AsyncClient, filament_factory, db_session):
"""Verify filament can be deleted."""
filament = await filament_factory()
response = await async_client.delete(f"/api/v1/filaments/{filament.id}")
response = await async_client.delete(f"/api/v1/filament-catalog/{filament.id}")
assert response.status_code == 200
# Verify deleted
response = await async_client.get(f"/api/v1/filaments/{filament.id}")
response = await async_client.get(f"/api/v1/filament-catalog/{filament.id}")
assert response.status_code == 404