Landscape Client

remove mocker

Bug #1592875 reported by David Britton
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Landscape Client
Fix Committed
Medium
Unassigned
Landscape Client backlog

Bug Description

Replace with mock that is in the stdlib in python3.

Related branches

lp://staging/~benji/landscape-client/remove-mocker-test_usermanager
Benji York (community): Approve (feng shui)
Geoff Teale (community): Approve
Bogdana Vereha (community): Approve
Diff: 24 lines (+6/-5)
1 file modified
landscape/manager/tests/test_usermanager.py (+6/-5)
lp://staging/~tealeg/landscape-client/mocker-replace-test-fs
🤖 Landscape Builder: Approve (test results)
Geoff Teale (community): Approve
Chad Smith: Approve
Данило Шеган (community): Approve
Diff: 51 lines (+14/-11)
1 file modified
landscape/lib/tests/test_fs.py (+14/-11)
lp://staging/~chad.smith/landscape-client/mocker-to-mock-cephusage
Chad Smith: Abstain
Benji York (community): Approve
Geoff Teale (community): Approve
Diff: 34 lines (+7/-6)
1 file modified
landscape/monitor/tests/test_cephusage.py (+7/-6)
lp://staging/~bogdana/landscape-client/mocker-test-service
Bogdana Vereha (community): Approve
Данило Шеган (community): Approve
Chad Smith: Approve
Diff: 38 lines (+10/-8)
1 file modified
landscape/broker/tests/test_service.py (+10/-8)
lp://staging/~danilo/landscape-client/mock-tests-deployment
🤖 Landscape Builder: Approve (test results)
Данило Шеган (community): Approve
Bogdana Vereha (community): Approve
Free Ekanayaka (community): Approve
Diff: 62 lines (+10/-19)
1 file modified
landscape/tests/test_deployment.py (+10/-19)
lp://staging/~bogdana/landscape-client/remove-mocker-test-service
🤖 Landscape Builder: Approve (test results)
Benji York (community): Approve
Free Ekanayaka (community): Approve
Diff: 28 lines (+4/-4)
1 file modified
landscape/tests/test_service.py (+4/-4)
lp://staging/~free.ekanayaka/landscape-client/mockify-lib-network-tests
Chad Smith: Approve
🤖 Landscape Builder: Approve (test results)
Chris Glass (community): Approve
Diff: 287 lines (+89/-97)
1 file modified
landscape/lib/tests/test_network.py (+89/-97)
lp://staging/~tribaal/landscape-client/remove-mocker-process
🤖 Landscape Builder: Approve (test results)
Free Ekanayaka (community): Approve
Benji York (community): Approve
Diff: 60 lines (+16/-15)
1 file modified
landscape/lib/tests/test_process.py (+16/-15)
lp://staging/~chad.smith/landscape-client/mocker-to-mock-computer-uptime
Simon Poirier (community): Approve
Free Ekanayaka (community): Approve
🤖 Landscape Builder: Approve (test results)
Chad Smith: Approve
Diff: 28 lines (+3/-4)
1 file modified
landscape/monitor/tests/test_computeruptime.py (+3/-4)
lp://staging/~free.ekanayaka/landscape-client/mockify-lib-disk-tests
Chad Smith: Approve
Chris Glass (community): Approve
🤖 Landscape Builder: Approve (test results)
Diff: 224 lines (+49/-56)
1 file modified
landscape/lib/tests/test_disk.py (+49/-56)
lp://staging/~simpoir/landscape-client/mocker_to_mock_test_facade
Bogdana Vereha (community): Approve
🤖 Landscape Builder: Approve (test results)
Данило Шеган (community): Approve
Diff: 120 lines (+32/-32)
1 file modified
landscape/package/tests/test_facade.py (+32/-32)
lp://staging/~tribaal/landscape-client/remove-mocker-temperature
🤖 Landscape Builder: Approve (test results)
Bogdana Vereha (community): Approve
Chad Smith: Approve
Diff: 33 lines (+6/-7)
1 file modified
landscape/monitor/tests/test_temperature.py (+6/-7)
lp://staging/~bogdana/landscape-client/mocker-monitor-test-memoryinfo
🤖 Landscape Builder: Approve (test results)
Chris Glass (community): Approve
Chad Smith: Approve
Diff: 30 lines (+7/-7)
1 file modified
landscape/monitor/tests/test_memoryinfo.py (+7/-7)
lp://staging/~danilo/landscape-client/mock-tests-broker-exchange
🤖 Landscape Builder: Approve (test results)
Free Ekanayaka (community): Approve
Chris Glass (community): Approve
Diff: 55 lines (+12/-9)
1 file modified
landscape/broker/tests/test_exchange.py (+12/-9)
lp://staging/~free.ekanayaka/landscape-client/mockify-lib-bootstrap-tests
Simon Poirier (community): Approve
🤖 Landscape Builder: Approve (test results)
Chris Glass (community): Approve
Diff: 158 lines (+63/-68)
1 file modified
landscape/lib/tests/test_bootstrap.py (+63/-68)
lp://staging/~chad.smith/landscape-client/mocker-to-mock-swiftusage
🤖 Landscape Builder: Approve (test results)
Chris Glass (community): Approve
Free Ekanayaka (community): Approve
Diff: 29 lines (+3/-7)
1 file modified
landscape/monitor/tests/test_swiftusage.py (+3/-7)
lp://staging/~tribaal/landscape-client/remove-mocker-load-average
Geoff Teale (community): Approve
🤖 Landscape Builder: Approve (test results)
Bogdana Vereha (community): Approve
Diff: 30 lines (+7/-7)
1 file modified
landscape/monitor/tests/test_loadaverage.py (+7/-7)
lp://staging/~tribaal/landscape-client/remove-mocker-network-devices
🤖 Landscape Builder: Approve (test results)
Данило Шеган (community): Approve
Benji York (community): Approve
Diff: 25 lines (+5/-6)
1 file modified
landscape/monitor/tests/test_networkdevice.py (+5/-6)
lp://staging/~tribaal/landscape-client/remove-mocker-reboot-required
Chad Smith: Approve
🤖 Landscape Builder: Approve (test results)
Geoff Teale (community): Approve
Diff: 27 lines (+6/-5)
1 file modified
landscape/monitor/tests/test_rebootrequired.py (+6/-5)
lp://staging/~tealeg/landscape-client/mocker-replace-test-fd
🤖 Landscape Builder: Approve (test results)
Chris Glass (community): Approve
Free Ekanayaka (community): Approve
Diff: 119 lines (+35/-40)
1 file modified
landscape/lib/tests/test_fd.py (+35/-40)
lp://staging/~chad.smith/landscape-client/mocker-to-mock-processorinfo
🤖 Landscape Builder: Approve (test results)
Bogdana Vereha (community): Approve
Alberto Donato: Approve
Diff: 34 lines (+6/-5)
1 file modified
landscape/monitor/tests/test_processorinfo.py (+6/-5)
lp://staging/~bogdana/landscape-client/mocker-monitor-test-monitor
Chris Glass (community): Approve
Chad Smith: Approve
🤖 Landscape Builder: Approve (test results)
Diff: 25 lines (+6/-4)
1 file modified
landscape/monitor/tests/test_monitor.py (+6/-4)
lp://staging/~ack/landscape-client/mock-test-aptsources
Benji York (community): Approve
Chris Glass (community): Approve
🤖 Landscape Builder: Approve (test results)
Diff: 75 lines (+30/-20)
1 file modified
landscape/manager/tests/test_aptsources.py (+30/-20)
lp://staging/~tribaal/landscape-client/remove-mocker-test-updatemanager
Bogdana Vereha (community): Approve
🤖 Landscape Builder: Approve (test results)
Chad Smith: Approve
Diff: 27 lines (+6/-5)
1 file modified
landscape/monitor/tests/test_updatemanager.py (+6/-5)
lp://staging/~bogdana/landscape-client/mocker-monitor-test-service
Chad Smith: Approve
Данило Шеган (community): Approve
🤖 Landscape Builder: Approve (test results)
Diff: 27 lines (+8/-7)
1 file modified
landscape/monitor/tests/test_service.py (+8/-7)
lp://staging/~tribaal/landscape-client/remove-mocker-computerinfo
Данило Шеган (community): Approve
Bogdana Vereha (community): Approve
🤖 Landscape Builder: Approve (test results)
Diff: 56 lines (+13/-15)
1 file modified
landscape/monitor/tests/test_computerinfo.py (+13/-15)
lp://staging/~chad.smith/landscape-client/mocker-to-mock-activeprocess
🤖 Landscape Builder: Approve (test results)
Adam Collard (community): Approve
Simon Poirier (community): Approve
Diff: 34 lines (+5/-5)
1 file modified
landscape/monitor/tests/test_activeprocessinfo.py (+5/-5)
lp://staging/~danilo/landscape-client/mock-tests-sysinfo
🤖 Landscape Builder: Approve (test results)
Adam Collard (community): Approve
Bogdana Vereha (community): Approve
Diff: 44 lines (+6/-7)
1 file modified
landscape/sysinfo/tests/test_sysinfo.py (+6/-7)
lp://staging/~bogdana/landscape-client/mocker-monitor-test-usermonitor
Chad Smith: Approve
Free Ekanayaka (community): Approve
🤖 Landscape Builder: Approve (test results)
Diff: 59 lines (+10/-10)
1 file modified
landscape/monitor/tests/test_usermonitor.py (+10/-10)
lp://staging/~simpoir/landscape-client/mocker_to_mock_test_load
🤖 Landscape Builder: Approve (test results)
Free Ekanayaka (community): Approve
Bogdana Vereha (community): Approve
Diff: 26 lines (+5/-6)
1 file modified
landscape/sysinfo/tests/test_load.py (+5/-6)
lp://staging/~adam-collard/landscape-client/mocker-replace-configuration
Chad Smith: Approve
🤖 Landscape Builder: Approve (test results)
Chris Glass (community): Approve
Diff: 916 lines (+289/-365)
1 file modified
landscape/tests/test_configuration.py (+289/-365)
lp://staging/~free.ekanayaka/landscape-client/mockify-package-taskhandler-tests
Bogdana Vereha (community): Approve
Chris Glass (community): Approve
🤖 Landscape Builder: Approve (test results)
Diff: 447 lines (+115/-155)
1 file modified
landscape/package/tests/test_taskhandler.py (+115/-155)
lp://staging/~ack/landscape-client/migrate-test-scriptexecution
Chris Glass (community): Approve
Free Ekanayaka (community): Approve
🤖 Landscape Builder: Approve (test results)
Diff: 297 lines (+104/-63)
1 file modified
landscape/manager/tests/test_scriptexecution.py (+104/-63)
lp://staging/~tribaal/landscape-client/remove-mocker-amq
🤖 Landscape Builder: Approve (test results)
Alberto Donato: Approve
Simon Poirier (community): Approve
Diff: 119 lines (+28/-32)
1 file modified
landscape/broker/tests/test_amp.py (+28/-32)
lp://staging/~bogdana/landscape-client/mocker-monitor-test-mountinfo
Alberto Donato: Approve
🤖 Landscape Builder: Approve (test results)
Chris Glass (community): Approve
Diff: 76 lines (+14/-19)
1 file modified
landscape/monitor/tests/test_mountinfo.py (+14/-19)
lp://staging/~bogdana/landscape-client/mocker-monitor-test-plugin
Benji York (community): Approve
🤖 Landscape Builder: Approve (test results)
Chris Glass (community): Approve
Diff: 46 lines (+8/-12)
1 file modified
landscape/monitor/tests/test_plugin.py (+8/-12)
lp://staging/~tribaal/landscape-client/remove-mocker-apt-preferences
Bogdana Vereha (community): Approve
Alberto Donato: Approve
🤖 Landscape Builder: Approve (test results)
Diff: 41 lines (+9/-10)
1 file modified
landscape/monitor/tests/test_aptpreferences.py (+9/-10)
lp://staging/~tribaal/landscape-client/remove-mocker-sysvconfig
Bogdana Vereha (community): Approve
Chad Smith: Approve
🤖 Landscape Builder: Approve (test results)
Diff: 62 lines (+18/-18)
1 file modified
landscape/tests/test_sysvconfig.py (+18/-18)
lp://staging/~bogdana/landscape-client/mocker-package-test-store
Geoff Teale (community): Approve
Chad Smith: Approve
🤖 Landscape Builder: Approve (test results)
Diff: 95 lines (+16/-50)
1 file modified
landscape/package/tests/test_store.py (+16/-50)
lp://staging/~chad.smith/landscape-client/mocker-to-mock-processkiller
Chris Glass (community): Approve
🤖 Landscape Builder: Approve (test results)
Bogdana Vereha (community): Approve
Diff: 58 lines (+7/-9)
1 file modified
landscape/manager/tests/test_processkiller.py (+7/-9)
lp://staging/~tribaal/landscape-client/remove-mocker-broker-client
🤖 Landscape Builder: Approve (test results)
Bogdana Vereha (community): Approve
Geoff Teale (community): Approve
Diff: 255 lines (+51/-53)
1 file modified
landscape/broker/tests/test_client.py (+51/-53)
lp://staging/~simpoir/landscape-client/mocker_to_mock_test_store
Chad Smith: Approve
Chris Glass (community): Approve
🤖 Landscape Builder: Approve (test results)
Diff: 46 lines (+11/-13)
1 file modified
landscape/broker/tests/test_store.py (+11/-13)
lp://staging/~tealeg/landscape-client/mocker-replace-test-hasservice
Alberto Donato: Approve
🤖 Landscape Builder: Approve (test results)
Chris Glass (community): Approve
Diff: 234 lines (+62/-61)
2 files modified
landscape/manager/haservice.py (+12/-11)
landscape/manager/tests/test_haservice.py (+50/-50)
lp://staging/~ack/landscape-client/migrate-test-scriptexecution-2
Adam Collard (community): Approve
Chris Glass (community): Approve
🤖 Landscape Builder: Approve (test results)
Diff: 335 lines (+83/-105)
1 file modified
landscape/manager/tests/test_scriptexecution.py (+83/-105)
lp://staging/~danilo/landscape-client/mock-tests-sysinfo-deployment
🤖 Landscape Builder: Approve (test results)
Benji York (community): Approve
Adam Collard (community): Approve
Diff: 180 lines (+64/-68)
1 file modified
landscape/sysinfo/tests/test_deployment.py (+64/-68)
lp://staging/~ack/landscape-client/mocker-replace-test-customgraph
Chris Glass (community): Approve
Bogdana Vereha (community): Approve
🤖 Landscape Builder: Approve (test results)
Diff: 142 lines (+26/-27)
1 file modified
landscape/manager/tests/test_customgraph.py (+26/-27)
lp://staging/~adam-collard/landscape-client/mocker-replace-configuration-part-deux
Данило Шеган (community): Approve
🤖 Landscape Builder: Approve (test results)
Chris Glass (community): Approve
Diff: 1387 lines (+397/-489)
1 file modified
landscape/tests/test_configuration.py (+397/-489)
lp://staging/~benji/landscape-client/demockerify-test_packagemonitor.py
Chad Smith: Approve
🤖 Landscape Builder: Approve (test results)
Adam Collard (community): Approve
Diff: 247 lines (+63/-70)
1 file modified
landscape/monitor/tests/test_packagemonitor.py (+63/-70)
lp://staging/~adam-collard/landscape-client/mocker-replace-configuration-iii
Simon Poirier (community): Approve
Chris Glass (community): Approve
🤖 Landscape Builder: Approve (test results)
Diff: 65 lines (+9/-20)
1 file modified
landscape/tests/test_configuration.py (+9/-20)
lp://staging/~simpoir/landscape-client/mocker_to_mock_test_registration
Benji York (community): Approve
🤖 Landscape Builder: Approve (test results)
Chris Glass (community): Approve
Diff: 184 lines (+26/-42)
1 file modified
landscape/broker/tests/test_registration.py (+26/-42)
lp://staging/~tribaal/landscape-client/remove-mocker-release-upgrade
Simon Poirier (community): Approve
Bogdana Vereha (community): Approve
🤖 Landscape Builder: Approve (test results)
Diff: 306 lines (+69/-78)
1 file modified
landscape/package/tests/test_releaseupgrader.py (+69/-78)
lp://staging/~bogdana/landscape-client/mocker-manager-test-packagemanager
Adam Collard (community): Approve
🤖 Landscape Builder: Approve (test results)
Chris Glass (community): Approve
Diff: 418 lines (+110/-132)
1 file modified
landscape/manager/tests/test_packagemanager.py (+110/-132)
lp://staging/~benji/landscape-client/demockerify-test_activeprocessinfo.py
🤖 Landscape Builder: Approve (test results)
Chad Smith: Approve
Simon Poirier (community): Approve
Diff: 50 lines (+16/-17)
1 file modified
landscape/monitor/tests/test_activeprocessinfo.py (+16/-17)
lp://staging/~chad.smith/landscape-client/mocker-to-mock-package-reporter
🤖 Landscape Builder: Approve (test results)
Chris Glass (community): Approve
Adam Collard (community): Approve
Diff: 214 lines (+37/-54)
1 file modified
landscape/package/tests/test_reporter.py (+37/-54)
Landscape: Pending requested
Landscape: Pending requested
Diff: 336 lines (+53/-117)
2 files modified
landscape/package/tests/test_reporter.py (+37/-53)
landscape/package/tests/test_store.py (+16/-64)
lp://staging/~adam-collard/landscape-client/mocker-replace-watchdog-run-tests
Free Ekanayaka (community): Approve
🤖 Landscape Builder: Approve (test results)
Chris Glass (community): Approve
Diff: 98 lines (+23/-25)
1 file modified
landscape/tests/test_watchdog.py (+23/-25)
lp://staging/~ack/landscape-client/mocker-out-test-watchdog
Simon Poirier (community): Approve
Benji York (community): Approve
🤖 Landscape Builder: Approve (test results)
Diff: 294 lines (+109/-83)
1 file modified
landscape/tests/test_watchdog.py (+109/-83)
lp://staging/~benji/landscape-client/mocker-purge-test_reporter.py
🤖 Landscape Builder: Approve (test results)
Bogdana Vereha (community): Approve
Simon Poirier (community): Approve
Diff: 138 lines (+37/-35)
1 file modified
landscape/package/tests/test_reporter.py (+37/-35)
lp://staging/~tealeg/landscape-client/mocker-replace-test-changer
🤖 Landscape Builder: Approve (test results)
Simon Poirier (community): Approve
Alberto Donato: Approve
Diff: 460 lines (+125/-206)
1 file modified
landscape/package/tests/test_changer.py (+125/-206)
lp://staging/~adam-collard/landscape-client/mocker-replace-watchdog-service-tests
Adam Collard (community): Approve
Bogdana Vereha (community): Approve
Alberto Donato: Approve
🤖 Landscape Builder: Approve (test results)
Diff: 426 lines (+111/-204)
1 file modified
landscape/tests/test_watchdog.py (+111/-204)
lp://staging/~chad.smith/landscape-client/mocker-to-mock-package-reporter-2
Alberto Donato: Approve
Chris Glass (community): Approve
Geoff Teale (community): Approve
🤖 Landscape Builder: Approve (test results)
Diff: 252 lines (+56/-70)
1 file modified
landscape/package/tests/test_reporter.py (+56/-70)
Landscape: Pending requested
Landscape: Pending requested
Diff: 5508 lines (+1402/-1726)
20 files modified
landscape/broker/tests/test_client.py (+51/-53)
landscape/broker/tests/test_registration.py (+26/-42)
landscape/broker/tests/test_store.py (+11/-13)
landscape/lib/cloud.py (+1/-1)
landscape/manager/haservice.py (+12/-11)
landscape/manager/tests/test_customgraph.py (+26/-27)
landscape/manager/tests/test_haservice.py (+50/-50)
landscape/manager/tests/test_packagemanager.py (+110/-132)
landscape/manager/tests/test_processkiller.py (+7/-9)
landscape/manager/tests/test_scriptexecution.py (+83/-105)
landscape/monitor/tests/test_activeprocessinfo.py (+16/-17)
landscape/monitor/tests/test_packagemonitor.py (+63/-70)
landscape/package/tests/test_changer.py (+125/-206)
landscape/package/tests/test_releaseupgrader.py (+69/-78)
landscape/package/tests/test_reporter.py (+130/-159)
landscape/package/tests/test_store.py (+16/-64)
landscape/package/tests/test_taskhandler.py (+2/-2)
landscape/sysinfo/tests/test_deployment.py (+65/-70)
landscape/tests/test_configuration.py (+406/-509)
landscape/tests/test_watchdog.py (+133/-108)
lp://staging/~adam-collard/landscape-client/mocker-replace-reporter-run
🤖 Landscape Builder: Approve (test results)
Chad Smith: Approve
Geoff Teale (community): Approve
Diff: 300 lines (+50/-81)
1 file modified
landscape/package/tests/test_reporter.py (+50/-81)
lp://staging/~danilo/landscape-client/mock-tests-watchdog-middle
🤖 Landscape Builder: Approve (test results)
Geoff Teale (community): Approve
Adam Collard (community): Approve
Diff: 201 lines (+66/-50)
1 file modified
landscape/tests/test_watchdog.py (+66/-50)
lp://staging/~adam-collard/landscape-client/drop-mocker
Björn Tillenius (community): Approve
Alberto Donato: Approve
🤖 Landscape Builder: Approve (test results)
Diff: 2287 lines (+51/-2164)
3 files modified
Makefile (+1/-1)
landscape/tests/helpers.py (+50/-8)
landscape/tests/mocker.py (+0/-2155)
Free Ekanayaka (free.ekanayaka)
Changed in landscape-client:
importance: Undecided → Medium
status: New → In Progress
🤖 Landscape Builder (landscape-builder)
Changed in landscape-client:
milestone: none → backlog
status: In Progress → Fix Committed
Данило Шеган (danilo)
Changed in landscape-client:
status: Fix Committed → In Progress
🤖 Landscape Builder (landscape-builder)
Changed in landscape-client:
status: In Progress → Fix Committed
Adam Collard (adam-collard)
Changed in landscape-client:
status: Fix Committed → In Progress
🤖 Landscape Builder (landscape-builder)
Changed in landscape-client:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.