Merge lp:~allenap/launchpad/localpackagediffs-filter-by-package-set-bug-809786-refactor into lp:launchpad
Status: | Merged |
---|---|
Approved by: | Gavin Panella |
Approved revision: | no longer in the source branch. |
Merged at revision: | 13461 |
Proposed branch: | lp:~allenap/launchpad/localpackagediffs-filter-by-package-set-bug-809786-refactor |
Merge into: | lp:launchpad |
Diff against target: |
518 lines (+171/-88) 6 files modified
Makefile (+11/-5) lib/lp/app/javascript/testing/testrunner.js (+29/-0) lib/lp/registry/browser/distroseries.py (+4/-0) lib/lp/registry/javascript/distroseries.initseries.js (+55/-30) lib/lp/registry/javascript/tests/test_distroseries.initseries.html (+29/-16) lib/lp/registry/javascript/tests/test_distroseries.initseries.js (+43/-37) |
To merge this branch: | bzr merge lp:~allenap/launchpad/localpackagediffs-filter-by-package-set-bug-809786-refactor |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Benji York (community) | code | Approve | |
Review via email: mp+68233@code.launchpad.net |
Commit message
[r=benji][bug=809786] Refactor some DistroSeries JavaScript and add a declarative approach for running YUI tests.
Description of the change
This moves a few DistroSeries related bits of JavaScript around and
fixes some lint. The only thing that really needs reviewing is the new
code in lib/lp/
/**
* Merely loading this script into a page will cause it to look for a
* list of suites in the document using the selector ul#suites>li. If
* found, the text within each node is considered to be a test module
* name. This is then loaded, and its "suite" property passed to
* Runner.run().
*
* Here's how to declare the suites to run:
*
* <ul id="suites">
* <li>lp.
* </ul>
*
*/
The test is then declared as a regular YUI module instead of
containing test run code.
This branch looks good.