Re: [PATCH] tests/avocado/machine_s390_ccw_virtio: Adapt test to new default resolution

[Halil Pasic]

On Mon, 21 Feb 2022 11:19:33 +0100
Thomas Huth <thuth@redhat.com> wrote:

> QEMU's default screen resolution recently changed to 1280x800, so the
> resolution in the screen shot header changed of course, too.
> 
> Reported-by: Peter Maydell <peter.maydell@linaro.org>
> Fixes: de72c4b7cd ("edid: set default resolution to 1280x800 (WXGA)")
> Signed-off-by: Thomas Huth <thuth@redhat.com>

Looks good!

Acked-by: Halil Pasic <pasic@linux.ibm.com>
> ---
>  tests/avocado/machine_s390_ccw_virtio.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/tests/avocado/machine_s390_ccw_virtio.py 
> b/tests/avocado/machine_s390_ccw_virtio.py
> index bd03d7160b..438a6f4321 100644
> --- a/tests/avocado/machine_s390_ccw_virtio.py
> +++ b/tests/avocado/machine_s390_ccw_virtio.py
> @@ -248,7 +248,7 @@ def test_s390x_fedora(self):
>              line = ppmfile.readline()
>              self.assertEqual(line, b"P6\n")
>              line = ppmfile.readline()
> -            self.assertEqual(line, b"1024 768\n")
> +            self.assertEqual(line, b"1280 800\n")
>              line = ppmfile.readline()
>              self.assertEqual(line, b"255\n")
>              line = ppmfile.readline(256)