modules/fm-orchestrator
1
0
Fork 0
mirror of https://pagure.io/fm-orchestrator.git synced 2026-04-05 11:48:33 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Use patch.dict instead of modifying the environment variable in test_standalone_metrics_server

Browse Source
This commit is contained in:
mprahl
2020-01-06 16:45:04 -05:00
parent 5ecde6dc5f
commit 59980b1219
1 changed files with 8 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
tests/test_common/test_monitor.py
View File

@@ -38,15 +38,15 @@ def test_standalone_metrics_server_disabled_by_default():
def test_standalone_metrics_server():
os.environ["MONITOR_STANDALONE_METRICS_SERVER_ENABLE"] = "true"
reload_module(module_build_service.common.monitor)
with mock.patch.dict(os.environ, {"MONITOR_STANDALONE_METRICS_SERVER_ENABLE": "true"}):
reload_module(module_build_service.common.monitor)
r = requests.get("http://127.0.0.1:10040/metrics")
count = len([
l for l in r.text.splitlines()
if (l.startswith("# TYPE") and "_created " not in l)
])
assert count == num_of_metrics
r = requests.get("http://127.0.0.1:10040/metrics")
count = len([
l for l in r.text.splitlines()
if (l.startswith("# TYPE") and "_created " not in l)
])
assert count == num_of_metrics
@mock.patch("module_build_service.common.monitor.builder_failed_counter.labels")