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

Simplify/clarify test suite by using pytest fixtures #582

Open
vtnate opened this issue Sep 14, 2023 · 0 comments
Open

Simplify/clarify test suite by using pytest fixtures #582

vtnate opened this issue Sep 14, 2023 · 0 comments
Assignees
@vtnate
Labels
tests Anything contained within automated testing

Comments

@vtnate
Copy link
Contributor

vtnate commented Sep 14, 2023 •
edited
Loading

Expected Behavior

Tests should not duplicate effort, but also not silently require other tests to have already run.
We could use PyTest fixtures to call other tests and make this more error-proof for developers.

Actual Behavior

Some tests require others to be run, without notifying developers of that requirement. Other tests duplicate test code to not silently require another test.

Steps to Reproduce

Review tests in test_uo_des.py and test_gmt_lib.py
@vtnate vtnate added the tests Anything contained within automated testing label Sep 14, 2023
@vtnate vtnate self-assigned this Sep 14, 2023
@vtnate vtnate mentioned this issue Sep 14, 2023
Merged
@vtnate vtnate changed the title Simplify test suite by using pytest fixtures Simplify/clarify test suite by using pytest fixtures Sep 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vtnate vtnate

Labels
tests Anything contained within automated testing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@vtnate