<div dir="ltr"><div>Hi Alexandre,</div><div><br></div><div>this one will be harder to test... it only happens when cmd is not in batch mode (ie cmd reading directly from console APIs)</div><div>all the tests in cmd/tests are done in batch mode (ie cmd reading from file APIs)</div><div><br></div><div>since the expansion of variables is done currently done differently in batch mode vs non batch mode in cmd, adding a test to current infrastructure won't cover the use case</div><div>(and it already works correctly in batch mode)<br></div><div><br></div><div>it would require a new test harness inside cmd/tests to generate console input events instead of just passing the file handle</div><div><br></div><div>I'll provide the tests UC for the other patches, but I'm rather in favor of applying this patch without a test case</div><div><br></div><div>A+<br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, 5 Jan 2022 at 22:58, Marvin <<a href="mailto:testbot@winehq.org">testbot@winehq.org</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">Thank you for your contribution to Wine!<br>
<br>
This is an automated notification to let you know that your patch has<br>
been reviewed and its status set to "Needs tests".<br>
<br>
This means that you need to write some test cases demonstrating that<br>
the patch is correct.<br>
<br>
If you do not understand the reason for this status, disagree with our<br>
assessment, or are simply not sure how to proceed next, please ask for<br>
clarification by replying to this email.<br>
</blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail_signature">-- <br>Eric Pouech</div>