[PATCH v6 08/12] efi_loader: Disable ANSI output for tests

Tom Rini trini at konsulko.com
Tue Oct 1 02:24:42 CEST 2024


On Tue, Oct 01, 2024 at 01:38:56AM +0200, Heinrich Schuchardt wrote:
> On 26.09.24 23:59, Simon Glass wrote:
> > We don't want ANSI characters written in tests since it is a pain to
> > check the output with ut_assert_nextline() et al.
> > 
> > Provide a way to tests to request that ANSI characters not be sent.
> > 
> > Add a proper function comment while we are here, to encourage others.
> > 
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> 
> Please, consider prior review before resubmitting patches.
> 
> As responded to all prior submissions:
> 
> We want to test the code running on actual machines.
> We don't want to have sandbox code everywhere.
> 
> I cannot see any test that is not passing due to the current behavior.

The pytests for the EFI selftests are unreliable for me, on Raspberry Pi
3, more often in 32bit mode than 64bit mode, but I feel like I see it
there too. And when they fail, the console log is full of ANSI escape
sequences. Is this specific test a test you run regularly on real
hardware?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20240930/775986c4/attachment.sig>


More information about the U-Boot mailing list