
12 Jul
2023
12 Jul
'23
3:49 a.m.
On 7/12/23 02:24, Simon Glass wrote:
Hi Marek,
Hi,
On Tue, 11 Jul 2023 at 17:10, Marek Vasut marex@denx.de wrote:
On 7/12/23 00:51, Simon Glass wrote:
This test was written for the incorrect use of assertions. Update it to build with the previous approach, where tests fail at the first assertion.
So basically ut_assert_nextline() does not automatically bail, you have to make it expand to return -ERRNO and check for that errno again with ut_assertok() at the callsite ?
Yes, or any non-zero value. The call stack unwinds since each function that fails returns a non-zero value.
It would be nice to spell this out explicitly in the commit message. This behavior really is inobvious.