<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Nov 25, 2020 at 9:17 PM Cyril Hrubis <<a href="mailto:chrubis@suse.cz">chrubis@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi!<br>
> > I agree to add an expected value check for TEST_PASS(), e.g<br>
> > <br>
> > #define TEST_PASS(SCALL, EXPECTED, ...)<br>
> <br>
> I would rather keep this simple case as it is, since many function<br>
> return zero on success and added more specialized functions if needed as<br>
> it was done with TEST_FD()<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Ok.</div></div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Is anyone against pushing this initial patchset as it is?<br></blockquote><div> </div><div><div class="gmail_default" style="font-size:small">Another bit of concern I have is the TEST_FAIL might cause</div><div class="gmail_default" style="font-size:small">confusion in code reading. It looks like the test will be failed here</div><div class="gmail_default" style="font-size:small">but in finish the test report pass, maybe we'd better tell people the</div></div><div class="gmail_default" style="font-size:small">failure is expected?</div><div class="gmail_default" style="font-size:small">(that was just my feeling on TEST_FAIL <span style="color:rgb(51,51,51);font-size:14px">at the first </span><span style="color:rgb(51,51,51);font-size:14px">glance</span>)</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Or rename them to:</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">TEST_EXP_PASS</div><div class="gmail_default" style="font-size:small">TEST_EXP_FAIL</div><div class="gmail_default" style="font-size:small">TEST_EXP_FD</div></div><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>