Merge lp://staging/~sinzui/juju-ci-tools/manual-add-cloud into lp://staging/juju-ci-tools
Proposed by
Curtis Hovey
| Status: | Merged |
|---|---|
| Merged at revision: | 1979 |
| Proposed branch: | lp://staging/~sinzui/juju-ci-tools/manual-add-cloud |
| Merge into: | lp://staging/juju-ci-tools |
| Diff against target: |
305 lines (+119/-41) 2 files modified
assess_add_cloud.py (+47/-12) tests/test_assess_add_cloud.py (+72/-29) |
| To merge this branch: | bzr merge lp://staging/~sinzui/juju-ci-tools/manual-add-cloud |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Juju Release Engineering | Pending | ||
|
Review via email:
|
|||
Description of the change
Updated manual add-cloud tests.
This branch updated the manual provider scenario to match the recent changes in Juju. The cloud-spec, long name and invalid name tests are now the same as other providers. The long endpoint tests were reset to basic (because that requires authentication).
This branch introduced the CloudValidation to help determine the which validation is supported by a version of Juju and for a provider. We can extend it to ask questions about support instead of adding exceptions and complicated logic to the iter_clouds().
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
