♻️ folder_utils is now compatible with user

[GitHK]

What do these changes do?

Previously to manage a folder at the DB layer, you would need to provide a single gid.
Now, you can pass in all the gids to which the users has access to.

Related issue/s

• Project Folders in User Workspace (Nov.) osparc-issues#716

How to test

Dev-ops checklist

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

[codecov]

Codecov Report

Attention: Patch coverage is 89.28571% with 9 lines in your changes missing coverage. Please review.

Project coverage is 89.2%. Comparing base (cafbf96) to head (f6902d3).
Report is 458 commits behind head on master.

Files	Patch %	Lines
...ase/src/simcore_postgres_database/utils_folders.py	90.1%	5 Missing and 3 partials ⚠️
...ore_service_webserver/folders/_folders_handlers.py	66.6%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff            @@
##           master   #6148      +/-   ##
=========================================
+ Coverage    84.5%   89.2%    +4.6%     
=========================================
  Files          10    1262    +1252     
  Lines         214   53440   +53226     
  Branches       25     714     +689     
=========================================
+ Hits          181   47671   +47490     
- Misses         23    5609    +5586     
- Partials       10     160     +150     

Flag	Coverage Δ
integrationtests	64.9% <33.3%> (?)
unittests	86.8% <89.2%> (+2.2%)	⬆️

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

Files	Coverage Δ
.../simcore_service_webserver/folders/_folders_api.py	100.0% <ø> (ø)
...simcore_service_webserver/projects/_folders_api.py	96.5% <ø> (ø)
...ore_service_webserver/folders/_folders_handlers.py	91.5% <66.6%> (ø)
...ase/src/simcore_postgres_database/utils_folders.py	90.7% <90.1%> (ø)

... and 1225 files with indirect coverage changes

[matusdrobuliak66]

As discussed blocking before review. And thanks!

[sanderegg]

waiting for unblock

[GitHK]

I will most likely pick this up when I'm back. It requires some changes. Please wait before reviewing it further, until we have not decided on how the internals are supposed to work.

[sanderegg]

1200 line... can we maybe go through it together later today? thanks!

[matusdrobuliak66]

Lets discuss after today folders meeting with POs, I think we might simplify the design significantly.

[matusdrobuliak66]

Thanks!

[sanderegg]

still find the errors completely not understandable. Also still don't get what the enforce_permission thing does. Also having default in private functions seems a bit dangerous. But I will not block 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