Re: [PATCH 1/2] tests/acceptance: Extract boot_integratorcp() from test_integratorcp()

[Wainer dos Santos Moschetta]

On 1/31/20 7:11 PM, Philippe Mathieu-Daudé wrote:
> As we want to re-use this code, extract it as a new function.
> Since we are using the PL011 serial console, add a Avocado tag
> to ease filtering of tests.
>
> Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> ---
>   tests/acceptance/machine_arm_integratorcp.py | 16 ++++++++++------
>   1 file changed, 10 insertions(+), 6 deletions(-)


Reviewed-by: Wainer dos Santos Moschetta <address@hidden>


> diff --git a/tests/acceptance/machine_arm_integratorcp.py 
> b/tests/acceptance/machine_arm_integratorcp.py
> index 4f9ab40f2f..748819916d 100644
> --- a/tests/acceptance/machine_arm_integratorcp.py
> +++ b/tests/acceptance/machine_arm_integratorcp.py
> @@ -19,12 +19,7 @@ class IntegratorMachine(Test):
>   
>       timeout = 90
>   
> -    @skipUnless(os.getenv('AVOCADO_ALLOW_UNTRUSTED_CODE'), 'untrusted code')
> -    def test_integratorcp(self):
> -        """
> -        :avocado: tags=arch:arm
> -        :avocado: tags=machine:integratorcp
> -        """
> +    def boot_integratorcp(self):
>           kernel_url = ('https://github.com/zayac/qemu-arm/raw/master/'
>                         'arm-test/kernel/zImage.integrator')
>           kernel_hash = '0d7adba893c503267c946a3cbdc63b4b54f25468'
> @@ -40,4 +35,13 @@ class IntegratorMachine(Test):
>                            '-initrd', initrd_path,
>                            '-append', 'printk.time=0 console=ttyAMA0')
>           self.vm.launch()
> +
> +    @skipUnless(os.getenv('AVOCADO_ALLOW_UNTRUSTED_CODE'), 'untrusted code')
> +    def test_integratorcp(self):
> +        """
> +        :avocado: tags=arch:arm
> +        :avocado: tags=machine:integratorcp
> +        :avocado: tags=device:pl011
> +        """
> +        self.boot_integratorcp()
>           wait_for_console_pattern(self, 'Log in as root')