urbanopt / modelica-builder / 19997804988

93%

Build Type

push

github

Committed by web-flow

Commit Message

Allow package parser to include subpackages (#129)

* allow package parser to include subpackages

* remove package name of junk

* Update modelica_builder/package_parser.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update modelica_builder/package_parser.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update modelica_builder/package_parser.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* create_subpackage is now defaulted to false to deep api consistent

* Update modelica_builder/package_parser.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Update modelica_builder/package_parser.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

* Add test coverage for immediate subpackage access after add_model (#131)

* Initial plan

* Add test case for immediate subpackage access after add_model

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>

* ensure path exists

* Prevent inconsistent state in add_model when path is None (#130)

* Initial plan

* Add validation to prevent inconsistent state in add_model when path is None

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

* precommit

* precommit

* fix test

---------

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

* 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 a... (continued)

Coverage Stats

146 of 178 new or added lines in 2 files covered. (82.02%)

35 existing lines in 2 files now uncovered.

2029 of 2193 relevant lines covered (92.52%)

1.85 hits per line