Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport][ipa-4-9] ipatests: test_ipahealthcheck: print a message if a system is healthy #5891

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
14 changes: 13 additions & 1 deletion ipatests/test_integration/test_ipahealthcheck.py
Expand Up @@ -286,7 +286,7 @@ def test_run_ipahealthcheck_list_source(self):
for source in sources_avail:
assert source in result.stdout_text

def test_human_output(self, restart_service):
def test_human_severity(self, restart_service):
"""
Test that in human output the severity value is correct

Expand All @@ -306,6 +306,18 @@ def test_human_output(self, restart_service):
assert output == \
"ERROR: ipahealthcheck.meta.services.sssd: sssd: not running"

def test_human_output(self):
"""
Test if in case no failures were found, informative string is printed
in human output.

https://pagure.io/freeipa/issue/8892
"""
returncode, output = run_healthcheck(self.master, output_type="human",
failures_only=True)
assert returncode == 0
assert output == "No issues found."

def test_ipa_healthcheck_after_certupdate(self):
"""
Verify that ipa-certupdate hasn't messed up tracking
Expand Down