Merge lp://staging/~veebers/juju-ci-tools/adding_model_destory_test into lp://staging/juju-ci-tools
| Status: | Needs review |
|---|---|
| Proposed branch: | lp://staging/~veebers/juju-ci-tools/adding_model_destory_test |
| Merge into: | lp://staging/juju-ci-tools |
| Diff against target: |
266 lines (+257/-0) 2 files modified
assess_destroy_model.py (+122/-0) tests/test_assess_destroy_model.py (+135/-0) |
| To merge this branch: | bzr merge lp://staging/~veebers/juju-ci-tools/adding_model_destory_test |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Curtis Hovey (community) | code | Approve | |
|
Review via email:
|
|||
Commit message
Add functional test for bug: https:/
Description of the change
Add functional test for bug: https:/
This test was request by core devs in case of regression.
It does a couple of things against the grain of normal usage of jujupy, namely using the 'switch' command. This is due to the nature of the bug and test.
It also has a gross workaround where it uses a sleep around the creation and deletion of a model due to bug https:/
Tests that deleting the in-focus model removes the current model and the ability to switch to a new model to set it.
Unmerged revisions
- 1561. By Christopher Lee
-
Actually use unit tests and with a fix.
- 1560. By Christopher Lee
-
Add gross workaround for bug (1613960)
- 1559. By Christopher Lee
-
Add reference to bug this test is to cover
- 1558. By Christopher Lee
-
Fix status command check
- 1557. By Christopher Lee
-
Fixed call to 'switch'.
- 1556. By Christopher Lee
-
Initial pass of simple test
Thank you.