[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PULL 3/6] iotests.py: Fix type check errors in wait_migration()
From: |
Kevin Wolf |
Subject: |
[PULL 3/6] iotests.py: Fix type check errors in wait_migration() |
Date: |
Tue, 3 Nov 2020 16:26:55 +0100 |
Commit 1847a4a8c20 clarified that event_wait() can return None (though
only with timeout=0) and commit f12a282ff47 annotated it as returning
Optional[QMPMessage].
Type checks in wait_migration() fail because of the unexpected optional
return type:
iotests.py:750: error: Value of type variable "Msg" of "log" cannot be
"Optional[Dict[str, Any]]"
iotests.py:751: error: Value of type "Optional[Dict[str, Any]]" is not indexable
iotests.py:754: error: Value of type "Optional[Dict[str, Any]]" is not indexable
Fortunately, the non-zero default timeout is used in the event_wait()
call, so we can make mypy happy by just asserting this.
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Message-Id: <20201027163806.290960-2-kwolf@redhat.com>
Reviewed-by: John Snow <jsnow@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
---
tests/qemu-iotests/iotests.py | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/tests/qemu-iotests/iotests.py b/tests/qemu-iotests/iotests.py
index 63d2ace93c..28388a0fbc 100644
--- a/tests/qemu-iotests/iotests.py
+++ b/tests/qemu-iotests/iotests.py
@@ -747,6 +747,10 @@ class VM(qtest.QEMUQtestMachine):
def wait_migration(self, expect_runstate: Optional[str]) -> bool:
while True:
event = self.event_wait('MIGRATION')
+ # We use the default timeout, and with a timeout, event_wait()
+ # never returns None
+ assert event
+
log(event, filters=[filter_qmp_event])
if event['data']['status'] in ('completed', 'failed'):
break
--
2.28.0
- [PULL 0/6] Block layer patches, Kevin Wolf, 2020/11/03
- [PULL 1/6] qmp: fix aio_poll() assertion failure on Windows, Kevin Wolf, 2020/11/03
- [PULL 3/6] iotests.py: Fix type check errors in wait_migration(),
Kevin Wolf <=
- [PULL 5/6] iotests: Use Python 3 style super(), Kevin Wolf, 2020/11/03
- [PULL 2/6] qemu-img convert: Free @sn_opts in all error cases, Kevin Wolf, 2020/11/03
- [PULL 6/6] block/vvfat: Fix bad printf format specifiers, Kevin Wolf, 2020/11/03
- [PULL 4/6] iotests: Disable unsubscriptable-object in pylint, Kevin Wolf, 2020/11/03
- Re: [PULL 0/6] Block layer patches, Peter Maydell, 2020/11/03