On 11/12/24 08:14, Thomas Huth wrote:
On 12/11/2024 07.28, Cédric Le Goater wrote:
These were introduced in the avocado tests to workaround read issues
when interacting with console. They are no longer necessary.

Signed-off-by: Cédric Le Goater <c...@redhat.com>
---
  tests/functional/test_arm_aspeed.py | 7 ++-----
  1 file changed, 2 insertions(+), 5 deletions(-)

diff --git a/tests/functional/test_arm_aspeed.py 
b/tests/functional/test_arm_aspeed.py
index bdd5771a794e..07a8905e2365 100755
--- a/tests/functional/test_arm_aspeed.py
+++ b/tests/functional/test_arm_aspeed.py
@@ -136,10 +136,8 @@ def do_test_arm_aspeed_buildroot_start(self, image, 
cpu_id, pattern='Aspeed EVB'
          self.wait_for_console_pattern('lease of 10.0.2.15')
          # the line before login:
          self.wait_for_console_pattern(pattern)

Could we please wait for the "login:" string now instead, to make sure that 
there is no race between detecting the previous line and the real login prompt?

Yes. I have done that in other tests. I should do it in this test also.

Thanks,

C.


Reply via email to