Closed vishwa2710 closed 3 weeks ago
The changes introduce new parameterized test methods in the TestState
class to enhance testing for the State.from_dict
method in the bindings/python/test/trajectory/test_state.py
file. Additionally, modifications to the from_dict
method in bindings/python/tools/python/ostk/astrodynamics/pytrajectory/pystate.py
include the introduction of a new constant for standardizing key formats and improved error handling for frame identification, ensuring backward compatibility with older key names.
File Path | Change Summary |
---|---|
bindings/python/test/trajectory/test_state.py |
Added methods: test_from_dict_cannonical_success and test_from_dict_cannonical_failure to TestState class. |
bindings/python/tools/python/ostk/astrodynamics/pytrajectory/pystate.py |
Added constant CANONICAL_FORMAT and updated from_dict method to enhance key handling, error management, and compatibility. |
In the land of code where rabbits play,
New tests hop in to brighten the day.
With keys so neat and errors caught,
OurState
is ready, robustly wrought!
So let’s leap high and twirl about,
For every change, we sing and shout! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 90.92%. Comparing base (
88e4315
) to head (50d3058
). Report is 1 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Summary by CodeRabbit
New Features
State
objects from dictionaries, ensuring both successful and failure scenarios are validated.from_dict
method.Bug Fixes
from_dict
method, ensuring appropriate exceptions are raised for invalid data.