remove classes#72
Open
AdamsMP92 wants to merge 5 commits into
Open
Conversation
Compare the exported get_tests result against the registered calculator test modules instead of only checking the total number of tests. This catches missing exports, duplicate names, non-function exports, and accidental non-test entries.
for more information, see https://pre-commit.ci
This was referenced Jul 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Refactor calculator tests from classes to functions
Flatten micromagnetictests calculator test classes into individual pytest
functions with shared setup moved into fixtures. Export the new function-level
tests explicitly so mumax3c and oommfc can continue importing the shared test
suite while making future test selection and removal more granular.