🐛 fix services:access endpoint for product group

[matusdrobuliak66]

What do these changes do?

• Issue reported: #1540 A share permission warning is incorrectly popping up. The reason is that listing user groups filters out Products because Product groups are resources with Read access set to False. In a previous PR, I already prepared the groundwork for this fix by renaming the previous function list_all_user_groups to list_user_groups_with_read_access. The new function, list_all_user_groups, now correctly returns all groups of the user.

Related issue/s

• closes 🐛 Wrong sharee permission - if shared with product group osparc-issues#1540

How to test

pytest tests/unit/with_dbs/02/test_projects_nodes_handlers__services_access.py

Dev-ops checklist

• No ENV changes or I properly updated ENV (read the instruction)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.1%. Comparing base (cafbf96) to head (ef86d1a).
Report is 450 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##           master   #6213      +/-   ##
=========================================
- Coverage    84.5%   82.1%    -2.5%     
=========================================
  Files          10     577     +567     
  Lines         214   29430   +29216     
  Branches       25     234     +209     
=========================================
+ Hits          181   24175   +23994     
- Misses         23    5195    +5172     
- Partials       10      60      +50     

Flag	Coverage Δ
integrationtests	64.9% <33.3%> (?)
unittests	87.6% <100.0%> (+3.0%)	⬆️

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

Files	Coverage Δ
...core_service_webserver/projects/_nodes_handlers.py	91.1% <100.0%> (ø)

... and 586 files with indirect coverage changes

[elisabettai]

Yeeeeeeeeeeeeah. 🎉 🎉 🎉
Also see the same case on fogbugz (we should tag it properly when this one goes to staging): https://z43.manuscript.com/f/cases/202180/Wrong-message-when-sharing-a-project-on-sim4life-io-science

[pcrespov]

Can you pls create a test that reproduces #1540?
The test should fail with the previous version and pass with your fix

[sanderegg]

it would be good, if that is an expected behavior, that there is a test for it. Otherwise anyone might inadvertently break it.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[odeimaiz]

Thanks!