Implement jstests.py --include and --exclude

RESOLVED FIXED in Firefox 54

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: sfink, Assigned: sfink)

Tracking

unspecified
mozilla54
Points:
---

Firefox Tracking Flags

(firefox54 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
--exclude is handy for skipping large swathes of tests. May be useful if we need to skip eg test262. --include is redundant with the list of tests at the end of the command line, but with the current setup there is a place where it's easy to add options to jstests.py, but hard to append things to the overall command line.
(Assignee)

Comment 1

2 years ago
Created attachment 8832273 [details] [diff] [review]
Implement jstests.py --include and --exclude options for including and excluding files or directories

Sorry about the totally random review request.
Attachment #8832273 - Flags: review?(nfitzgerald)
(In reply to Steve Fink [:sfink] [:s:] from comment #1)
> Sorry about the totally random review request.

No need to apologize!
Comment on attachment 8832273 [details] [diff] [review]
Implement jstests.py --include and --exclude options for including and excluding files or directories

Review of attachment 8832273 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/tests/lib/manifest.py
@@ +367,5 @@
>  
>      return True
>  
>  
> +def _split_files_and_dirs(location, paths):

A doc comment here would be nice.

@@ +376,5 @@
> +            dirs.add(path[0:-1])
> +        elif os.path.isdir(os.path.join(location, path)):
> +            dirs.add(path)
> +        else:
> +            files.add(path)

Maybe worth checking whether or not the path exists?
Attachment #8832273 - Flags: review?(nfitzgerald) → review+

Comment 4

2 years ago
Pushed by sfink@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/3ae32e151ec0
Implement jstests.py --include and --exclude options for including and excluding files or directories, r=fitzgen

Comment 6

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/3ae32e151ec0
https://hg.mozilla.org/mozilla-central/rev/adc12dd054ca
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox54: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.