Closed Bug 919635 Opened 6 years ago Closed 6 years ago

Support Steeplechase tests in tree

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED
mozilla27

People

(Reporter: ted, Assigned: ted)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

For the new Steeplechase test harness I'm writing we want test manifests. This patch builds on the patch from bug 901990 to add test manifests for Steeplechase. Fairly simple patch.
Blocks: steeplechase
Depends on: 901990
The next step here is to package these tests into the test package.
Comment on attachment 808686 [details] [diff] [review]
Add manifests for Steeplechase tests to moz.build

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

This is almost too simple.
Attachment #808686 - Flags: review?(gps) → review+
I hate test packaging. I think I'm going to move that all into moz.build soon. In the interest of getting this landed I'll go with this instead.
Attachment #809311 - Flags: review?(gps)
Comment on attachment 809311 [details] [diff] [review]
package Steeplechase tests and extra bits in test package

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

I can't wait for this make file to die.
Attachment #809311 - Flags: review?(gps) → review+
Once more with feeling (and a one-line whitespace docstring comment fix to fix that pedantic unit test):
remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/162e33423f59
remote:   https://hg.mozilla.org/integration/mozilla-inbound/rev/f4087a24f020
https://hg.mozilla.org/mozilla-central/rev/162e33423f59
https://hg.mozilla.org/mozilla-central/rev/f4087a24f020
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.