FlexMeasures / flexmeasures / 15319615606

78%

push

github

feat: implement role modification permission checks for users (#1465)

* feat: implement role modification permission checks for users

Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>

* Update flexmeasures/data/models/validation_utils.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: JDev <45713692+joshuaunity@users.noreply.github.com>

* Update flexmeasures/data/models/validation_utils.py

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: JDev <45713692+joshuaunity@users.noreply.github.com>

* chore: correct functoin name

Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>

* refactor: relocate fuinction and refactor function logic to be more accurate

Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>

* refactor: refactored error handling to raise error rather than return http response

Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>

* refactor: remove try and except

Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>

* refactor: more concise code

Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>

* chore: adding test - work in progress

Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>

* fix: workaround for Flask-Marshmallow not getting the right db session in tests

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

* chore: expand test case

Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>

* chore: modify test name

Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>

* chore: removed wrong test case

Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>

* refactor: relocate test case and added new utility test case

Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>

* chore: removed auth test prefix

Signed-off-by: joshuaunity <oghenerobojosh01@gmail.com>

* docs: fix accidental typo?

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

* fix: the other test in the fresh_db test module was supposed to use the fresh... (continued)

4007 of 6938 branches covered (57.75%)

Branch coverage included in aggregate %.

27 of 34 new or added lines in 3 files covered. (79.41%)

8649 of 11932 relevant lines covered (72.49%)

7.13 hits per line