Merge ~aluria/charm-prometheus-ceph-exporter:fix-tests into charm-prometheus-ceph-exporter:master

Proposed by Alvaro Uria
Status: Merged
Approved by: Drew Freiberger
Approved revision: 6d918b4a4520cb24e6065f41e519b05b69173060
Merged at revision: 6d918b4a4520cb24e6065f41e519b05b69173060
Proposed branch: ~aluria/charm-prometheus-ceph-exporter:fix-tests
Merge into: charm-prometheus-ceph-exporter:master
Diff against target: 31 lines (+5/-0)
2 files modified
src/tests/functional/tests/tests.yaml (+3/-0)
src/tests/unit/test_reactive_ceph_exporter.py (+2/-0)
Reviewer Review Type Date Requested Status
Drew Freiberger (community) Approve
BootStack Reviewers Pending
Review via email: mp+392396@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Drew Freiberger (afreiberger) wrote :

Lgtm

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/src/tests/functional/tests/tests.yaml b/src/tests/functional/tests/tests.yaml
index fe40a00..082e66e 100644
--- a/src/tests/functional/tests/tests.yaml
+++ b/src/tests/functional/tests/tests.yaml
@@ -16,3 +16,6 @@ target_deploy_status:
16 grafana:16 grafana:
17 workload-status: active17 workload-status: active
18 workload-status-message: Started grafana-server18 workload-status-message: Started grafana-server
19 nrpe:
20 workload-status: blocked
21 workload-status-message: Nagios server not configured or related
diff --git a/src/tests/unit/test_reactive_ceph_exporter.py b/src/tests/unit/test_reactive_ceph_exporter.py
index bf9e810..0e4b5ad 100644
--- a/src/tests/unit/test_reactive_ceph_exporter.py
+++ b/src/tests/unit/test_reactive_ceph_exporter.py
@@ -49,6 +49,7 @@ class MockCephClient:
49 """Class constructor."""49 """Class constructor."""
50 pass50 pass
5151
52 @property
52 def auth(self):53 def auth(self):
53 """Fake authentication."""54 """Fake authentication."""
54 return True55 return True
@@ -57,6 +58,7 @@ class MockCephClient:
57 """Hard-code monitor hosts."""58 """Hard-code monitor hosts."""
58 return ["1.2.3.4", "5.6.7.8"]59 return ["1.2.3.4", "5.6.7.8"]
5960
61 @property
60 def key(self):62 def key(self):
61 """Hard-code ceph client key."""63 """Hard-code ceph client key."""
62 return "mockcephclientkey"64 return "mockcephclientkey"

Subscribers

People subscribed via source and target branches