Sisyphus - missing crashtest in default unittest list

RESOLVED FIXED

Status

--
trivial
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: bc, Assigned: bc)

Tracking

Trunk
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

595 bytes, patch
cmtalbert
: review+
Details | Diff | Splinter Review
(Assignee)

Description

9 years ago
Created attachment 430705 [details] [diff] [review]
patch
Attachment #430705 - Flags: review?(ctalbert)
(Assignee)

Comment 1

9 years ago
Created attachment 431115 [details] [diff] [review]
patch v2

add crashtest, remove "browser-chrome" since it doesn't exist and is really mochitest-browser-chrome anyway.
Attachment #430705 - Attachment is obsolete: true
Attachment #431115 - Flags: review?(ctalbert)
Attachment #430705 - Flags: review?(ctalbert)

Comment 2

9 years ago
Comment on attachment 431115 [details] [diff] [review]
patch v2

r+ because you are indeed doing what you claim to be doing here: adding crashtest.  

However, is there a reason that you don't have xpcshell listed?
Attachment #431115 - Flags: review?(ctalbert) → review+
(Assignee)

Comment 3

9 years ago
I need to file a bug on xpcshell tests to get the log files sent to stdout. I'll do that this evening.
(Assignee)

Comment 4

9 years ago
http://hg.mozilla.org/qa/sisyphus/rev/652f613c18ee
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.