[PATCH v2] test/py: fix runtest wrapper for pytest 6

Simon Glass sjg at chromium.org
Mon Feb 1 21:44:18 CET 2021


On Sat, 30 Jan 2021 at 20:12, Stephen Warren <swarren at wwwdotorg.org> wrote:
>
> The implementation of pytest_runtest_protocol() must call
> pytest_runtest_logstart() and pytest_runtest_logfinish(). This appears to
> be necessary even in pytest 5.2.1 judging by the default version of
> pytest_runtest_protocol(), but evidently some form of code reorganization
> in pytest only made this have a practical effect in the newer version. I'd
> previously been under the impression that 100% of the required work of
> pytest_runtest_protocol() was handled by the fact it called
> runtestprotocol() as its implementation. However, it appears that custom
> implementations do need to do a little more than this.
>
> Reported-by: Heinrich Schuchardt <xypron.glpk at gmx.de>
> Signed-off-by: Stephen Warren <swarren at wwwdotorg.org>
> ---
> v2: Rebased on a marginally newer commit, hence removed the change to
> test/py/test.py. Cleaned up return statement.
> ---
>  test/py/conftest.py | 5 ++++-
>  1 file changed, 4 insertions(+), 1 deletion(-)
>

Reviewed-by: Simon Glass <sjg at chromium.org>


More information about the U-Boot mailing list