iotests: Remove the superfluous 2nd check for the availability of quorum
diff mbox series

Message ID 20200129141751.32652-1-thuth@redhat.com
State New
Headers show
Series
  • iotests: Remove the superfluous 2nd check for the availability of quorum
Related show

Commit Message

Thomas Huth Jan. 29, 2020, 2:17 p.m. UTC
Commit d9df28e7b07 ("iotests: check whitelisted formats") added the
modern @iotests.skip_if_unsupported() to the functions in this test,
so we don't need the old explicit test here anymore.

Signed-off-by: Thomas Huth <thuth@redhat.com>
---
 tests/qemu-iotests/139 | 3 ---
 1 file changed, 3 deletions(-)

Comments

Alberto Garcia Jan. 29, 2020, 2:27 p.m. UTC | #1
On Wed 29 Jan 2020 03:17:51 PM CET, Thomas Huth wrote:
> Commit d9df28e7b07 ("iotests: check whitelisted formats") added the
> modern @iotests.skip_if_unsupported() to the functions in this test,
> so we don't need the old explicit test here anymore.
>
> Signed-off-by: Thomas Huth <thuth@redhat.com>

Reviewed-by: Alberto Garcia <berto@igalia.com>

Berto
Andrey Shinkevich Jan. 31, 2020, 12:17 p.m. UTC | #2
On 29/01/2020 17:17, Thomas Huth wrote:
> Commit d9df28e7b07 ("iotests: check whitelisted formats") added the
> modern @iotests.skip_if_unsupported() to the functions in this test,
> so we don't need the old explicit test here anymore.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>   tests/qemu-iotests/139 | 3 ---
>   1 file changed, 3 deletions(-)
> 
> diff --git a/tests/qemu-iotests/139 b/tests/qemu-iotests/139
> index cbb5a76530..2f075888ba 100755
> --- a/tests/qemu-iotests/139
> +++ b/tests/qemu-iotests/139
> @@ -344,9 +344,6 @@ class TestBlockdevDel(iotests.QMPTestCase):
>   
>       @iotests.skip_if_unsupported(['quorum'])
>       def testQuorum(self):
> -        if not iotests.supports_quorum():
> -            return
> -
>           self.addQuorum('quorum0', 'node0', 'node1')
>           # We cannot remove the children of a Quorum device
>           self.delBlockDriverState('node0', expect_error = True)
> 

Reviewed-by: Andrey Shinkevich <andrey.shinkevich@virtuozzo.com>
Tested-by: Andrey Shinkevich <andrey.shinkevich@virtuozzo.com>
Max Reitz Feb. 6, 2020, 1:54 p.m. UTC | #3
On 29.01.20 15:17, Thomas Huth wrote:
> Commit d9df28e7b07 ("iotests: check whitelisted formats") added the
> modern @iotests.skip_if_unsupported() to the functions in this test,
> so we don't need the old explicit test here anymore.
> 
> Signed-off-by: Thomas Huth <thuth@redhat.com>
> ---
>  tests/qemu-iotests/139 | 3 ---
>  1 file changed, 3 deletions(-)

Thanks, applied to my block branch:

https://git.xanclic.moe/XanClic/qemu/commits/branch/block

Max

Patch
diff mbox series

diff --git a/tests/qemu-iotests/139 b/tests/qemu-iotests/139
index cbb5a76530..2f075888ba 100755
--- a/tests/qemu-iotests/139
+++ b/tests/qemu-iotests/139
@@ -344,9 +344,6 @@  class TestBlockdevDel(iotests.QMPTestCase):
 
     @iotests.skip_if_unsupported(['quorum'])
     def testQuorum(self):
-        if not iotests.supports_quorum():
-            return
-
         self.addQuorum('quorum0', 'node0', 'node1')
         # We cannot remove the children of a Quorum device
         self.delBlockDriverState('node0', expect_error = True)