Message ID | 20220223123127.3206042-1-thuth@redhat.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
Series | tests/qemu-iotests/040: Skip TestCommitWithFilters without 'throttle' | expand |
On 23.02.22 13:31, Thomas Huth wrote: > iotest 040 already has some checks for the availability of the 'throttle' > driver, but some new code has been added in the course of time that > depends on 'throttle' but does not check for its availability. Add > a check to the TestCommitWithFilters class so that this iotest now > also passes again if 'throttle' has not been enabled in the QEMU > binaries. > > Signed-off-by: Thomas Huth <thuth@redhat.com> > --- > tests/qemu-iotests/040 | 1 + > 1 file changed, 1 insertion(+) Thanks, applied to my block branch: https://gitlab.com/hreitz/qemu/-/commits/block Hanna
diff --git a/tests/qemu-iotests/040 b/tests/qemu-iotests/040 index 6af5ab9e76..0e1cfd7e49 100755 --- a/tests/qemu-iotests/040 +++ b/tests/qemu-iotests/040 @@ -744,6 +744,7 @@ class TestCommitWithFilters(iotests.QMPTestCase): pattern_file) self.assertFalse('Pattern verification failed' in result) + @iotests.skip_if_unsupported(['throttle']) def setUp(self): qemu_img('create', '-f', iotests.imgfmt, self.img0, '64M') qemu_img('create', '-f', iotests.imgfmt, self.img1, '64M')
iotest 040 already has some checks for the availability of the 'throttle' driver, but some new code has been added in the course of time that depends on 'throttle' but does not check for its availability. Add a check to the TestCommitWithFilters class so that this iotest now also passes again if 'throttle' has not been enabled in the QEMU binaries. Signed-off-by: Thomas Huth <thuth@redhat.com> --- tests/qemu-iotests/040 | 1 + 1 file changed, 1 insertion(+)