Skip to content

STY: Remove unnecessary parentheses in test parameterization args - #132

Merged
arokem merged 1 commit into
tee-ar-ex:masterfrom
jhlegarreta:sty/remove-redundant-parentheses
Aug 21, 2026
Merged

STY: Remove unnecessary parentheses in test parameterization args#132
arokem merged 1 commit into
tee-ar-ex:masterfrom
jhlegarreta:sty/remove-redundant-parentheses

Conversation

@jhlegarreta

Copy link
Copy Markdown
Contributor

Remove unnecessary parentheses around single-argument cases in test parameterizations.

Fixes:

Remove redundant parentheses

raised locally by IDE.

Reduces visual clutter, makes the code lighter and improves readability.

@arokem

arokem commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

This looks good to me. Could you please rebase?

Remove unnecessary parentheses around single-argument cases in test
parameterizations.

Fixes:
```
Remove redundant parentheses
```

raised locally by IDE.

Reduces visual clutter, makes the code lighter and improves readability.
@jhlegarreta
jhlegarreta force-pushed the sty/remove-redundant-parentheses branch from c1ed9a1 to 00458d2 Compare August 21, 2026 11:57
@jhlegarreta

Copy link
Copy Markdown
Contributor Author

Rebased.

@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.53%. Comparing base (2fbe5c7) to head (00458d2).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #132   +/-   ##
=======================================
  Coverage   86.53%   86.53%           
=======================================
  Files          13       13           
  Lines        2896     2896           
=======================================
  Hits         2506     2506           
  Misses        390      390           
Flag Coverage Δ
unittests 86.53% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@arokem
arokem merged commit 9e33b8a into tee-ar-ex:master Aug 21, 2026
19 checks passed
@jhlegarreta
jhlegarreta deleted the sty/remove-redundant-parentheses branch August 21, 2026 16:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants