Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Tests should use subTest #25

Closed
mdickinson opened this issue Jun 2, 2021 · 0 comments · Fixed by #27
Closed

Tests should use subTest #25

mdickinson opened this issue Jun 2, 2021 · 0 comments · Fixed by #27
Labels
enhancement New feature or request

Comments

@mdickinson
Copy link
Member

We have various tests that loop over lots of inputs; those should be using the subTests machinery.
@mdickinson mdickinson added bug Something isn't working enhancement New feature or request and removed bug Something isn't working labels Jun 2, 2021
@mdickinson mdickinson mentioned this issue Aug 12, 2022
Merged
mdickinson added a commit that referenced this issue Aug 12, 2022
@mdickinson
Blacken tests; use subTest (#27)
482a81f 
Two stylistic updates to the existing tests:

* Use subTest for tests in for loops
* Apply black to the test module

Closes #25.
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Update tests (style) enthought/ibm2ieee
1 participant
@mdickinson