From c7803a281482088970521b96b49492975e99fe3a Mon Sep 17 00:00:00 2001 From: Dan Helfman Date: Sun, 9 Jul 2017 10:27:34 -0700 Subject: [PATCH] Adding a "does not raise" test for displaying errors. --- borgmatic/config/validate.py | 2 +- borgmatic/tests/integration/config/test_validate.py | 7 +++++++ 2 files changed, 8 insertions(+), 1 deletion(-) diff --git a/borgmatic/config/validate.py b/borgmatic/config/validate.py index 82b4f93b9..312517626 100644 --- a/borgmatic/config/validate.py +++ b/borgmatic/config/validate.py @@ -59,7 +59,7 @@ def parse_configuration(config_filename, schema_filename): return parsed_result -def display_validation_error(validation_error): # pragma: no cover +def display_validation_error(validation_error): ''' Given a Validation_error, display its error messages to stderr. ''' diff --git a/borgmatic/tests/integration/config/test_validate.py b/borgmatic/tests/integration/config/test_validate.py index 897ea410a..90e223d13 100644 --- a/borgmatic/tests/integration/config/test_validate.py +++ b/borgmatic/tests/integration/config/test_validate.py @@ -111,3 +111,10 @@ def test_parse_configuration_raises_for_validation_error(): with pytest.raises(module.Validation_error): module.parse_configuration('config.yaml', 'schema.yaml') + + +def test_display_validation_error_does_not_raise(): + flexmock(sys.modules['builtins']).should_receive('print') + error = module.Validation_error('config.yaml', ('oops', 'uh oh')) + + module.display_validation_error(error)