[Buildroot] [PATCH v2 1/6] testing/infra/emulator: allow to specify pexpect timeout

Ricardo Martincoski ricardo.martincoski at gmail.com
Sat Jul 15 04:21:57 UTC 2017


Hello,

On Tue, Jul 11, 2017 at 11:40 PM, Andrey Smirnov wrote:

> Some commands take more than 5 seconds to complete under QEMU, so add
> provisions to allow individual unit-test to specify different duration
> to avoid false negative test failures.
> 
> Signed-off-by: Andrey Smirnov <andrew.smirnov at gmail.com>

Reviewed-by: Ricardo Martincoski <ricardo.martincoski at gmail.com>
[I executed all tests in the current master with this patch]
Tested-by: Ricardo Martincoski <ricardo.martincoski at gmail.com>

[snip]
> -    def run(self, cmd):
> +    def run(self, cmd, timeout=-1):

Indeed.
>From the docs: "When the keyword argument timeout is -1 (default), then TIMEOUT
will raise after the default value specified by the class timeout attribute."

Regards,
Ricardo


More information about the buildroot mailing list