Merge lp://qastaging/~frankban/juju-quickstart/views-params into lp://qastaging/juju-quickstart
Proposed by
Francesco Banconi
Status: | Merged |
---|---|
Merged at revision: | 111 |
Proposed branch: | lp://qastaging/~frankban/juju-quickstart/views-params |
Merge into: | lp://qastaging/juju-quickstart |
Diff against target: |
975 lines (+301/-172) 6 files modified
quickstart/cli/params.py (+51/-0) quickstart/cli/views.py (+70/-74) quickstart/manage.py (+11/-4) quickstart/tests/cli/test_params.py (+87/-0) quickstart/tests/cli/test_views.py (+71/-90) quickstart/tests/test_manage.py (+11/-4) |
To merge this branch: | bzr merge lp://qastaging/~frankban/juju-quickstart/views-params |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Juju GUI Hackers | Pending | ||
Review via email: mp+245753@code.qastaging.launchpad.net |
Description of the change
Refactor view parameters.
Implement a Params object used to store
common view parameters. This way it will
be easier to add parameters in the future
(e.g. one callable to remove stale jenv files).
New code include the params module and a fix
to the code handling the listing of jenv files
in the index view: now the header message is
only displayed if jenv files actually exist.
The rest of the diff is mechanical: i.e. replacing
the single view arguments with the params named
tuple.
Tests: `make check`.
QA:
Use the quickstart interactive session and
check everything works ok.
To post a comment you must log in.
Reviewers: mp+245753_ code.launchpad. net,
Message:
Please take a look.
Description:
Refactor view parameters.
Implement a Params object used to store
common view parameters. This way it will
be easier to add parameters in the future
(e.g. one callable to remove stale jenv files).
New code include the params module and a fix
to the code handling the listing of jenv files
in the index view: now the header message is
only displayed if jenv files actually exist.
The rest of the diff is mechanical: i.e. replacing
the single view arguments with the params named
tuple.
Tests: `make check`.
QA:
Use the quickstart interactive session and
check everything works ok.
https:/ /code.launchpad .net/~frankban/ juju-quickstart /views- params/ +merge/ 245753
(do not edit description out of merge proposal)
Please review this at https:/ /codereview. appspot. com/194030043/
Affected files (+303, -172 lines): cli/params. py cli/views. py manage. py tests/cli/ test_params. py tests/cli/ test_views. py tests/test_ manage. py
A [revision details]
A quickstart/
M quickstart/
M quickstart/
A quickstart/
M quickstart/
M quickstart/