Hi Simon, On Sun, Jul 3, 2016 at 8:40 AM, Simon Glass <s...@chromium.org> wrote: > Some tests want to execute a sequence of commands. Add a helper for this. > > Signed-off-by: Simon Glass <s...@chromium.org>
Reviewed-by: Teddy Reed <teddy.r...@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@lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot -- Teddy Reed V _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
