TODO comments about mslex

pomponchik · Aug 13, 2024 · 90b2a1c · 90b2a1c
1 parent ca9936a
commit 90b2a1c
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/suby/proxy_module.py b/suby/proxy_module.py
@@ -102,6 +102,7 @@ def convert_arguments(arguments: Tuple[Union[str, Path], ...], split: bool) -> L
                     try:
                         for sub_argument in shlex_split(argument):
                             converted_arguments.append(sub_argument)
+                    # TODO: delete this no cover comments when argument checking will be added to mslex.
                     except Exception as e:  # pragma: no cover
                         raise WrongCommandError(f'The expression "{argument}" cannot be parsed.') from e  # pragma: no cover
                 else:

diff --git a/tests/test_proxy_module.py b/tests/test_proxy_module.py
@@ -518,6 +518,7 @@ def test_multiple_args_without_split(command):
     assert result.returncode == 0
 
 
+# TODO: delete this skip when argument checking will be added to mslex.
 @pytest.mark.skipif(platform.system() == 'Windows', reason='There is no errors like this in mslex.')
 @pytest.mark.parametrize(
     ['command', 'exception_message'],