Skip to content

Conversation

@mandli
Copy link
Member

@mandli mandli commented May 22, 2025

This PR changes the way we are doing testing on a larger scale and adds some new capabilities:

  • Uses pytest more directly than the previous tests based on UnitTest
  • Significant code simplification of both the test scaffoloding and individual tests
  • Gives full control of the output (temporary) directory to the test via the commandline pytest --basetemp=
  • Can specify which setrun is used by path and removes problems with collision of names
  • Uses some pytest fixtures to provide other functionality meaning that they can also be expanded

There are a number of other advantages to this approach including using other plugins for PyTest in the future.

@mandli mandli marked this pull request as draft May 22, 2025 04:05
@mandli mandli requested a review from rjleveque July 28, 2025 16:37
@mandli mandli marked this pull request as ready for review July 28, 2025 16:37
@mandli
Copy link
Member Author

mandli commented Jul 28, 2025

We can merge this or not, but it provides a proof of concept that we could do this. I think this may help a lot with more complex testing, such as with the adjoint tests, and help with the use of alternative setrun.py files from arbitrary locations.

@mandli
Copy link
Member Author

mandli commented Sep 15, 2025

Moved the generic code for the test runner into clawutil now, see clawpack/clawutil#96 for details.

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