Skip to content

refactor(fixtures): add default external model repo location#48

Merged
wpbonelli merged 2 commits into
MODFLOW-ORG:developfrom
wpbonelli:refactor-fixtures
Jan 7, 2023
Merged

refactor(fixtures): add default external model repo location#48
wpbonelli merged 2 commits into
MODFLOW-ORG:developfrom
wpbonelli:refactor-fixtures

Conversation

@wpbonelli
Copy link
Copy Markdown
Member

@wpbonelli wpbonelli commented Jan 6, 2023

  • Search for model repos in a default location relative to the working directory. If the consuming project's tests are run from an autotest folder in the project root, and model repos live next to the consuming project, models will be found by fixtures. This makes model-finding fixtures usable without setting the REPOS_PATH environment variable, which was previously required.
  • Don't skip any models by default when loading via fixtures. Consuming projects can pytest.skip() in the test function or use get_model_paths() or get_namefile_paths() directly if more control is required over which models are loaded.
  • Expand/clarify documentation

@wpbonelli wpbonelli merged commit 9987209 into MODFLOW-ORG:develop Jan 7, 2023
@wpbonelli wpbonelli deleted the refactor-fixtures branch January 7, 2023 12:52
@github-actions github-actions Bot mentioned this pull request Jan 7, 2023
wpbonelli added a commit that referenced this pull request Jan 7, 2023
* ci(release): update to development version 0.1.3
* docs: update/expand docs (#47)
* refactor(fixtures): update defaults for model-finding fixtures (#48)
* fix(fixtures): fix test_model_mf6 fixture node id (#49)
* ci(release): set version to 0.1.3, update changelog

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: w-bonelli <wbonelli@ucar.edu>
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.

1 participant