From e228ad305e7e890388a5b816460addfc9197707f Mon Sep 17 00:00:00 2001 From: isaak Date: Wed, 21 Feb 2024 21:13:59 -0700 Subject: [PATCH] Fix flake8 errors. --- test/test_abbreviations.py | 1 + test/test_canary.py | 1 - test/test_collect_method_kwargs.py | 2 +- test/test_execute.py | 2 -- test/test_generate_method_kwargs.py | 5 ++--- test/test_invalidTargets.py | 1 + 6 files changed, 5 insertions(+), 7 deletions(-) diff --git a/test/test_abbreviations.py b/test/test_abbreviations.py index a54a33a..7e8d3f3 100644 --- a/test/test_abbreviations.py +++ b/test/test_abbreviations.py @@ -1,5 +1,6 @@ from ..CliFunction import DefaultArgumentParser + class TestAbbreviations: parser = DefaultArgumentParser() diff --git a/test/test_canary.py b/test/test_canary.py index aea9186..a27b86d 100644 --- a/test/test_canary.py +++ b/test/test_canary.py @@ -6,7 +6,6 @@ class TestCanary: @pytest.mark.skip(reason="This test is a canary for verifying CI works as intended.") def test_python_version(self): - version = sys.version if sys.version.startswith("3.8"): assert True is False diff --git a/test/test_collect_method_kwargs.py b/test/test_collect_method_kwargs.py index 65888e8..eb1bfef 100644 --- a/test/test_collect_method_kwargs.py +++ b/test/test_collect_method_kwargs.py @@ -1,4 +1,4 @@ -from ..CliFunction import DefaultArgumentParser, Targets +from ..CliFunction import Targets class TestCollectMethodKwargs: diff --git a/test/test_execute.py b/test/test_execute.py index 7f46954..53f537f 100644 --- a/test/test_execute.py +++ b/test/test_execute.py @@ -1,5 +1,3 @@ -import pytest - from ..CliFunction import Targets diff --git a/test/test_generate_method_kwargs.py b/test/test_generate_method_kwargs.py index 6008995..cfa1d1e 100644 --- a/test/test_generate_method_kwargs.py +++ b/test/test_generate_method_kwargs.py @@ -1,5 +1,3 @@ -import pytest - from ..CliFunction import DefaultArgumentParser @@ -15,7 +13,7 @@ def some_args(*, arg: str = "hi"): return arg @staticmethod - def bool_args(*, arg: bool=False): + def bool_args(*, arg: bool = False): return arg @staticmethod @@ -49,6 +47,7 @@ def test_failed_type_coercion(self): assert self.t.generate_method_kwargs(args=['d.py', 'types2', '--retries=5.4'], function=self.types2) is None assert self.t.generate_method_kwargs(args=['d.py', 'types2', '--retries=True'], function=self.types2) is None assert self.t.generate_method_kwargs(args=['d.py', 'types2', '--retries=5'], function=self.types2) == {'retries': 5} + def test_shorthands(self): assert self.t.generate_method_kwargs(args=['CliFunction', 'bool_args', '-a'], function=self.bool_args) == {"arg": True} assert self.t.generate_method_kwargs(args=['CliFunction', 'some_args', '-a=hello'], function=self.some_args) == {"arg": 'hello'} diff --git a/test/test_invalidTargets.py b/test/test_invalidTargets.py index 3016f87..c8d3d9d 100644 --- a/test/test_invalidTargets.py +++ b/test/test_invalidTargets.py @@ -21,6 +21,7 @@ def noKwargs(myarg, myarg2): """But it does have docstrings""" pass + def camelCase(): """camel Case""" pass