Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Tests should not have more than 2 parameters #983

Open
RalfKoban opened this issue Aug 12, 2024 · 1 comment · Fixed by #1024
Open

Tests should not have more than 2 parameters #983

RalfKoban opened this issue Aug 12, 2024 · 1 comment · Fixed by #1024
Labels
Area: analyzer Area: maintainability feature open question

Comments

@RalfKoban
Copy link
Owner

To simplify tests, they should have at most 2 parameters. More than those 2 makes the tests difficult to read.
@RalfKoban RalfKoban changed the title Tests should not more than 2 parameters Tests should not have more than 2 parameters Aug 12, 2024
@RalfKoban
Copy link
Owner Author

Should the limit better be 3 parameters instead of 2 ?

@RalfKoban RalfKoban mentioned this issue Sep 7, 2024
Merged
@github-project-automation github-project-automation bot moved this from In progress to Done in MiKo Analyzers Sep 7, 2024
@RalfKoban RalfKoban moved this from Done to Review in progress in MiKo Analyzers Sep 7, 2024
@RalfKoban RalfKoban reopened this Sep 7, 2024
@github-project-automation github-project-automation bot moved this from Review in progress to In progress in MiKo Analyzers Sep 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Area: analyzer Area: maintainability feature open question
Projects
MiKo Analyzers
Status: On hold
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Added new rule MiKo_3122 that reports tests having more than 2 parameters RalfKoban/MiKo-Analyzers
1 participant
@RalfKoban