Merge lp://staging/~jkakar/landscape-client/grr-mocking into lp://staging/~landscape/landscape-client/trunk
Proposed by
Jamu Kakar
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Jamu Kakar | ||||
Approved revision: | 228 | ||||
Merged at revision: | not available | ||||
Proposed branch: | lp://staging/~jkakar/landscape-client/grr-mocking | ||||
Merge into: | lp://staging/~landscape/landscape-client/trunk | ||||
Diff against target: |
96 lines (+61/-3) 2 files modified
landscape/manager/eucalyptus.py (+2/-2) landscape/manager/tests/test_eucalyptus.py (+59/-1) |
||||
To merge this branch: | bzr merge lp://staging/~jkakar/landscape-client/grr-mocking | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Free Ekanayaka (community) | Approve | ||
Muharem Hrnjadovic (community) | Approve | ||
Review via email: mp+22929@code.staging.launchpad.net |
Description of the change
This branch fixes the function signatures for start_service_hub and
get_eucalyptus_
when they were refactored from methods to functions, and because of
the use of mocking they were not tested directly and so no tests
actually broke. I've fixed the signatures and added (mocking-based,
ugh) tests to ensure they can be called as expected.
To post a comment you must log in.
Took me a while to understand what's going :) Looks good!