-
Notifications
You must be signed in to change notification settings - Fork 177
Open
Labels
scope:testsScope: Changes EL client test cases in `./tests`Scope: Changes EL client test cases in `./tests`type:choreType: ChoreType: Choretype:refactorType: RefactorType: Refactor
Description
The unused arguments in the tests directory may have important side effects or indicate a bug in the test code. Removing them requires more careful investigation.
@SamWilsn, could you please revert the changes and apply
# noqa: arg001
to all of the instances found under./tests/
. Let's make an issue to follow up and do that work in a subsequent PR. Then I think we can get this merged.Originally posted by @danceratopz in #2166 (comment)
danceratopz
Metadata
Metadata
Assignees
Labels
scope:testsScope: Changes EL client test cases in `./tests`Scope: Changes EL client test cases in `./tests`type:choreType: ChoreType: Choretype:refactorType: RefactorType: Refactor