Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Make use of tags for running only icaljs/libcal tests

RESOLVED FIXED in 4.2

Status

Calendar
Build Config
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Fallen, Assigned: Fallen)

Tracking

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
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.
(Assignee)

Comment 1

2 years ago
Created attachment 8587667 [details] [diff] [review]
Fix - v1
Attachment #8587667 - Flags: review?(geoff)
Comment on attachment 8587667 [details] [diff] [review]
Fix - v1

Nice. I've been wanting this for some time.
Attachment #8587667 - Flags: review?(geoff) → review+
(Assignee)

Comment 3

2 years ago
Pushed to comm-central changeset 4d54b17e9075
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 4.1
(Assignee)

Updated

2 years ago
Target Milestone: 4.1 → 4.2
You need to log in before you can comment on or make changes to this bug.