FlexMeasures / flexmeasures / 6562014495

78%

push

github

Change the Asset constraint `(account_id, name)` to `(parent_asset_id, name)`. (#874)

* remove generic_asset_name_account_id_key constraint

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* fix constrain definition

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add constraint name

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* remove test

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* remove test

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add generic_asset_name_parent_asset_id_key constraint

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add ondelete="cascade" option to parent_asset_id

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* update CLI message to delete asset

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* make it work

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* validate that the name is unique among the asset "brothers"

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* rename migration revision

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* improve description

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* refactor: minimize line changes

Signed-off-by: F.N. Claessen <felix@seita.nl>

* add more test cases

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* add account id in the filter

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

* edit changelog entry

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>

---------

Signed-off-by: Victor Garcia Reolid <victor@seita.nl>
Signed-off-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: F.N. Claessen <felix@seita.nl>

3589 of 6078 branches covered (0.0%)

Branch coverage included in aggregate %.

9 of 9 new or added lines in 3 files covered. (100.0%)

6447 of 9116 relevant lines covered (70.72%)

2.82 hits per line