Skip to content

Commit

Permalink
Fix the integration test due to PR 374
Browse files Browse the repository at this point in the history
PR#374 introduced a change in behaviour where the stdout/stderr is not
stored if a handler is supplied.  This shouldn't affect most people as,
if you are using a handler, then the code is already doing something
with stdout/stderr.

Signed-off-by: Alex Kavanagh <alex.kavanagh@canonical.com>
  • Loading branch information
ajkavanagh committed Oct 10, 2019
1 parent 8a3f58d commit 8b5ba45
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion integration/test_containers.py
Original file line number Diff line number Diff line change
Expand Up @@ -209,7 +209,8 @@ def stdout_handler(msg):
)

self.assertEqual(0, result.exit_code)
self.assertEqual(test_msg, result.stdout)
# if a handler is supplied then there is no stdout in result
self.assertEqual('', result.stdout)
self.assertEqual('', result.stderr)
self.assertEqual(stdout_msgs, [test_msg])

Expand Down

0 comments on commit 8b5ba45

Please sign in to comment.