<div dir="ltr"><div>Honestly, I prefer if the bug number isn't in the filename at all.  Tab completing specific tests is just more annoying if you're globbing or whatever around bug numbers.<br><br></div>I'll check the comments if I need a bug number.<br>
<div><br>-dave<br></div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Sep 18, 2013 at 9:23 AM, Gijs Kruitbosch <span dir="ltr"><<a href="mailto:gijskruitbosch@gmail.com" target="_blank">gijskruitbosch@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">TL;DR: I'd like to propose having a policy to require new tests (or updates to existing tests) to have descriptive file names - which means more than just a bug number.<br>

<br>
Longer explanation: some of us were talking on IRC about how we have lots of tests that have filenames that only have a bug number as useful information. For example:<br>
<br>
$ ls -1 browser/base/content/test/ | grep 'browser_bug[0-9]*.js' | wc -l<br>
108<br>
<br>
Of course, bug numbers are useful because then we can read descriptions on bugzilla. But TBPL, my terminal window, and the MXR directory listings are not bugzilla.<br>
<br>
In other word, this makes it harder to know what tests are testing, and to split them up into directories that make topical sense. Furthermore, if a test breaks, it's not immediately obvious whether the breakage is topical to that test, the fault of that test itself or some other test that ran earlier, and if it's some other test: which tests are or are not likely suspects for having broken assumptions of tests running after themselves.<br>

<br>
Gavin suggested we could just have a policy to require test filenames to be more than just bug numbers. I concur. How do others feel about this?<br>
<br>
~ Gijs<br>
______________________________<u></u>_________________<br>
firefox-dev mailing list<br>
<a href="mailto:firefox-dev@mozilla.org" target="_blank">firefox-dev@mozilla.org</a><br>
<a href="https://mail.mozilla.org/listinfo/firefox-dev" target="_blank">https://mail.mozilla.org/<u></u>listinfo/firefox-dev</a><br>
</blockquote></div><br></div>