Appease linter

ecmwf-ifs · Jul 13, 2023 · 16f022a · 16f022a
1 parent 1143498
commit 16f022a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 9 deletions.
diff --git a/tests/test_lint/test_linter.py b/tests/test_lint/test_linter.py
@@ -48,7 +48,7 @@ class TestRule(GenericRule):
         config = {'key': 'default_value'}
 
         @classmethod
-        def check(cls, ast, rule_report, config):
+        def check(cls, ast, rule_report, config, **kwargs):
             assert len(config) == 1
             assert 'key' in config
             assert config['key'] == 'default_value'
@@ -58,7 +58,7 @@ class TestRule2(GenericRule):
         config = {'key': 'default_value'}
 
         @classmethod
-        def check(cls, ast, rule_report, config):
+        def check(cls, ast, rule_report, config, **kwargs):
             assert len(config) == 2
             assert 'key' in config
             assert config['key'] == 'non_default_value'
@@ -215,15 +215,15 @@ class AssignmentComplainRule(GenericRule):
         docs = {'id': '13.37'}
 
         @classmethod
-        def check_subroutine(cls, subroutine, rule_report, config):  # pylint: disable=unused-argument
+        def check_subroutine(cls, subroutine, rule_report, config, **kwargs):  # pylint: disable=unused-argument
             for node in FindNodes(Assignment).visit(subroutine.ir):
                 rule_report.add(cls.__name__ + '_' + str(node.source.lines[0]), node)
 
     class VariableComplainRule(GenericRule):
         docs = {'id': '23.42'}
 
         @classmethod
-        def check_subroutine(cls, subroutine, rule_report, config):  # pylint: disable=unused-argument
+        def check_subroutine(cls, subroutine, rule_report, config, **kwargs):  # pylint: disable=unused-argument
             for node, variables in FindVariables(with_ir_node=True).visit(subroutine.body):
                 for var in variables:
                     rule_report.add(cls.__name__ + '_' + str(var), node)
@@ -278,15 +278,15 @@ class AssignmentComplainRule(GenericRule):
         docs = {'id': '13.37'}
 
         @classmethod
-        def check_subroutine(cls, subroutine, rule_report, config):  # pylint: disable=unused-argument
+        def check_subroutine(cls, subroutine, rule_report, config, **kwargs):  # pylint: disable=unused-argument
             for node in FindNodes(Assignment).visit(subroutine.ir):
                 rule_report.add(cls.__name__ + '_' + str(node.source.lines[0]), node)
 
     class VariableComplainRule(GenericRule):
         docs = {'id': '23.42'}
 
         @classmethod
-        def check_subroutine(cls, subroutine, rule_report, config):  # pylint: disable=unused-argument
+        def check_subroutine(cls, subroutine, rule_report, config, **kwargs):  # pylint: disable=unused-argument
             for node, variables in FindVariables(with_ir_node=True).visit(subroutine.body):
                 for var in variables:
                     rule_report.add(cls.__name__ + '_' + str(var), node)
@@ -511,7 +511,7 @@ class TestRule(GenericRule):
         fixable = True
 
         @classmethod
-        def check_subroutine(cls, subroutine, rule_report, config):
+        def check_subroutine(cls, subroutine, rule_report, config, **kwargs):
             if not subroutine.name.isupper():
                 rule_report.add(f'Subroutine name "{subroutine.name}" is not upper case', subroutine)
 

diff --git a/tests/test_lint/test_reporter.py b/tests/test_lint/test_reporter.py
@@ -152,7 +152,7 @@ class RandomFailingRule(GenericRule):
         config = {'dummy_key': 'dummy value'}
 
         @classmethod
-        def check_subroutine(cls, subroutine, rule_report, config):
+        def check_subroutine(cls, subroutine, rule_report, config, **kwargs):
             if fail_on and fail_on in subroutine.name:
                 rule_report.add(cls.__name__, subroutine)
 
@@ -191,7 +191,7 @@ class RandomFailingRule(GenericRule):
         config = {'dummy_key': 'dummy value'}
 
         @classmethod
-        def check_subroutine(cls, subroutine, rule_report, config):
+        def check_subroutine(cls, subroutine, rule_report, config, **kwargs):
             if fail_on and fail_on in subroutine.name:
                 rule_report.add(cls.__name__, subroutine)