[PATCH v7 14/16] test: py: vboot: add test for global image signature

[Philippe REYNES]

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.reynes at 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.reynes@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