New trace-test harness format is not tinderbox-compatible.

RESOLVED FIXED

Status

()

RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: graydon, Assigned: dmandelin)

Tracking

Trunk
x86
Windows Vista
Points:
---
Bug Flags:
wanted1.9.2 +
in-testsuite -

Firefox Tracking Flags

(status1.9.2 beta1-fixed)

Details

(Whiteboard: fixed-in-tracemonkey)

Attachments

(1 attachment)

(Reporter)

Description

9 years ago
The old trace-test format was something tinderbox could parse (not initially, but I changed it to be so before wiring it into "make check"). Unless I'm not seeing it, I don't think the new one writes out such a format. Therefore trace-test just went from "being checked by tinderbox" to "being ignored by tinderbox".

Maybe make the "make check" invocation pass a flag that makes it print in tinderbox-compatible form? Like the old one.
(Assignee)

Comment 1

9 years ago
Sounds easy enough. What's the format? Is there a spec? I looked at the Tinderbox MDC article but I didn't find anything like that. The closest I found was https://developer.mozilla.org/en/How_to_add_a_build-time_test, which (apparently wrongly) says that zero-vs-nonzero exit code is enough.

Comment 2

9 years ago
That document sort of assumes you're not doing a long series of tests and assertions, in which case not printing out play-by-play details on each test is fine.  For something like this, however, you want output more like what Mochitest prints out when it runs a directory of tests; the best documentation for that at the moment is really just a log of a Mochitest run.  (It's probably worth adding that to the above page, just probably hasn't occurred to anyone yet.)
(Assignee)

Comment 3

9 years ago
Created attachment 394932 [details] [diff] [review]
Patch
Attachment #394932 - Flags: review?
(Assignee)

Updated

9 years ago
Attachment #394932 - Flags: review? → review?(graydon)
(Reporter)

Updated

9 years ago
Attachment #394932 - Flags: review?(graydon) → review+
(Assignee)

Comment 4

9 years ago
Pushed to TM as bbe749f9502e.
Whiteboard: fixed-in-tracemonkey

Comment 5

9 years ago
http://hg.mozilla.org/mozilla-central/rev/bbe749f9502e
Status: NEW → RESOLVED
Last Resolved: 9 years ago
Resolution: --- → FIXED

Comment 6

9 years ago
http://hg.mozilla.org/releases/mozilla-1.9.2/rev/b81225219a74
status1.9.2: --- → beta1-fixed
Flags: wanted1.9.2+

Updated

9 years ago
Flags: in-testsuite-
You need to log in before you can comment on or make changes to this bug.