Hi Tom,

Le 26/03/2022 à 00:02, Tom Rini a écrit :
On Fri, Mar 25, 2022 at 11:54:18PM +0100, Philippe REYNES wrote:
Hi Tom,


Le 25/03/2022 à 18:11, Tom Rini a écrit :
On Mon, Mar 14, 2022 at 03:57:43PM +0100, Philippe Reynes wrote:

Adds test units for the pre-load header signature.

Signed-off-by: Philippe Reynes <philippe.rey...@softathome.com>
---
   test/py/tests/test_vboot.py                   | 145 ++++++++++++++++--
   test/py/tests/vboot/sandbox-binman-pss.dts    |  25 +++
   test/py/tests/vboot/sandbox-binman.dts        |  24 +++
   .../tests/vboot/sandbox-u-boot-global-pss.dts |  28 ++++
   test/py/tests/vboot/sandbox-u-boot-global.dts |  27 ++++
   test/py/tests/vboot/simple-images.its         |  36 +++++
   6 files changed, 269 insertions(+), 16 deletions(-)
   create mode 100644 test/py/tests/vboot/sandbox-binman-pss.dts
   create mode 100644 test/py/tests/vboot/sandbox-binman.dts
   create mode 100644 test/py/tests/vboot/sandbox-u-boot-global-pss.dts
   create mode 100644 test/py/tests/vboot/sandbox-u-boot-global.dts
   create mode 100644 test/py/tests/vboot/simple-images.its
This leads to CI failures such as:
https://source.denx.de/u-boot/u-boot/-/jobs/412046
and I guess maybe we need something new in test/py/requirements.txt
maybe?
I think that this issue should be fixed with this patch :

https://patchwork.ozlabs.org/project/uboot/patch/20220127140314.10264-2-philippe.rey...@softathome.com/
OK, but that's a rather specific hard-coded path to add, and won't work
in my setup for example since I use mktemp to make the base dir for all
my tests.

I have sent a v8 with a change in test_vboot to use PYTHONPATH, so the CI should be fixed.

And this patch is no longer needed.


Regards,

Philippe


Reply via email to