FlexMeasures / flexmeasures / 17267178169

push

github

fix: get rid RuntimeWarning: '<' not supported between instances (#1650)

* fix: get rid RuntimeWarning: '<' not supported between instances of 'Sensor' and 'Sensor', sort order is undefined for incomparable objects.

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

* feat: raise more informative errors when ID attribute is None

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

* fix: fixture was returning sensors with IDs not yet assigned

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

* docs: changelog entry

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

* refactor: move new util function to the intended module

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

* fix: error message

* feat: mention class name only rather than entire path

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

* feat: add test for unhashable entities used as dict keys

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

* refactor: shorten test code

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

* docs: give test a docstring

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

---------

Signed-off-by: F.N. Claessen <felix@seita.nl>
Co-authored-by: Nicolas Höning <nicolas@seita.nl>

1250 of 1851 branches covered (67.53%)

Branch coverage included in aggregate %.

13 of 20 new or added lines in 2 files covered. (65.0%)

9032 of 12284 relevant lines covered (73.53%)

5.5 hits per line