Skip to content

Commit d18dcf6

Browse files
author
Andrei Neagu
committed
extended tests
1 parent 4cbe0e1 commit d18dcf6

File tree

1 file changed

+29
-6
lines changed

1 file changed

+29
-6
lines changed

services/director-v2/tests/unit/test_api_routes_docker_networks.py

Lines changed: 29 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -40,15 +40,38 @@ def mock_env(
4040

4141

4242
@pytest.fixture
43-
async def docker_client() -> AsyncIterable[Docker]:
43+
async def async_docker_client() -> AsyncIterable[Docker]:
4444
async with Docker() as client:
4545
yield client
4646

4747

48-
async def test_routes_are_protected(client: TestClient, docker_client: Docker):
49-
network_name = "a_test_network"
50-
network = Network(name=network_name)
51-
48+
@pytest.mark.parametrize(
49+
"network_name, network",
50+
[
51+
("test_a_network", Network(name="test_a_network")),
52+
(
53+
"test_overlay_network",
54+
Network(
55+
name="test_overlay_network",
56+
driver="overlay",
57+
labels={
58+
"io.simcore.zone": "mock_value",
59+
"com.simcore.description": "interactive for node: mock_value",
60+
"uuid": "mock_value",
61+
},
62+
attachable=True,
63+
internal=False,
64+
),
65+
),
66+
],
67+
)
68+
async def test_routes_are_protected(
69+
docker_swarm: None,
70+
client: TestClient,
71+
async_docker_client: Docker,
72+
network_name: str,
73+
network: Network,
74+
):
5275
response = client.post("/v2/docker/networks/", json=network.model_dump(mode="json"))
5376
assert response.status_code == status.HTTP_200_OK, response.text
5477
network_id: DockerNetworkID = response.json()
@@ -60,6 +83,6 @@ async def test_routes_are_protected(client: TestClient, docker_client: Docker):
6083

6184
for name_or_id in (network_name, network_id):
6285
with pytest.raises(DockerError) as exc:
63-
await docker_client.networks.get(name_or_id)
86+
await async_docker_client.networks.get(name_or_id)
6487

6588
assert exc.value.status == status.HTTP_404_NOT_FOUND

0 commit comments

Comments
 (0)