refactor: consolidate geometry assertions and improve test robustness… #136
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.
… in TestDPM
Refactor geometry assertions into a private helper method _assert_geometry.
Fix off-by-one error in test_get_apparent_wind loop.
Update floating-point assertions to use pytest.approx for clearer intent.
Standardize string formatting to f-strings across the test suite.
Related Issue / Discussion:
Please delete the option which is not relevant.
Fixes Issue #XX
Relates to discussion [link]
Changes:
Please list the central functionalities that have been changed:
Further Details:
Summary:
Please mention the relevant motivation and context of your changes and shortly describe the behaviour before and after your modifications.
Dependencies:
Please list new dependencies that are required for this modification.
PR Checklist:
In the context of this PR, I:
Please consider that PRs which do not meet the requirements specified in the checklist will not be evaluated. Also, PRs with no activities will be closed after a reasonable amount of time.