minor: remove extra decorators

These were duplicates and had no effect.
This commit is contained in:
Chris Milne 2026-06-03 14:28:40 +01:00
parent f44c5b6f36
commit 1b54918bb4

View file

@ -9,82 +9,79 @@ from .conftest import default_client
@pytest.mark.anyio @pytest.mark.anyio
async def test_get_services_success(default_client: AsyncClient): async def test_get_services_success(default_client: AsyncClient):
resp = await default_client.get("/service/?org_id=1") resp = await default_client.get("/service/?org_id=1")
data = resp.json() data = resp.json()
assert resp.status_code == 200 assert resp.status_code == 200
assert "services" in data assert "services" in data
assert data["services"][0]["id"] == 1 assert data["services"][0]["id"] == 1
assert data["services"][0]["name"] == "Test Service" assert data["services"][0]["name"] == "Test Service"
@pytest.mark.anyio
@pytest.mark.parametrize( @pytest.mark.parametrize(
"query, expected_status", "query, expected_status",
[ [
("org_id=2", 404), ("org_id=2", 404),
("org_id=banana", 422), ("org_id=banana", 422),
("", 422), ("", 422),
], ],
) )
@pytest.mark.anyio @pytest.mark.anyio
async def test_get_services_failure(default_client: AsyncClient, query: str, expected_status: int): async def test_get_services_failure(default_client: AsyncClient, query: str, expected_status: int):
resp = await default_client.get(f"/service/?{query}") resp = await default_client.get(f"/service/?{query}")
assert resp.status_code == expected_status assert resp.status_code == expected_status
@pytest.mark.anyio @pytest.mark.anyio
async def test_post_service_success(default_client: AsyncClient): async def test_post_service_success(default_client: AsyncClient):
resp = await default_client.post("/service/", json={"name": "New Test Service"}) resp = await default_client.post("/service/", json={"name": "New Test Service"})
data = resp.json() data = resp.json()
assert resp.status_code == 200 assert resp.status_code == 200
assert "service" in data assert "service" in data
assert data["service"]["name"] == "New Test Service" assert data["service"]["name"] == "New Test Service"
assert data["service"]["id"] == 2 assert data["service"]["id"] == 2
assert type(data["service"]["api_key"]) == str assert type(data["service"]["api_key"]) == str
@pytest.mark.anyio
@pytest.mark.parametrize( @pytest.mark.parametrize(
"body, expected_status", "body, expected_status",
[ [
({"name": 42}, 422), ({"name": 42}, 422),
({}, 422), ({}, 422),
], ],
) )
@pytest.mark.anyio @pytest.mark.anyio
async def test_post_services_failure(default_client: AsyncClient, body: dict[str, str], expected_status: int): async def test_post_services_failure(default_client: AsyncClient, body: dict[str, str], expected_status: int):
resp = await default_client.post("/service/", json=body) resp = await default_client.post("/service/", json=body)
assert resp.status_code == expected_status assert resp.status_code == expected_status
@pytest.mark.anyio @pytest.mark.anyio
async def test_patch_service_success(default_client: AsyncClient): async def test_patch_service_success(default_client: AsyncClient):
resp = await default_client.patch("/service/key", json={"service_id": 1}) resp = await default_client.patch("/service/key", json={"service_id": 1})
data = resp.json() data = resp.json()
assert resp.status_code == 200 assert resp.status_code == 200
assert "service" in data assert "service" in data
assert data["service"]["name"] == "Test Service" assert data["service"]["name"] == "Test Service"
assert data["service"]["id"] == 1 assert data["service"]["id"] == 1
assert type(data["service"]["api_key"]) == str assert type(data["service"]["api_key"]) == str
@pytest.mark.anyio
@pytest.mark.parametrize( @pytest.mark.parametrize(
"body, expected_status", "body, expected_status",
[ [
({"service_id": 42}, 404), ({"service_id": 42}, 404),
({"service_id": "Test Service"}, 422), ({"service_id": "Test Service"}, 422),
({"service_id": ""}, 422), ({"service_id": ""}, 422),
({}, 422), ({}, 422),
], ],
) )
@pytest.mark.anyio @pytest.mark.anyio
async def test_patch_services_failure(default_client: AsyncClient, body: dict[str, str], expected_status: int): async def test_patch_services_failure(default_client: AsyncClient, body: dict[str, str], expected_status: int):
resp = await default_client.patch("/service/key", json=body) resp = await default_client.patch("/service/key", json=body)
assert resp.status_code == expected_status assert resp.status_code == expected_status