Skip to content

Commit

Permalink
Fix name
Browse files Browse the repository at this point in the history
Return executable's standard error as stderr, not stdout
  • Loading branch information
JanCBrammer committed Oct 25, 2024
1 parent 9fffb04 commit dd587af
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions INCHI-1-TEST/tests/test_executable/conftest.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ def pytest_addoption(parser):

@dataclass
class InchiResult:
stdout: str
stderr: str
inchi: str
aux_info: str

Expand Down Expand Up @@ -66,7 +66,7 @@ def _run_inchi_exe(molfile: str, args: str = "") -> InchiResult:
output = output_path.read_text()

return InchiResult(
stdout=result.stderr,
stderr=result.stderr, # contains log
inchi=parse_inchi_from_executable_output(output),
aux_info=parse_aux_info_from_executable_output(output),
)
Expand Down
2 changes: 1 addition & 1 deletion INCHI-1-TEST/tests/test_executable/test_github_40.py
Original file line number Diff line number Diff line change
Expand Up @@ -108,4 +108,4 @@ def molfile():
def test_spiro_compound_chiral(molfile, run_inchi_exe):
result = run_inchi_exe(molfile)

assert "Warning (Not chiral) structure #1." not in result.stdout
assert "Warning (Not chiral) structure #1." not in result.stderr
4 changes: 2 additions & 2 deletions INCHI-1-TEST/tests/test_executable/test_github_52.py
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ def test_empty_bondblock(molfile_empty_bondblock, run_inchi_exe):

assert (
"Error 71 (no InChI; Error: No V3000 CTAB end marker) inp structure #1."
not in result.stdout
not in result.stderr
)


Expand All @@ -50,5 +50,5 @@ def test_no_bondblock(molfile_no_bondblock, run_inchi_exe):

assert (
"Error 71 (no InChI; Error: No V3000 CTAB end marker) inp structure #1."
not in result.stdout
not in result.stderr
)

0 comments on commit dd587af

Please sign in to comment.