Skip to content

Commit

Permalink
Tests: User to be warned about incorrect delimiter
Browse files Browse the repository at this point in the history
Test if the error message is descriptive and informative when incorrect
delimiter is supplied in the configuration file.

Related: https://bugzilla.redhat.com/show_bug.cgi?id=2079739

Signed-off-by: Michal Polovka <[email protected]>
  • Loading branch information
miskopo committed May 27, 2022
1 parent 7e8ef44 commit 5720cc5
Showing 1 changed file with 32 additions and 0 deletions.
32 changes: 32 additions & 0 deletions tests/test_options.py
Original file line number Diff line number Diff line change
Expand Up @@ -104,3 +104,35 @@ def test_incorrect_output_type_cfg_file(mock_parse, mock_run, mock_service):

finally:
os.remove(config_path)


@patch('ipahealthcheck.core.core.run_service_plugins')
@patch('ipahealthcheck.core.core.run_plugins')
@patch('ipahealthcheck.core.core.parse_options')
def test_incorrect_delimiter_cfg_file(mock_parse, mock_run, mock_service):
"""
Test the error message is user-friendly if the incorrect delimiter is
used in cfg file
Related: https://bugzilla.redhat.com/show_bug.cgi?id=2079739
"""
mock_service.return_value = (Results(), [])
mock_run.return_value = Results()
mock_parse.return_value = options
fd, config_path = tempfile.mkstemp()
os.close(fd)
with open(config_path, "w") as fd:
fd.write('[default]\n')
fd.write('output_type;human\n')

try:
run = RunChecks([], config_path)

f = io.StringIO()
with redirect_stdout(f):
run.run_healthcheck()

assert "contains parsing errors" in f.getvalue()

finally:
os.remove(config_path)

0 comments on commit 5720cc5

Please sign in to comment.