Skip to content

Commit

Permalink
Make advanced logs error test work in all test environments (#5692)
Browse files Browse the repository at this point in the history
When developing/testing in a Supervised environment, the
systemd-journal-gatewayd socket is actually available. Mock the
socket Path file to make the test independent of the pytest
environment.
  • Loading branch information
agners authored Feb 28, 2025
1 parent cf5a0dc commit f4d69f1
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions tests/api/test_host.py
Original file line number Diff line number Diff line change
Expand Up @@ -358,11 +358,13 @@ async def test_advanced_logs_formatters(
async def test_advanced_logs_errors(api_client: TestClient):
"""Test advanced logging API errors."""
# coresys = coresys_logs_control
resp = await api_client.get("/host/logs")
assert resp.content_type == "text/plain"
assert resp.status == 400
content = await resp.text()
assert content == "No systemd-journal-gatewayd Unix socket available"
with patch("supervisor.host.logs.SYSTEMD_JOURNAL_GATEWAYD_SOCKET") as socket:
socket.is_socket.return_value = False
resp = await api_client.get("/host/logs")
assert resp.content_type == "text/plain"
assert resp.status == 400
content = await resp.text()
assert content == "No systemd-journal-gatewayd Unix socket available"

headers = {"Accept": "application/json"}
resp = await api_client.get("/host/logs", headers=headers)
Expand Down

0 comments on commit f4d69f1

Please sign in to comment.