630746 - Add a flag to jstests.py to skip extension tests

Status: RESOLVED FIXED

(Core :: JavaScript Engine, defect)

Description

[Jeff Walden [:Waldo]]

Attachment: Patch

...to make it easier for other engines to run only the tests that they *should* pass.  I realized we needed this when, a bit ago, I was responding to email involving other non-SpiderMonkey JS hackers explaining how to run our test suite and realized it wasn't really possible to do it non-noisily, not easily at least.

Comment 1

[David Mandelin [:dmandelin]]

Comment on attachment 508977 [details] [diff] [review]
Patch


>+def exclude_extension_tests(test_list):
>+    r = re.compile(r'^.*?/extensions/')
>+    return [_ for _ in test_list if r.match(_.path) is None]
>+

This can be just 

  def exclude_extension_tests(test_list):
      return [_ for _ in test_list if '/extensions/' not in _.path]

r+ with that (make sure I got it right).

Comment 2

[Jeff Walden [:Waldo]]

Actually, the extra method's kind of superfluous, just folded it into the location that would have called it -- pushed with those changes:

http://hg.mozilla.org/tracemonkey/rev/73efdea6fda0

Comment 3

[Paul Biggar]

I wonder would a better way to accomplish this have been to extend the [TEST-SPECS] syntax to allow a ! at the front. So the command would be:

 tests/jstests.py objdir/js !extensions

Comment 4

[Chris Leary [:cdleary] (not checking bugmail)]

http://hg.mozilla.org/mozilla-central/rev/73efdea6fda0