urbanopt / modelica-builder / 19981164782

93%

develop: 93%

Build Type

Pull #129

github

Committed by web-flow

Commit Message

Fix subpackage cache synchronization in PackageParser (#135)

* Initial plan

* Fix subpackage cache synchronization in add_model

Check if subpackage already exists in _subpackages cache before creating a new instance. This prevents duplicate PackageParser instances when the same subpackage is accessed through both add_model() and attribute access. Added test to verify the fix.

Co-authored-by: nllong <1907354+nllong@users.noreply.github.com>

* Address code review feedback

- Move shutil import to top of test file
- Prevent duplicate entries in order_data when add_model is called multiple times for same model
- Add test assertion to verify order_data doesn't contain duplicates

Co-authored-by: nllong <1907354+nllong@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: nllong <1907354+nllong@users.noreply.github.com>

Pull Request Pull Request #129: Allow package parser to include subpackages

Run Details

145 of 173 new or added lines in 2 files covered. (83.82%)

2 existing lines in 1 file now uncovered.

2028 of 2188 relevant lines covered (92.69%)

1.85 hits per line