FlexMeasures / flexmeasures / 10885749754

74%

push

github

Fix DataError Due to String Length Exceeding Column Constraint in AssetAuditLog (#1162)

* Add and implement truncate_event for handling string truncation in audit logs and asset attribute changes

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* add comment

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* Add doc string to truncate_event function and added comments

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* Create audit log entry for each Attribute/Field update

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* truncate each audit log entry if it exceeds 255

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* Update test_alter_an_asset accounting for new audit log changes: each update get's it's own entry

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* update event string with ';' for better readability

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* Update tests after adding ';' to event string

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

* Add BugFix entry in changelog

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>

---------

Signed-off-by: Mohamed Belhsan Hmida <mohamedbelhsanhmida@gmail.com>
Signed-off-by: Mohamed Belhsan Hmida <149331360+BelhsanHmida@users.noreply.github.com>

6224 of 10086 branches covered (61.71%)

Branch coverage included in aggregate %.

3 of 5 new or added lines in 2 files covered. (60.0%)

7870 of 10862 relevant lines covered (72.45%)

5.74 hits per line