lp://qastaging/~tealeg/landscape-client/mocker-replace-test-hasservice

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp://qastaging/~landscape/landscape-client/trunk at revision 884

Alberto Donato (community): Approve on 2016-06-15

🤖 Landscape Builder: Approve (test results) on 2016-06-15

Chris Glass (community): Approve on 2016-06-15

Diff: 234 lines (+62/-61)

2 files modified

landscape/manager/haservice.py (+12/-11)
landscape/manager/tests/test_haservice.py (+50/-50)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Bug #1592875: remove mocker

Medium

Fix Committed

Link a bug report

Related blueprints

869. By Geoff Teale on 2016-06-15

lint

868. By Geoff Teale on 2016-06-15

Assert call parameters instead of returning them... whoooops

867. By Geoff Teale on 2016-06-15

lint

866. By Geoff Teale on 2016-06-15

test_run_success_cluster_online ported from mocker to mock.

865. By Geoff Teale on 2016-06-15

test_run_success_cluster_standby ported from mocker to mock. *phew*

864. By Geoff Teale on 2016-06-15

test_incorrect_juju_unit uses patch instead of mocker.

863. By Geoff Teale on 2016-06-15

test_not_a_juju_computer uses patch instead of mocker.

862. By Geoff Teale on 2016-06-15

test_invalid_server_service_state_request uses patch instead of mocker.

861. By Simon Poirier on 2016-06-15

Merge mocker_to_mock_test_facade [f=1592875] [r=bogdana,landscape-builder,danilo] [a=Simon Poirier]
This branches migrates to mock l/p/t/test_facade

860. By Bogdana Vereha on 2016-06-15

Merge mocker-monitor-test-monitor [f=1592875] [r=tribaal,chad.smith,landscape-builder] [a=Bogdana Vereha]
Update monitor.tests.test_monitor to use mock