Skip to content

Add script to run tests #98

Closed
chraibi wants to merge 6 commits into
FireDynamics:masterfrom
chraibi:run_tests
Closed

Add script to run tests #98
chraibi wants to merge 6 commits into
FireDynamics:masterfrom
chraibi:run_tests

Conversation

@chraibi
Copy link
Copy Markdown
Contributor

@chraibi chraibi commented Nov 8, 2025

Add script to run tests.

This script does the following:

  • Detect which python manager is running. (I use uv)
  • Check if tar-files need extraction. If so, extract.
  • Run the tests.
  • Updates CI to test for different python versions.

This PR also moves a lonely test to a directory like the others.

I added some pytest ignore warning flags because of these numpy-warnings. I guess they hare harmless, but maybe you want to investigate them in another PR instead of ignoring them.

image

closes #97

- Moved one test to the directory along other tests
- Moved its fds file to the directory with data
- This is just to keep the directory clean.
- Detect used python manager
- Add pytest args
m-fehr added a commit that referenced this pull request May 4, 2026
Detects uv/python3 automatically, extracts .tgz test data only when

needed, runs full test suite with coverage. Based on PR #98 by chraibi,

adapted to run all tests (not just acceptance_tests/).
@m-fehr
Copy link
Copy Markdown
Contributor

m-fehr commented May 4, 2026

Thanks for this PR! We've adopted the core idea of your script (uv detection, smart tar extraction) and included an adapted version directly in tests/run_tests.sh (see commit 3521193). The main difference is that our version runs the full test suite instead of just acceptance_tests/.
The CI changes and test file move are covered by our modernization PR #113 which was just merged.
Closing this in favor of the above. Thanks again for the contribution!

@m-fehr m-fehr closed this May 4, 2026
@chraibi chraibi mentioned this pull request May 4, 2026
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.

Running the tests locally

2 participants