588665 - cfx's test runner package should be user-configurable

Status: RESOLVED FIXED

(Add-on SDK Graveyard :: General, defect)

Description

[Atul Varma [:atul]]

cfx currently uses the "test-harness" package to run tests, but if a developer wants to write their own test runner or use a third-party one, they shouldn't have to change the source code or replace the test-harness package.

So, it would be nice to have a "--test-runner-pkg" option.

Comment 1

[Atul Varma [:atul]]

Attachment: patch

Comment 2

[Atul Varma [:atul]]

I also wonder if it'd be useful to rename test-harness to test-runner, since that would be a more appropriate name for it.

Comment 3

[Myk Melez [:myk] [@mykmelez]]

Comment on attachment 467284 [details] [diff] [review]
patch

Looks good, seems to work, r=myk

(In reply to comment #2)
> I also wonder if it'd be useful to rename test-harness to test-runner, since
> that would be a more appropriate name for it.

Sure.  Note that you'll also have to update the name in test-packaging.js line 27:

  test.assertEqual(packaging.options.metadata['test-harness'].author,

Comment 4

[Atul Varma [:atul]]

Pushed!

Bug 588665 - cfx's test runner package should be user-configurable
Atul Varma [:atul]
http://hg.mozilla.org/labs/jetpack-sdk/rev/7c542d8c8981

Comment 5

[Myk Melez [:myk] [@mykmelez]]

The Add-on SDK is no longer a Mozilla Labs experiment and has become a big enough project to warrant its own Bugzilla product, so the "Add-on SDK" product has been created for it, and I am moving its bugs to that product.

To filter bugmail related to this change, filter on the word "looptid".