[U-Boot] [PATCH 10/14] test/py: Add a helper to run a list of U-Boot commands

Teddy Reed teddy.reed at gmail.com
Sun Jul 3 23:12:03 CEST 2016


Hi Simon,

On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass <sjg at chromium.org> wrote:
> Some tests want to execute a sequence of commands. Add a helper for this.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>

Reviewed-by: Teddy Reed <teddy.reed at gmail.com>

> ---
>
>  test/py/u_boot_console_base.py | 16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
>
> diff --git a/test/py/u_boot_console_base.py b/test/py/u_boot_console_base.py
> index 815fa64..b5aad7c 100644
> --- a/test/py/u_boot_console_base.py
> +++ b/test/py/u_boot_console_base.py
> @@ -216,6 +216,22 @@ class ConsoleBase(object):
>              self.cleanup_spawn()
>              raise
>
> +    def run_command_list(self, cmds):
> +        """Run a list of commands.
> +
> +        This is a helper function to call run_command() with default arguments
> +        for each command in a list.
> +
> +        Args:
> +            cmd: List of commands (each a string)

nit, in most other docstrings, these Args and Return lines have punctuation.

> +        Returns:
> +            Combined output of all commands, as a string
> +        """
> +        output = ''
> +        for cmd in cmds:
> +            output += self.run_command(cmd)

Although this fits the prototypes for other run_* functions, a list of
output strings is much easier to test and reason about. :/

> +        return output
> +
>      def ctrlc(self):
>          """Send a CTRL-C character to U-Boot.
>
> --
> 2.8.0.rc3.226.g39d4020
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot



-- 
Teddy Reed V


More information about the U-Boot mailing list