[PATCH 04/15] tests/functional: remove todo wrt avocado.utils.wait_for

Daniel P. Berrangé posted 15 patches 4 days, 2 hours ago
There is a newer version of this series
[PATCH 04/15] tests/functional: remove todo wrt avocado.utils.wait_for
Posted by Daniel P. Berrangé 4 days, 2 hours ago
We're not using avocado anymore, so while the TODO item is still
relevant, suggesting use of avocado.utils is not.

Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
---
 tests/functional/test_m68k_nextcube.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/tests/functional/test_m68k_nextcube.py b/tests/functional/test_m68k_nextcube.py
index 89385a134a..0124622c40 100755
--- a/tests/functional/test_m68k_nextcube.py
+++ b/tests/functional/test_m68k_nextcube.py
@@ -37,8 +37,7 @@ def check_bootrom_framebuffer(self, screenshot_path):
         self.vm.launch()
 
         self.log.info('VM launched, waiting for display')
-        # TODO: Use avocado.utils.wait.wait_for to catch the
-        #       'displaysurface_create 1120x832' trace-event.
+        # TODO: wait for the 'displaysurface_create 1120x832' trace-event.
         time.sleep(2)
 
         self.vm.cmd('human-monitor-command',
-- 
2.46.0


Re: [PATCH 04/15] tests/functional: remove todo wrt avocado.utils.wait_for
Posted by Alex Bennée 4 days, 1 hour ago
Daniel P. Berrangé <berrange@redhat.com> writes:

> We're not using avocado anymore, so while the TODO item is still
> relevant, suggesting use of avocado.utils is not.
>
> Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>

Reviewed-by: Alex Bennée <alex.bennee@linaro.org>

-- 
Alex Bennée
Virtualisation Tech Lead @ Linaro