Skip to content

Commit 16f8c42

Browse files
committed
Fix the test_script_is_runnable, which is now looking in the wrong place for the message. Since we are sending error messages to stderr, we cannot use stdout.
1 parent f5f8691 commit 16f8c42

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

tests/test_game_parsing.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,10 @@ def test_script_is_runnable():
4949
# VERIFY THE SCRIPT RAN SUCCESSFULLY.
5050
# A BOOT.STM is *required* so, the script will refuse to run far.
5151
# However, we can still tell if the script has been successfully invoked.
52-
script_invoked_successfully = ('BOOT.STM is missing' in result.stdout) and (result.returncode == 1)
52+
script_invoked_successfully = ('BOOT.STM is missing' in result.stderr) and (result.returncode == 1)
5353
if not script_invoked_successfully:
5454
raise AssertionError(
55-
f'Received a nonzero exit code when running `{CALLABLE_SCRIPT_NAME}` from command line!'
55+
f'Received a bad exit code when running `{CALLABLE_SCRIPT_NAME}` from command line!'
5656
f'\nstdout: {result.stdout}'
5757
f'\n\nstderr: {result.stderr}')
5858
finally:

0 commit comments

Comments
 (0)