diff --git a/CliFunction.py b/CliFunction.py index d85de8b..7539aa3 100644 --- a/CliFunction.py +++ b/CliFunction.py @@ -4,7 +4,7 @@ import re -class FunctionCliException(Exception): +class CliFunctionException(Exception): """ Common exception type for CliFunction. This ensures it is obvious when a problem occurs with the CLI wrapper vs the code being called into. @@ -193,21 +193,21 @@ def add_target(self, to_add): """ for func in self.targets: if func.__name__ == to_add.__name__: - raise FunctionCliException(f"duplicate target names: {func.__name__}") + raise CliFunctionException(f"duplicate target names: {func.__name__}") if to_add.__doc__ is None: - raise FunctionCliException( + raise CliFunctionException( "Bake requires doc-strings for target functions (denoted by a triple quoted comment as the first thing in the function body)") # pylint: disable=unused-variable names, varargs, varkw, defaults, kwonlyargs, kwonlydefaults, annotations = inspect.getfullargspec(to_add) if len(names) != 0 or defaults is not None: - raise FunctionCliException( + raise CliFunctionException( "Bake requires functions with arguments to use exclusively keyword arguments (denoted by a [*] as the first argument to the function)") if varargs is not None: - raise FunctionCliException("Bake does not support varargs") + raise CliFunctionException("Bake does not support varargs") if varkw is not None: - raise FunctionCliException("Bake does not support varargs") + raise CliFunctionException("Bake does not support varargs") self.targets.append(to_add) def function_help(self, func, pad: str = "") -> str: diff --git a/test/test_invalidTargets.py b/test/test_invalidTargets.py index cd9fde4..cb7b59f 100644 --- a/test/test_invalidTargets.py +++ b/test/test_invalidTargets.py @@ -1,6 +1,6 @@ import pytest -from ..CliFunction import Targets, FunctionCliException +from ..CliFunction import Targets, CliFunctionException def one(): @@ -33,10 +33,10 @@ def test_no_docstring(self): t = Targets() with pytest.raises(Exception) as e: t.add_target(noDocstring) - assert e.type == FunctionCliException + assert e.type == CliFunctionException def test_no_kwargs_only(self): t = Targets() with pytest.raises(Exception) as e: t.add_target(noKwargs) - assert e.type == FunctionCliException + assert e.type == CliFunctionException