On Tue, 9 Mar 2021 at 16:00, Wainer dos Santos Moschetta <waine...@redhat.com> wrote: > > For the sake of improve debuggability of tests which use the > wait_for_console_pattern(), this changed the _console_interaction() so that > the expected message is printed if the test fail. > > Signed-off-by: Wainer dos Santos Moschetta <waine...@redhat.com> > --- > While I was testing "[PATCH v2 2/2] tests/acceptance: Test ast2600 machine" > I could not clearly determine which of the called wait_for_console_pattern() > was failing. So this patch improves debuggability in such as situations.
Nice! Reviewed-by: Joel Stanley <j...@jms.id.au> > > tests/acceptance/avocado_qemu/__init__.py | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/tests/acceptance/avocado_qemu/__init__.py > b/tests/acceptance/avocado_qemu/__init__.py > index df167b142c..ed338caaba 100644 > --- a/tests/acceptance/avocado_qemu/__init__.py > +++ b/tests/acceptance/avocado_qemu/__init__.py > @@ -97,7 +97,8 @@ def _console_interaction(test, success_message, > failure_message, > break > if failure_message and failure_message in msg: > console.close() > - fail = 'Failure message found in console: %s' % failure_message > + fail = 'Failure message found in console: "%s". Expected: "%s"' > % \ > + (failure_message, success_message) > test.fail(fail) > > def interrupt_interactive_console_until_pattern(test, success_message, > -- > 2.29.2 > >
