Merge ~sudeephb/charm-prometheus-alertmanager:unittest_fix into charm-prometheus-alertmanager:master
Proposed by
Sudeep Bhandari
Status: | Merged |
---|---|
Approved by: | Ramesh Sattaru |
Approved revision: | b74426c1f470d62d2a2e32ea8a30d30f1263fc01 |
Merged at revision: | ad28b271cf03aed41094530e7079f320ef2bb41f |
Proposed branch: | ~sudeephb/charm-prometheus-alertmanager:unittest_fix |
Merge into: | charm-prometheus-alertmanager:master |
Diff against target: |
25 lines (+2/-4) 2 files modified
src/tests/unit/requirements.txt (+0/-1) src/tests/unit/test_reactive_alertmanager.py (+2/-3) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Ramesh Sattaru (community) | Approve | ||
Erhan Sunar (community) | Approve | ||
🤖 prod-jenkaas-bootstack (community) | continuous-integration | Approve | |
Review via email: mp+436510@code.launchpad.net |
Commit message
Use unittest.mock instead of external mock library.
Description of the change
Use unittest.mock instead of mock(https:/
To post a comment you must log in.
This merge proposal is being monitored by mergebot. Change the status to Approved to merge.