-
Notifications
You must be signed in to change notification settings - Fork 12
Closed
Description
We desire to add some tests to validate this PR: StackStorm/st2#4976
From the changes in the PR, if possible we should test the monkey patch on the following services:
- st2actions/st2actions/cmd/st2notifier.py
- st2actions/st2actions/cmd/st2resultstracker.py
- st2auth/st2auth/cmd/api.py
- st2common/bin/paramiko_ssh_evenlets_tester.py
- st2exporter/st2exporter/cmd/st2exporter_starter.py
- st2reactor/st2reactor/cmd/garbagecollector.py
- st2reactor/st2reactor/cmd/rulesengine.py
- st2reactor/st2reactor/cmd/sensormanager.py
- st2reactor/st2reactor/cmd/timersengine.py
- st2reactor/st2reactor/container/sensor_wrapper.py (@punkrokk / @amanda11)
-
st2reactor/tests/unit/test_sensor_wrapper.py
I'm not sure we can test all of these. Hopefully @Kami Can shed some light on an approach. Searching the web fails because monkey patch is a common testing strategy.