♻️ Frontend: Notifications cleanup

[GitHK]

What do these changes do?

This is a followup of #3937

• ♻️ refactors backend parts
• ✨ adds tests for alle 3 endpoints
• ♻️ user notification lists cannot grow to more than 10 per user

Related issue/s

• closes followup cleanup PR to 3937 #3965
• ✨ User Notifications (⚠️ devops) 🚨 #3937
• M1-12 Maintenance and DevOps osparc-issues#675

How to test

Checklist

• Unit tests for the changes exist
• Runs in the swarm

[codecov]

Codecov Report

Merging #4004 (49c7cff) into master (b91f344) will decrease coverage by 7.8%.
The diff coverage is 100.0%.

@@            Coverage Diff            @@
##           master   #4004      +/-   ##
=========================================
- Coverage    85.9%   78.1%    -7.8%     
=========================================
  Files         899     394     -505     
  Lines       38757   20345   -18412     
  Branches      849     137     -712     
=========================================
- Hits        33302   15906   -17396     
+ Misses       5233    4390     -843     
+ Partials      222      49     -173     

Flag	Coverage Δ
integrationtests	61.9% <58.8%> (+<0.1%)	⬆️
unittests	83.8% <100.0%> (+0.4%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...rc/simcore_service_webserver/user_notifications.py	100.0% <100.0%> (ø)
...er/src/simcore_service_webserver/users_handlers.py	96.2% <100.0%> (+29.6%)	⬆️

... and 607 files with indirect coverage changes

[codeclimate]

Code Climate has analyzed commit 49c7cff and detected 0 issues on this pull request.

View more on Code Climate.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.1% Duplication