Cc'ing Thomas

On 9/19/19 6:14 PM, Cleber Rosa wrote:
> Instead of looking for a specific error, let's relax the pattern
> because different errors have been seen (I'm consistenly getting 52)
> and the real goal of this test is to validate the framebuffer
> operation, and not to reproduce one specific error.

This might be because I wrote this test before Thomas added the serial
controller (commit b17bed5b1727e3aa9e37fc8e8c3222130ceab22f).

> Signed-off-by: Cleber Rosa <cr...@redhat.com>
> ---
>  tests/acceptance/machine_m68k_nextcube.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/acceptance/machine_m68k_nextcube.py 
> b/tests/acceptance/machine_m68k_nextcube.py
> index e09cab9f20..fcd2c58ee7 100644
> --- a/tests/acceptance/machine_m68k_nextcube.py
> +++ b/tests/acceptance/machine_m68k_nextcube.py
> @@ -116,6 +116,6 @@ class NextCubeMachine(Test):
>              if len(line):
>                  console_logger.debug(line)
>          self.assertIn('Testing the FPU, SCC', text)
> -        self.assertIn('System test failed. Error code 51', text)
> +        self.assertIn('System test failed. Error code', text)
>          self.assertIn('Boot command', text)
>          self.assertIn('Next>', text)
> 

Reviewed-by: Philippe Mathieu-Daudé <phi...@redhat.com>

Reply via email to