[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 02/26] Acceptance tests: refactor wait_for_console_pattern
From: |
Cleber Rosa |
Subject: |
Re: [PATCH 02/26] Acceptance tests: refactor wait_for_console_pattern |
Date: |
Mon, 28 Oct 2019 11:09:59 -0400 |
User-agent: |
Mutt/1.12.1 (2019-06-15) |
On Mon, Oct 28, 2019 at 08:58:01AM +0100, Aleksandar Markovic wrote:
> On Monday, October 28, 2019, Philippe Mathieu-Daudé <address@hidden>
> wrote:
>
> > From: Cleber Rosa <address@hidden>
> >
> > The same utility method is already present in two different test
> > files, so let's consolidate it into a single utility function.
> >
> > Signed-off-by: Cleber Rosa <address@hidden>
> > Message-Id: <address@hidden>
> > Reviewed-by: Philippe Mathieu-Daudé <address@hidden>
> > [PMD: failure_message is optional]
> > Reviewed-by: David Gibson <address@hidden>
> > Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> > ---
> > tests/acceptance/avocado_qemu/__init__.py | 25 +++++++++++++++++++++
> > tests/acceptance/boot_linux_console.py | 27 +++++------------------
> > tests/acceptance/linux_ssh_mips_malta.py | 18 +++------------
> > 3 files changed, 33 insertions(+), 37 deletions(-)
> >
> >
>
> You missed my r-b, that I gave here:
>
> https://lists.gnu.org/archive/html/qemu-devel/2019-10/msg06378.html
>
> A.
>
Hi Aleksandar,
I've collected your review, it'll be on my PR.
Thanks!
- Cleber.
- [PATCH 00/26] tests/acceptance: Queue for 4.2, Philippe Mathieu-Daudé, 2019/10/28
- [PATCH 01/26] python/qemu/machine: Allow to use other serial consoles than default, Philippe Mathieu-Daudé, 2019/10/28
- [PATCH 02/26] Acceptance tests: refactor wait_for_console_pattern, Philippe Mathieu-Daudé, 2019/10/28
- [PATCH 03/26] tests/acceptance: Fixe wait_for_console_pattern() hangs, Philippe Mathieu-Daudé, 2019/10/28
- [PATCH 04/26] tests/acceptance: Send <carriage return> on serial lines, Philippe Mathieu-Daudé, 2019/10/28
- [PATCH 05/26] tests/acceptance: Refactor exec_command_and_wait_for_pattern(), Philippe Mathieu-Daudé, 2019/10/28
- [PATCH 06/26] tests/acceptance: Rename avocado_qemu.Test as MachineTest, Philippe Mathieu-Daudé, 2019/10/28