Skip to content

Commit

Permalink
Add test dependencies as an optional dependency
Browse files Browse the repository at this point in the history
The changes in TopoToolbox#125 use opensimplex directly in the tests, and I
wanted to be clear that you need opensimplex to run the tests. I left
the "opensimplex" optional dependency in there to avoid breaking
workflows and because it does provide functionality that we use
outside of the tests.

I also added pytest to the "test" dependency. I believe this is the only
other package we use directly. All of the other test dependencies are
transitive dependencies. Cleanly installing everything using `pip
install .[test]` in a new virtual environment works, and the tests are
able to run successfully.
  • Loading branch information
wkearn committed Jan 16, 2025
1 parent 84140e0 commit 4c284ce
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,10 @@ classifiers = []

[project.optional-dependencies]
opensimplex = ["opensimplex"]
test = [
"pytest",
"opensimplex"
]

[project.urls]
Homepage = "https://topotoolbox.github.io"
Expand Down

0 comments on commit 4c284ce

Please sign in to comment.