<div dir="ltr">On Thu, Oct 19, 2017 at 3:14 PM, David Teller <span dir="ltr"><<a href="mailto:dteller@mozilla.com" target="_blank">dteller@mozilla.com</a>></span> wrote:<br><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">Reporting through Telemetry programmer errors in our code in the wild<br>
will definitely help.<br>
<br>
For tests, it depends on whether there is someone motivated enough to<br>
control the whitelisting and progressive un-whitelisting. I remember how<br>
painful it was to push forward the<br>
async-errors-should-cause-<wbr>tests-to-fail when some teams seemed to expect<br>
that fixing errors in their code wasn't their responsibility.<br></blockquote><div><br></div><div>Could we annotate source code with comments indicating what failures should be ignored? This also provides a forcing function for automation to understand source file and line mappings, which unlocks other tooling benefits (such as automatically filing bugs in the proper component based on the file(s) that threw the error).<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
On 19/10/17 14:02, Ted Mielczarek wrote:<br>
>> Part of the reason it was uncaught, is that there's no automated test<br>
>> coverage for that bit of code. It is a migration from one version to the<br>
>> other, but unless there is an explicit test (and I don't see one) that<br>
>> line won't be hit.<br>
><br>
> Given this bit, would any of the suggestions in this thread actually<br>
> help? If we're not exercising this code in automated tests, would "fail<br>
> tests on uncaught exceptions" make any difference?<br>
><br>
> I'm generally in favor of being stricter about errors in our test suites<br>
> etc, but I'm curious about whether we would actually have solved the<br>
> problem in question.<br>
><br>
> -Ted<br>
> ______________________________<wbr>_________________<br>
> firefox-dev mailing list<br>
> <a href="mailto:firefox-dev@mozilla.org">firefox-dev@mozilla.org</a><br>
> <a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/<wbr>listinfo/firefox-dev</a><br>
><br>
______________________________<wbr>_________________<br>
firefox-dev mailing list<br>
<a href="mailto:firefox-dev@mozilla.org">firefox-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/firefox-dev" rel="noreferrer" target="_blank">https://mail.mozilla.org/<wbr>listinfo/firefox-dev</a><br>
</div></div></blockquote></div><br></div></div>