[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 6/7] tests/acceptance: Add set_vm_arg() to the Test class
From: |
Willian Rampazzo |
Subject: |
Re: [PATCH v3 6/7] tests/acceptance: Add set_vm_arg() to the Test class |
Date: |
Tue, 22 Jun 2021 17:14:49 -0300 |
On Fri, Apr 30, 2021 at 10:35 AM Wainer dos Santos Moschetta
<wainersm@redhat.com> wrote:
>
> The set_vm_arg method is added to avocado_qemu.Test class on this
> change. Use that method to set (or replace) an argument to the list of
> arguments given to the QEMU binary.
>
> Suggested-by: Cleber Rosa <crosa@redhat.com>
> Signed-off-by: Wainer dos Santos Moschetta <wainersm@redhat.com>
> ---
> tests/acceptance/avocado_qemu/__init__.py | 21 +++++++++++++++++++++
> 1 file changed, 21 insertions(+)
>
> diff --git a/tests/acceptance/avocado_qemu/__init__.py
> b/tests/acceptance/avocado_qemu/__init__.py
> index 7f8e703757..14c6ae70c8 100644
> --- a/tests/acceptance/avocado_qemu/__init__.py
> +++ b/tests/acceptance/avocado_qemu/__init__.py
> @@ -240,6 +240,27 @@ def get_vm(self, *args, name=None):
> self._vms[name].set_machine(self.machine)
> return self._vms[name]
>
> + def set_vm_arg(self, arg, value):
> + """
> + Set an argument to list of extra arguments to be given to the QEMU
> + binary. If the argument already exists then its value is replaced.
> +
> + :param arg: the QEMU argument, such as "-cpu" in "-cpu host"
> + :type arg: str
> + :param value: the argument value, such as "host" in "-cpu host"
> + :type value: str
> + """
> + if not arg or not value:
> + return
> + if arg not in self.vm.args:
> + self.vm.args.extend([arg, value])
> + else:
> + idx = self.vm.args.index(arg) + 1
> + if idx < len(self.vm.args):
> + self.vm.args[idx] = value
> + else:
> + self.vm.args.append(value)
> +
Considering all args in self.vm.args are composed of [arg,value]:
Reviewed-by: Willian Rampazzo <willianr@redhat.com>
> def tearDown(self):
> for vm in self._vms.values():
> vm.shutdown()
> --
> 2.29.2
>
[Prev in Thread] |
Current Thread |
[Next in Thread] |
- Re: [PATCH v3 6/7] tests/acceptance: Add set_vm_arg() to the Test class,
Willian Rampazzo <=