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.

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

refactor(tests): use modflow-devtools fixtures and utilities #1665

Merged
merged 1 commit into from
Dec 20, 2022

Conversation

wpbonelli
Copy link
Member

@wpbonelli wpbonelli commented Dec 20, 2022

modflow-devtools now has test fixtures and utilities that can be shared by FloPy, Modflow6 and other repos. Configure modflow-devtools as a pytest plugin in autotest/conftest.py, allowing tests to use fixtures, CLI options and configuration hooks defined upstream. Update marker imports in test files. Trim DEVELOPER.md, removing sections on utilities moved to modflow-devtools, and link to the docs there instead. Also add info on using .env files to configure environment variables needed for tests, e.g. GITHUB_TOKEN.

Some code previously in autotest/conftest.py and now in modflow-devtools was contributed by @mwtoews. Since authorship was lost on the move between repos I added notes to docstrings but if I have missed any please let me know.

This PR was originally interwoven with #1621, extracted here for smaller and more focused changesets.

@codecov
Copy link

codecov bot commented Dec 20, 2022

Codecov Report

Merging #1665 (11e533e) into develop (fb72098) will decrease coverage by 1.2%.
The diff coverage is 0.0%.

@@            Coverage Diff            @@
##           develop   #1665     +/-   ##
=========================================
- Coverage     72.7%   71.5%   -1.3%     
=========================================
  Files          252     252             
  Lines        54554   53194   -1360     
=========================================
- Hits         39714   38036   -1678     
- Misses       14840   15158    +318     
Impacted Files Coverage Δ
flopy/mf6/utils/generate_classes.py 16.3% <0.0%> (+0.4%) ⬆️
flopy/mf6/modflow/mfgwtgwt.py 33.3% <0.0%> (-29.8%) ⬇️
flopy/mf6/modflow/mfgwfcsub.py 39.0% <0.0%> (-28.2%) ⬇️
flopy/mf6/modflow/mfgwtlkt.py 45.7% <0.0%> (-24.6%) ⬇️
flopy/mf6/modflow/mfgwtmwt.py 45.7% <0.0%> (-24.6%) ⬇️
flopy/mf6/modflow/mfgwtsft.py 45.7% <0.0%> (-24.6%) ⬇️
flopy/mf6/modflow/mfgwtuzt.py 45.7% <0.0%> (-24.6%) ⬇️
flopy/mf6/modflow/mfgwtcnc.py 46.1% <0.0%> (-23.5%) ⬇️
flopy/mf6/modflow/mfgwtsrc.py 46.1% <0.0%> (-23.5%) ⬇️
flopy/mf6/modflow/mfgwfvsc.py 47.6% <0.0%> (-22.7%) ⬇️
... and 91 more

@wpbonelli wpbonelli marked this pull request as ready for review December 20, 2022 03:55
@wpbonelli wpbonelli force-pushed the use-devtools branch 2 times, most recently from 675d7b4 to 40471d6 Compare December 20, 2022 15:25
@wpbonelli wpbonelli merged commit 96fc3ad into modflowpy:develop Dec 20, 2022
@wpbonelli wpbonelli deleted the use-devtools branch December 20, 2022 16:19
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