<div dir="ltr">Thanks for test suite.<div><br></div><div style>There are a few issues, though.</div><div style><br></div><div style>First, the new test cases are not correctly listed</div><div style>in the lists in the groups/0 function, so they will</div>
<div style>not actually be run.</div><div style><br></div><div style>Second, being very paranoid when writing test</div><div style>cases, I would make sure that the compiled</div><div style>code atually works (in this case, that the function</div>
<div style>with side effects is actually called).</div><div style><br></div><div style>Here is my suggestion how it can be done. If</div><div style>you are fine with my changes, we can simply</div><div style>squash my commit into yours.</div>
<div style><br></div><div style><a href="https://github.com/bjorng/otp/commit/4e21e16ebc1c012ac52b89ba1b38d6f98cfbac2c">https://github.com/bjorng/otp/commit/4e21e16ebc1c012ac52b89ba1b38d6f98cfbac2c</a><br></div><div style>
<br></div><div style>Note that since there is no good reason for</div><div style>keeping the test case case in a separate file,</div><div style>I have put all code for unused_multiple_values</div><div style>into core_fold_SUITE itself.</div>
<div style><br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Mon, Feb 4, 2013 at 12:16 PM, Anthony Ramine <span dir="ltr"><<a href="mailto:n.oxyde@gmail.com" target="_blank">n.oxyde@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I added a commit with two test cases, please refetch.<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Anthony Ramine<br>
<br>
Le 4 févr. 2013 à 07:17, Björn Gustavsson a écrit :<br>
</font></span><div class="HOEnZb"><div class="h5"><br>
><br>
> On Sun, Feb 3, 2013 at 12:32 PM, Anthony Ramine <<a href="mailto:n.oxyde@gmail.com">n.oxyde@gmail.com</a>> wrote:<br>
> Hi,<br>
><br>
> I've added some details and squashed the two commits together,<br>
> please refetch.<br>
><br>
> In which suite should I put such a test case?<br>
><br>
><br>
> I suggest core_fold_SUITE.<br>
><br>
> --<br>
> Björn Gustavsson, Erlang/OTP, Ericsson AB<br>
<br>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>Björn Gustavsson, Erlang/OTP, Ericsson AB
</div>