Merge lp://staging/~abentley/juju-ci-tools/assess-cloud-display into lp://staging/juju-ci-tools

Proposed by Aaron Bentley
Status: Merged
Merged at revision: 1776
Proposed branch: lp://staging/~abentley/juju-ci-tools/assess-cloud-display
Merge into: lp://staging/juju-ci-tools
Diff against target: 312 lines (+291/-1)
3 files modified
assess_cloud_display.py (+123/-0)
jujupy.py (+4/-1)
tests/test_assess_cloud_display.py (+164/-0)
To merge this branch: bzr merge lp://staging/~abentley/juju-ci-tools/assess-cloud-display
Reviewer Review Type Date Requested Status
Curtis Hovey (community) code Approve
Review via email: mp+312182@code.staging.launchpad.net

Commit message

Add assess-cloud-display.

Description of the change

This branch tests the display of a supplied cloud config.

This enables us to test that add-cloud-interactive is interoperable with these display commands. Both list-clouds and show-cloud are tested.

In theory, main could be an actual unittest test runner, but that would take a lot longer for me to do, since I don't have the relevent experience.

To post a comment you must log in.
Revision history for this message
Curtis Hovey (sinzui) wrote :

Thank you.

review: Approve (code)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches