Skip to content

Conversation

@tanii1125
Copy link
Contributor

@tanii1125 tanii1125 commented Dec 7, 2025

Summary of Changes-

This PR Modernize test assertions in testsuite\MDAnalysisTests\analysis\test_msd.py

Changes made-

  1. Replaced deprecated assert_almost_equal with assert_allclose using tolerances equivalent to the previous decimal keyword.
  2. Replaced scalar floating-point comparisons with pytest.approx() where appropriate.
  3. Cleaned up imports (removed unused deprecated functions).
  4. Ensured all updated assertions preserve original test behavior.

Thanks, and please let me know if any further adjustments are needed.


📚 Documentation preview 📚: https://mdanalysis--5169.org.readthedocs.build/en/5169/

@codecov
Copy link

codecov bot commented Dec 7, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.15%. Comparing base (bbcef1b) to head (d17591b).

❗ There is a different number of reports uploaded between BASE (bbcef1b) and HEAD (d17591b). Click for more details.

HEAD has 3 uploads less than BASE
Flag BASE (bbcef1b) HEAD (d17591b)
5 2
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #5169      +/-   ##
===========================================
- Coverage    92.72%   86.15%   -6.57%     
===========================================
  Files          180      180              
  Lines        22472    22472              
  Branches      3188     3188              
===========================================
- Hits         20837    19361    -1476     
- Misses        1177     2658    +1481     
+ Partials       458      453       -5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@tanii1125 tanii1125 closed this Dec 8, 2025
@tanii1125 tanii1125 deleted the modernize-test-msd-3743 branch December 8, 2025 16:06
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