Last Comment Bug 1150707 - Make use of tags for running only icaljs/libcal tests
: Make use of tags for running only icaljs/libcal tests
Status: RESOLVED FIXED
:
Product: Calendar
Classification: Client Software
Component: Build Config (show other bugs)
: Trunk
: All All
-- normal (vote)
: 4.2
Assigned To: Philipp Kewisch [:Fallen]
:
:
Mentors:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2015-04-02 14:43 PDT by Philipp Kewisch [:Fallen]
Modified: 2015-04-05 08:41 PDT (History)
1 user (show)
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
Fix - v1 (845 bytes, patch)
2015-04-02 14:45 PDT, Philipp Kewisch [:Fallen]
geoff: review+
Details | Diff | Splinter Review

Description User image Philipp Kewisch [:Fallen] 2015-04-02 14:43:15 PDT
bug 987360 added a tags property which will help us run only icaljs tests, or only libical tests.

Usage:

mach xpcshell-test --tags icaljs  # run only ical.js tests
mach xpcshell-test --tags libical # run only libical tests

The patch here will not work until they switch to space delimiters, which should happen soon enough that its ok to push this.

For a brief moment I thought about adding more tags like "alarms" to specific tests, but given our tests run fairly fast I'm not sure how useful that would be. It might tempt to just run tests partially when writing patches.
Comment 1 User image Philipp Kewisch [:Fallen] 2015-04-02 14:45:01 PDT
Created attachment 8587667 [details] [diff] [review]
Fix - v1
Comment 2 User image Geoff Lankow (:darktrojan) 2015-04-02 15:58:47 PDT
Comment on attachment 8587667 [details] [diff] [review]
Fix - v1

Nice. I've been wanting this for some time.
Comment 3 User image Philipp Kewisch [:Fallen] 2015-04-05 08:37:28 PDT
Pushed to comm-central changeset 4d54b17e9075

Note You need to log in before you can comment on or make changes to this bug.