| Message ID | 20220221101933.307525-1-thuth@redhat.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | tests/avocado/machine_s390_ccw_virtio: Adapt test to new default resolution | expand |
On Mon, Feb 21, 2022 at 11:19:33AM +0100, Thomas Huth 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> > --- > tests/avocado/machine_s390_ccw_virtio.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Daniel P. Berrangé <berrange@redhat.com> Regards, Daniel
On 21/2/22 11:19, Thomas Huth 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> > --- > tests/avocado/machine_s390_ccw_virtio.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
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)
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)
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> --- tests/avocado/machine_s390_ccw_virtio.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)