spine-tools / Spine-Database-API

Database interface to Spine generic data model
https://www.tools-for-energy-system-modelling.org/
GNU Lesser General Public License v3.0
6 stars 5 forks source link

Merge 0.8-dev to master #393

Closed soininen closed 2 months ago

soininen commented 2 months ago

Re spine-tools/Spine-Toolbox#2692

Checklist before merging

codecov[bot] commented 2 months ago

Codecov Report

Attention: Patch coverage is 83.55030% with 417 lines in your changes are missing coverage. Please review.

Project coverage is 78.25%. Comparing base (828ff38) to head (cd995e0).

Files Patch % Lines
spinedb_api/db_mapping.py 69.02% 117 Missing and 19 partials :warning:
spinedb_api/mapped_items.py 89.00% 29 Missing and 14 partials :warning:
spinedb_api/parameter_value.py 77.89% 34 Missing and 8 partials :warning:
spinedb_api/query.py 77.57% 19 Missing and 5 partials :warning:
...ccf80441_replace_values_with_reference_to_list_.py 36.11% 22 Missing and 1 partial :warning:
spinedb_api/db_mapping_query_mixin.py 95.41% 16 Missing and 3 partials :warning:
spinedb_api/db_mapping_commit_mixin.py 86.27% 11 Missing and 3 partials :warning:
spinedb_api/export_mapping/export_mapping.py 87.27% 9 Missing and 5 partials :warning:
spinedb_api/import_mapping/import_mapping.py 90.34% 12 Missing and 2 partials :warning:
spinedb_api/spine_db_server.py 56.25% 14 Missing :warning:
... and 16 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #393 +/- ## ========================================== + Coverage 75.33% 78.25% +2.92% ========================================== Files 76 75 -1 Lines 10985 9511 -1474 Branches 2391 2014 -377 ========================================== - Hits 8275 7443 -832 + Misses 2272 1735 -537 + Partials 438 333 -105 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

soininen commented 2 months ago

Toolbox unit tests will run in PR spine-tools/Spine-Toolbox#2731, we do not need modify the GitHub action temporarily for that in this branch.