Skip to content

Commit 75e481c

Browse files
authored
Fix hardcoded OS path in edge provider (#67039)
1 parent ac39596 commit 75e481c

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

  • providers/edge3/tests/unit/edge3/worker_api/routes

providers/edge3/tests/unit/edge3/worker_api/routes/test_logs.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,7 @@
1717
from __future__ import annotations
1818

1919
from collections.abc import Sequence
20+
from pathlib import Path
2021
from typing import TYPE_CHECKING
2122

2223
import pytest
@@ -53,8 +54,8 @@ def setup_test_cases(self, dag_maker, session: Session):
5354
def test_logfile_path(self, session: Session):
5455
p: str = logfile_path(dag_id=DAG_ID, task_id=TASK_ID, run_id=RUN_ID, try_number=1, map_index=-1)
5556
assert p
56-
assert f"dag_id={DAG_ID}/run_id={RUN_ID}/task_id={TASK_ID}/attempt=1" in p
57-
assert "/-1" not in p
57+
assert str(Path(f"dag_id={DAG_ID}") / f"run_id={RUN_ID}" / f"task_id={TASK_ID}" / "attempt=1") in p
58+
assert "-1" not in Path(p).parts
5859

5960
def test_push_logs(self, session: Session):
6061
log_data = PushLogsBody(

0 commit comments

Comments
 (0)