<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><span style="font-family:arial">On 6 June 2014 16:21, Chris McMahon </span><span dir="ltr" style="font-family:arial"><<a href="mailto:cmcmahon@wikimedia.org" target="_blank">cmcmahon@wikimedia.org</a>></span><span style="font-family:arial"> wrote:</span><br>

</div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">

<div class="">On Fri, Jun 6, 2014 at 4:16 PM, Matthew Flaschen <span dir="ltr"><<a href="mailto:mflaschen@wikimedia.org" target="_blank">mflaschen@wikimedia.org</a>></span> wrote:<br>
</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">Are they voting or non-voting?<br></div></blockquote><div><br></div><div>I think browser tests should always be non-voting.  Therein lies a day-long training session... </div>

</div></div></div></blockquote><div><br></div><div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">​And FWIW I fundamentally disagree. :-)</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">

<br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">Non-voting tests are routinely ignored. If your change breaks a test, you should either fix your code or update the test; both of these actions are conscious responses to the outcome of the test, and if they're not part of the pre-merge workflow they too rarely don't happen.</div>

</div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_default" style="font-family:arial,helvetica,sans-serif">J.</div></div>-- <br>James D. Forrester<br>Product Manager, VisualEditor<br>

Wikimedia Foundation, Inc.<br><br><a href="mailto:jforrester@wikimedia.org" target="_blank">jforrester@wikimedia.org</a> | @jdforrester
</div></div>