Skip to content

Commit

Permalink
Fix test for analysis keyword-only arguments
Browse files Browse the repository at this point in the history
  • Loading branch information
crnh committed Feb 7, 2025
1 parent 5c7637d commit 1c7c161
Showing 1 changed file with 1 addition and 4 deletions.
5 changes: 1 addition & 4 deletions tests/analyses/new/test_base.py
Original file line number Diff line number Diff line change
Expand Up @@ -81,9 +81,6 @@ class MockAnalysisSettings:
class MockAnalysis(BaseAnalysisWrapper[MockAnalysisData, MockAnalysisSettings]):
TYPE = "MockAnalysis"

_needs_config_file = False
_needs_text_output_file = False

def __init__(
self,
*,
Expand Down Expand Up @@ -140,7 +137,7 @@ def test_analyses_correct_analysis_name(self, cls):

@pytest.mark.parametrize("cls", analysis_wrapper_classes)
def test_init_all_keyword_only_parameters(self, cls):
all(p.kind.name == "KEYWORD_ONLY" for _, p in inspect.signature(cls).parameters.items())
assert all(p.kind.name == "KEYWORD_ONLY" for _, p in inspect.signature(cls).parameters.items())

@pytest.mark.parametrize("cls", analysis_wrapper_classes)
def test_init_contains_all_settings(self, cls):
Expand Down

0 comments on commit 1c7c161

Please sign in to comment.