Skip to content

Commit 12816d3

Browse files
kannon92mr-c
andcommitted
Update tests/test_stdout_stderr_log_dir.py
Co-authored-by: Michael R. Crusoe <[email protected]>
1 parent d4e7a64 commit 12816d3

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

tests/test_stdout_stderr_log_dir.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,12 +7,14 @@
77
from .util import get_data, get_main_output, needs_docker
88

99

10-
def test_log_dir_echo_output() -> None:
10+
def test_log_dir_echo_output(tmp_path: Path) -> None:
1111
_, stdout, stderr = get_main_output(
12-
["--log-dir", "logs", get_data("tests/echo.cwl"), "--inp", "hello"]
12+
["--log-dir", str(tmp_path), get_data("tests/echo.cwl"), "--inp", "hello"]
1313
)
1414
assert "completed success" in stderr, stderr
1515
assert json.loads(stdout)["out"].strip("\n") == "hello"
16+
# and then check in `tmp_path` that there is no stdout log file, since `echo.cwl` uses `stdout` itself.
17+
# should there be an empty stderr log, though?
1618

1719

1820
def test_log_dir_echo_no_output() -> None:

0 commit comments

Comments
 (0)