FlexMeasures / flexmeasures / 24364189283

81%

push

github

Fix asset status page crash on old `sensors_to_show` format (#2103)

* Initial plan

* Fix KeyError 'sensors' crash on asset status page for old sensors_to_show format

Agent-Logs-Url: https://github.com/FlexMeasures/flexmeasures/sessions/09f0ee2e-71f7-4389-a447-23f20fee1364

Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>

* Move SensorsToShowSchema import to module level

Agent-Logs-Url: https://github.com/FlexMeasures/flexmeasures/sessions/09f0ee2e-71f7-4389-a447-23f20fee1364

Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>

* Fix test: flush session before reading sensor IDs, reset sensors_to_show after test

Agent-Logs-Url: https://github.com/FlexMeasures/flexmeasures/sessions/5ce56f2a-9e55-408b-9659-f12da4d6800a

Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>

* Revert "Fix test: flush session before reading sensor IDs, reset sensors_to_show after test"

This reverts commit 6980c7a6c.

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

* fix: flush sensor IDs

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

* feat: agent lesson learned

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

* fix: reset module-scoped fixture state

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

* Fix test: flush session before reading sensor IDs, reset sensors_to_show after test

Agent-Logs-Url: https://github.com/FlexMeasures/flexmeasures/sessions/5ce56f2a-9e55-408b-9659-f12da4d6800a

Co-authored-by: Flix6x <30658763+Flix6x@users.noreply.github.com>

* Revert "Fix test: flush session before reading sensor IDs, reset sensors_to_show after test"

This reverts commit 2b53bf495.

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

* delete: redundant flushes

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

* refactor: cleaner looking code

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

* docs: remove space

Signed-off-by: F.N. Claessen <claessen@se... (continued)

2 of 2 new or added lines in 1 file covered. (100.0%)

12021 of 15149 relevant lines covered (79.35%)

0.79 hits per line