lp:~frankban/juju-quickstart/initial-vivid-support
- Get this branch:
- bzr branch lp:~frankban/juju-quickstart/initial-vivid-support
Branch merges
- Juju GUI Hackers: Pending requested
-
Diff: 202 lines (+36/-28)6 files modifiedquickstart/__init__.py (+1/-1)
quickstart/app.py (+3/-3)
quickstart/netutils.py (+3/-3)
quickstart/settings.py (+9/-7)
quickstart/tests/test_app.py (+9/-9)
quickstart/tests/test_netutils.py (+11/-5)
Branch information
Recent revisions
- 112. By Francesco Banconi
-
Reorganize view code.
Replace most of the view closures with external
private functions.
This has several good side effects in my opinion:
- view functions are shorter;
- we know which objects are used for each callback
function, because we need to explicitly pass them
as arguments;
- external functions can be reused by multiple views
if required (and we have the _use example here);
- if an external callback become less straightforward,
it is now possible to directly unit test it.Tests: `make check`.
QA:
If unit tests pass we have some confidence that
the interactive session is not broken.
However, some time spent on it would be great,
creating/editing/ destroying environments, setting
them as default, canceling the changes, starting
one of them, etc...
Thanks a lot.R=matthew.scott, bac
CC=
https://codereview. appspot. com/194080043
Branch metadata
- Branch format:
- Branch format 7
- Repository format:
- Bazaar repository format 2a (needs bzr 1.16 or later)
- Stacked on:
- lp:juju-quickstart