Merge lp:~bac/juju-gui/1083935 into lp:juju-gui/experimental
Proposed by
Brad Crittenden
Status: | Merged |
---|---|
Merged at revision: | 294 |
Proposed branch: | lp:~bac/juju-gui/1083935 |
Merge into: | lp:juju-gui/experimental |
Diff against target: |
221 lines (+146/-7) 6 files modified
app/views/topology/mega.js (+4/-4) app/views/topology/panzoom.js (+0/-1) package.json (+1/-1) test/index.html (+1/-0) test/test_application_notifications.js (+2/-1) test/test_panzoom.js (+138/-0) |
To merge this branch: | bzr merge lp:~bac/juju-gui/1083935 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Juju GUI Hackers | Pending | ||
Review via email: mp+141135@code.launchpad.net |
Description of the change
Add tests for panzoom.
Tests are added for panzoom. Due to agreed upon time-boxing, the testing is
not thorough but is a start. Another bug to complete the testing will be
filed.
To post a comment you must log in.
Reviewers: mp+141135_ code.launchpad. net,
Message:
Please take a look.
Description:
Add tests for panzoom.
Tests are added for panzoom. Due to agreed upon time-boxing, the
testing is
not thorough but is a start. Another but to complete the testing will
be
filed.
https:/ /code.launchpad .net/~bac/ juju-gui/ 1083935/ +merge/ 141135
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/7001047/
Affected files: topology/ mega.js topology/ panzoom. js application_ notifications. js panzoom. js
A [revision details]
M app/views/
M app/views/
M test/index.html
M test/test_
A test/test_