alex
19987d68a0
Replace all AsyncClient instantiations in test_sources.py and test_jobs.py with the shared fixture from conftest.py. This ensures proper test database isolation via dependency overrides. - Remove imports of httpx AsyncClient, ASGITransport, and app - Add client fixture parameter to all test functions - Remove async with blocks around HTTP requests - test_engine.py already uses db fixture correctly, no changes needed
89 lines
2.7 KiB
Python
89 lines
2.7 KiB
Python
import pytest
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_create_source(client):
|
|
response = await client.post("/api/sources/", json={
|
|
"name": "Test Source",
|
|
"type": "local",
|
|
"config": {"path": "/tmp/test"}
|
|
})
|
|
assert response.status_code == 200
|
|
data = response.json()
|
|
assert data["name"] == "Test Source"
|
|
assert data["type"] == "local"
|
|
assert "id" in data
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_list_sources(client):
|
|
# Create source first
|
|
await client.post("/api/sources/", json={
|
|
"name": "Test Source",
|
|
"type": "local",
|
|
"config": {"path": "/tmp/test"}
|
|
})
|
|
|
|
response = await client.get("/api/sources/")
|
|
assert response.status_code == 200
|
|
data = response.json()
|
|
assert len(data) >= 1
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_get_source(client):
|
|
create_resp = await client.post("/api/sources/", json={
|
|
"name": "Test Source",
|
|
"type": "local",
|
|
"config": {"path": "/tmp/test"}
|
|
})
|
|
source_id = create_resp.json()["id"]
|
|
|
|
response = await client.get(f"/api/sources/{source_id}")
|
|
assert response.status_code == 200
|
|
assert response.json()["id"] == source_id
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_delete_source(client):
|
|
create_resp = await client.post("/api/sources/", json={
|
|
"name": "Delete Me",
|
|
"type": "local",
|
|
"config": {"path": "/tmp/test"}
|
|
})
|
|
source_id = create_resp.json()["id"]
|
|
|
|
response = await client.delete(f"/api/sources/{source_id}")
|
|
assert response.status_code == 200
|
|
|
|
# Verify deletion
|
|
get_resp = await client.get(f"/api/sources/{source_id}")
|
|
assert get_resp.status_code == 404
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_update_source(client):
|
|
create_resp = await client.post("/api/sources/", json={
|
|
"name": "Original Name",
|
|
"type": "local",
|
|
"config": {"path": "/tmp/test"}
|
|
})
|
|
source_id = create_resp.json()["id"]
|
|
|
|
response = await client.put(f"/api/sources/{source_id}", json={
|
|
"name": "Updated Name"
|
|
})
|
|
assert response.status_code == 200
|
|
assert response.json()["name"] == "Updated Name"
|
|
assert response.json()["type"] == "local" # Unchanged
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_get_source_not_found(client):
|
|
response = await client.get("/api/sources/99999")
|
|
assert response.status_code == 404
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_update_source_not_found(client):
|
|
response = await client.put("/api/sources/99999", json={"name": "Test"})
|
|
assert response.status_code == 404
|
|
|
|
@pytest.mark.asyncio
|
|
async def test_delete_source_not_found(client):
|
|
response = await client.delete("/api/sources/99999")
|
|
assert response.status_code == 404
|