Skip to content

Add anatomical tests#43

Merged
kaitj merged 9 commits into
mainfrom
tests/anatomical
Feb 6, 2026
Merged

Add anatomical tests#43
kaitj merged 9 commits into
mainfrom
tests/anatomical

Conversation

@kaitj
Copy link
Copy Markdown
Contributor

@kaitj kaitj commented Feb 6, 2026

Adds the following:

  • Unit tests
  • Integration tests (only that files exist)
  • Full Pipeline (only that outputs are as expected)

Note that all of the integration tests have been marked with slow (from testing locally). We can probably do more with these than just checking that the files exist, but would be harder to test the contents and making sure they are correct.

cc: @jpillai00

@kaitj kaitj requested a review from nx10 February 6, 2026 19:59
@kaitj kaitj self-assigned this Feb 6, 2026
@kaitj kaitj added the testing Testing of existing code label Feb 6, 2026
Comment thread src/rbc/core/utils.py Outdated
@kaitj kaitj force-pushed the tests/anatomical branch 3 times, most recently from 868236a to 9cbe66d Compare February 6, 2026 21:23
@kaitj kaitj requested a review from nx10 February 6, 2026 21:29
kaitj added 8 commits February 6, 2026 16:33
Also add __init__.py so test module names don't have to be unique
* Remove upload of coverage

* Split up tests

- Unit tests on all OSes
- Integration (and e2e) only on ubuntu

* Remove last step of codecov

* Fix marker call for slow tests

* Move tests back under one workflow
@kaitj
Copy link
Copy Markdown
Contributor Author

kaitj commented Feb 6, 2026

@nx10 said "you can merge" 🫨

@kaitj kaitj merged commit 82c2008 into main Feb 6, 2026
6 checks passed
@kaitj kaitj deleted the tests/anatomical branch February 6, 2026 21:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testing Testing of existing code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Implement Tissue Segmentation (FSL-FAST) Merge Initial Anatomical Preprocessing PR

2 participants