Addition of test for missing aggregation function when require_all_aggregators=True

[talagayev]

Fixes #4650

Changes made in this Pull Request:

• Addition of test_missing_aggregator in test_results.py to cover ValueError case when
  the aggregation is missing and require_all_aggregators=True in ResultsGroup.merge()

PR Checklist

• Tests?
• Docs?
• CHANGELOG updated?
• Issue raised/referenced?

Developers certificate of origin

• I certify that this contribution is covered by the LGPLv2.1+ license as defined in our LICENSE and adheres to the Developer Certificate of Origin.

---

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

[pep8speaks]

Hello @talagayev! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! 🍻

Comment last updated at 2024-11-26 07:54:52 UTC

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.65%. Comparing base (c48962e) to head (a554d3b).
Report is 1 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #4770      +/-   ##
===========================================
- Coverage    93.66%   93.65%   -0.02%     
===========================================
  Files          177      189      +12     
  Lines        21742    22808    +1066     
  Branches      3055     3055              
===========================================
+ Hits         20365    21360     +995     
- Misses         930     1001      +71     
  Partials       447      447              

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

[yuxuanzhuang]

lgtm! just waiting for the CI.

[talagayev]

Nice, much appreciated :)

[IAlibay]

@yuxuanzhuang we really shouldn't have merged things "in between releases"..