[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Acceptance test machine_m68k_nextcube.py: relax the error co
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH] Acceptance test machine_m68k_nextcube.py: relax the error code pattern |
Date: |
Thu, 19 Sep 2019 18:19:40 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.7.0 |
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 <address@hidden>
> ---
> 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é <address@hidden>