FlexMeasures / flexmeasures / 21213226020 / 4

78%

main: 78%

push

github

Fix/return validation errors instead of db errors (#1923)

* feat: test whether all Marshmallow Field subclasses in FlexMeasures that override
    _deserialize() call super()._deserialize()

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

* fix: ensure that all Marshmallow Field subclasses in FlexMeasures that override
    _deserialize() call super()._deserialize()

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

* feat: cover only fields that actually query the db as part of their _deserialize method

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

* refactor: streamline obj/data argument names with those of the superclass methods

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

* refactor: streamline value argument name with that of the superclass methods

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

* feat: add more clarity using type annotations

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

* refactor: superclass _deserialize also uses value as its first argument

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

* refactor: no need to rename classes upon import

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

* style: mypy

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

* fix: set required=False rather than load_default=None

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

* refactor: update argument names of _deserialize and _serialize in other fields, too

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

* fix: arguments are now optional

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

* delete: empty module

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

* refactor: the exact same JSON field was defined twice

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

* docs: changelog entries

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

* docs: fix typo

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

* docs: missing punctuation

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

* docs: fix typo

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

* fix: prevent pre-c... (continued)

1591 of 2094 branches covered (75.98%)

Branch coverage included in aggregate %.

22192 of 28274 relevant lines covered (78.49%)

0.78 hits per line