diff options
| author | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2018-02-24 18:25:07 +0900 |
|---|---|---|
| committer | Tristan Van Berkom <tristan.vanberkom@codethink.co.uk> | 2018-02-24 18:29:56 +0900 |
| commit | c4f70392cc28fcf73ad0ef503b44f87162dff381 (patch) | |
| tree | 4e903abd4c5999b46b1738050b8b38e8bdf757ec | |
| parent | 00c6f4689cd107a81a05c07bbf73afe4ad347341 (diff) | |
| download | buildstream-c4f70392cc28fcf73ad0ef503b44f87162dff381.tar.gz | |
testutils/runcli.py: Print something helpful if BuildStream fails to exit properly
When testing a recent patch, I had missed a return of the exit code
in `bst shell` resulting in the CLI exiting with `None`; this patch
makes the fixture more helpful, where previously it just printed `None`
| -rw-r--r-- | tests/testutils/runcli.py | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/tests/testutils/runcli.py b/tests/testutils/runcli.py index 3479364f7..a913c4aa2 100644 --- a/tests/testutils/runcli.py +++ b/tests/testutils/runcli.py @@ -293,6 +293,7 @@ class Cli(): exit_code = e.code if not isinstance(exit_code, int): + sys.stdout.write('Program exit code was not an integer: ') sys.stdout.write(str(exit_code)) sys.stdout.write('\n') exit_code = 1 |