Skip to content

Commit

Permalink
Fix a bunch of linting problems in the unit tests.
Browse files Browse the repository at this point in the history
  • Loading branch information
Isaak-Malers committed Feb 23, 2024
1 parent 43c3a0f commit d87e329
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 6 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/code-quality.yml
Original file line number Diff line number Diff line change
Expand Up @@ -32,4 +32,4 @@ jobs:
- name: Run PyLint on test directory
run: |
pylint --disable=line-too-long,invalid-name,missing-module-docstring,import-error,missing-class-docstring,comparison-of-constants,missing-function-docstring ./test/*.py
pylint --disable=line-too-long,invalid-name,missing-module-docstring,import-error,missing-class-docstring,comparison-of-constants,missing-function-docstring,too-few-public-methods ./test/*.py
4 changes: 4 additions & 0 deletions test/test_generate_method_kwargs.py
Original file line number Diff line number Diff line change
Expand Up @@ -17,10 +17,14 @@ def bool_args(*, arg: bool = False):
return arg

@staticmethod
# pylint: disable=too-many-arguments
# pylint: disable=unused-argument
def complex_method(*, arg1: str, arg2: str, arg3: str, arg4: str, arg5: str, arg6: str, arg7: str, arg8: str, arg9: str):
return "wtf"

@staticmethod
# pylint: disable=too-many-arguments
# pylint: disable=unused-argument
def types(*, st: str, bo: bool, inn: int, fl: float):
return "yay types"

Expand Down
5 changes: 0 additions & 5 deletions test/test_invalidTargets.py
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,10 @@

def one():
"""one"""
pass


def two():
"""two"""
pass


def noDocstring():
Expand All @@ -19,17 +17,14 @@ def noDocstring():

def noKwargs(myarg, myarg2):
"""But it does have docstrings"""
pass


def camelCase():
"""camel Case"""
pass


def snake_case():
"""snake_case"""
pass


class TestInvalidTargets:
Expand Down

0 comments on commit d87e329

Please sign in to comment.