<html>
<head>
<meta http-equiv="content-type" content="text/html; charset=utf-8">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<p>Hello firefox-dev,</p>
<p>A while back I tried to start moving (browser) mochitests out of
browser/base/content/test/general/ to more topical directories. To
copy my reasoning from the original metabug:</p>
<p>
<blockquote type="cite">
<meta http-equiv="content-type" content="text/html;
charset=utf-8">
</blockquote>
<blockquote type="cite">It takes ages to run [locally], there's a
lot of tests that can influence each other, it's hard to isolate
a problem if tests do influence each other, it's hard to run
tests relevant for the change you're making, it takes long on
infra, if something breaks (cough bug 1253956 cough) everything
gets mass-disabled... there are a lot of reasons to not want to
have a directory with over 300 tests in.</blockquote>
</p>
<p>To add to that: browser mochitests are some of our slowest tests.
As far as I can tell, the mochitest-bc chunk on infra that
contains general/ right now *runs no other directories at all*. On
<a
href="https://public-artifacts.taskcluster.net/KIBQPE3_QZCAnjxL8jd9zA/0/public/logs/live_backing.log">linux
e10s debug</a>, that directory takes 50 minutes (3400s) to run.<br>
</p>
<p>There are a lot more topical subdirs now, and there's really no
good reason, IMO, to keep adding random tests to this directory.
Which does seem to keep happening: while the number of tests was
266 after I moved 40-odd tests to the urlbar/ directory 10 months
ago, and more tests were moved out of general/ since, the number
of tests in general/ right now is 267 (we'll run a ~90% subset of
those depending on platform, debug on/off and e10s on/off).<br>
</p>
<p>I'd like to add a commit/push hook that makes adding more tests
impossible unless you include some magic words, similar to the
webidl DOM peer review hooks.<br>
</p>
<p>Would anyone object to this happening?</p>
<p>Gijs</p>
</body>
</html>