summaryrefslogtreecommitdiffstats
path: root/roles/openshift_health_checker/test
diff options
context:
space:
mode:
authorScott Dodson <sdodson@redhat.com>2017-09-12 16:39:40 -0400
committerGitHub <noreply@github.com>2017-09-12 16:39:40 -0400
commit4ced3e42ea8d077210b22a72f6132f004ed6dfe3 (patch)
treecb84ccd00e788a02296126962125c949c0e5f688 /roles/openshift_health_checker/test
parentf8393b7aef78c92437fe78092821a8d0a11c22cc (diff)
parent5932c90c13dc415ab1448711961d398c51f97b8b (diff)
downloadopenshift-4ced3e42ea8d077210b22a72f6132f004ed6dfe3.tar.gz
openshift-4ced3e42ea8d077210b22a72f6132f004ed6dfe3.tar.bz2
openshift-4ced3e42ea8d077210b22a72f6132f004ed6dfe3.tar.xz
openshift-4ced3e42ea8d077210b22a72f6132f004ed6dfe3.zip
Merge pull request #5352 from rhcarvalho/issue5311-harden-failure-summary-plugin
Skip failure dedup instead of crashing
Diffstat (limited to 'roles/openshift_health_checker/test')
-rw-r--r--roles/openshift_health_checker/test/zz_failure_summary_test.py15
1 files changed, 15 insertions, 0 deletions
diff --git a/roles/openshift_health_checker/test/zz_failure_summary_test.py b/roles/openshift_health_checker/test/zz_failure_summary_test.py
index 0fc258133..69f27653c 100644
--- a/roles/openshift_health_checker/test/zz_failure_summary_test.py
+++ b/roles/openshift_health_checker/test/zz_failure_summary_test.py
@@ -65,6 +65,21 @@ import pytest
},
],
),
+ # if a failure contain an unhashable value, it will not be deduplicated
+ (
+ [
+ {
+ 'host': 'master1',
+ 'msg': {'unhashable': 'value'},
+ },
+ ],
+ [
+ {
+ 'host': 'master1',
+ 'msg': {'unhashable': 'value'},
+ },
+ ],
+ ),
])
def test_deduplicate_failures(failures, deduplicated):
assert deduplicate_failures(failures) == deduplicated