<div dir="ltr">Hi,<div><br>Besides from cultivating an entire test suite for the erl_tidy module, which is outside of the scope for the patch, if I were to provide a test for the addition, I would essentially be testing a simple print function. Would I not?</div>
<div><br></div><div>Some pointers on what to test would be a great help, otherwise I'm a bit a loss as to what to test.</div><div><br></div><div>Regards,<br>Aaron</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">
On Thu, Jul 11, 2013 at 6:54 PM, Andrew Thompson <span dir="ltr"><<a href="mailto:andrew@hijacked.us" target="_blank">andrew@hijacked.us</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On Thu, Jul 11, 2013 at 04:48:12PM +0200, Björn-Egil Dahlberg wrote:<br>
> It is great that the new option is documented!<br>
><br>
> Especially since syntax_tools is lacking tests, I think it's crucial<br>
> that tests are added.<br>
><br>
> It is our policy to require tests for all new features unless the<br>
> feature is already covered under some other test.<br>
<br>
</div>Not that I don't applaud this policy, but it can be *extremely*<br>
difficult to add new tests to a untested/minimally tested codebase.<br>
<br>
I know when I've made contributions in the past I've tried to update the<br>
test suites where applicable, but when there's no tests I've sort of<br>
thrown up my hands and just submitted the patch.<br>
<br>
I don't know the best solution to this, but pushing all the work on the<br>
contributor seems a little unfair. Maybe the OTP team can help out with<br>
testing in such cases.<br>
<span class="HOEnZb"><font color="#888888"><br>
Andrew<br>
</font></span><div class="HOEnZb"><div class="h5">_______________________________________________<br>
erlang-patches mailing list<br>
<a href="mailto:erlang-patches@erlang.org">erlang-patches@erlang.org</a><br>
<a href="http://erlang.org/mailman/listinfo/erlang-patches" target="_blank">http://erlang.org/mailman/listinfo/erlang-patches</a><br>
</div></div></blockquote></div><br></div>