Skip to content

Commit

Permalink
Fix the correct xml form for stderr in file_reporter
Browse files Browse the repository at this point in the history
  • Loading branch information
xjules committed Nov 28, 2023
1 parent ff3a55b commit e48db18
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions tests/unit_tests/job_runner/test_file_reporter.py
Original file line number Diff line number Diff line change
Expand Up @@ -94,7 +94,6 @@ def test_report_with_successful_exit_message_argument(reporter):
assert '"status": "Success"' in content, "status.json missing Success status"


@pytest.mark.xfail(reason="Needs reimplementation")
@pytest.mark.usefixtures("use_tmpdir")
def test_report_with_failed_exit_message_argument(reporter):
msg = Exited(Job({"name": "job1"}, 0), 1).with_error("massive_failure")
Expand All @@ -111,7 +110,7 @@ def test_report_with_failed_exit_message_argument(reporter):
"<reason>massive_failure</reason>" in content
), "ERROR file missing reason"
assert (
"<stderr: Not redirected>" in content
"<stderr>\nstderr: Not redirected\n</stderr>" in content
), "ERROR had invalid stderr information"
with open(STATUS_json, "r", encoding="utf-8") as f:
content = "".join(f.readlines())
Expand Down

0 comments on commit e48db18

Please sign in to comment.