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

Fix scenario alternatives not removed correctly #418

Closed PiispaH closed 3 weeks ago

PiispaH commented 3 weeks ago

When deletions were made in a scenario so that the rank order changed, all the scenario alternatives were not removed. This caused an error when the updated scen alt with the new rank number was attempted to be added back to the db.

Fixes spine-tools/Spine-Toolbox#2830

Checklist before merging

codecov[bot] commented 3 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 78.28%. Comparing base (b0f789c) to head (af484db).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #418 +/- ## ======================================= Coverage 78.27% 78.28% ======================================= Files 72 72 Lines 9515 9517 +2 Branches 2033 2034 +1 ======================================= + Hits 7448 7450 +2 Misses 1731 1731 Partials 336 336 ```

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