🐛Stop pruning /inputs folder after unzipping input port

[sanderegg]

What do these changes do?

https://github.com/ITISFoundation/osparc-simcore/pull/7697/files#diff-72320e9071f977f6ae6bb9a112730cdb7d5f1c8854cbfb08a2a80816196c5e83 introduced a bug in the dynamic-sidecar:

• when an input file is a zip file and the port type is not application/zip, then it gets downloaded and extracted in the corresponding port_key/ subfolder.
• Since the above mentioned PR, the folder was defined as a PrunableFolder which takes a snapshot of the current file tree under that folder. Once the prune function is called it deletes anything in that folder
• in the case of the zip file, the folder assigned to the PrunableFolder object is 1 too high in the hierarchy, thus this deletes all the other input_x folders around

This PR fixes this.

Others:

• add a more robust test around notifying service when comp_tasks table is modified

Related issue/s

• fixes Bug: Inputs not downloaded to Service osparc-issues#1942

How to test

Dev-ops

[codecov]

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 60.87%. Comparing base (22374df) to head (05235b4).
Report is 1 commits behind head on master.

❗ There is a different number of reports uploaded between BASE (22374df) and HEAD (05235b4). Click for more details.

HEAD has 32 uploads less than BASE

Flag	BASE (22374df)	HEAD (05235b4)
unittests	33	1

Additional details and impacted files

@@             Coverage Diff             @@
##           master    #8016       +/-   ##
===========================================
- Coverage   87.88%   60.87%   -27.01%     
===========================================
  Files        1853      696     -1157     
  Lines       71478    32808    -38670     
  Branches     1258      176     -1082     
===========================================
- Hits        62817    19972    -42845     
- Misses       8297    12778     +4481     
+ Partials      364       58      -306     

Flag	Coverage Δ		*Carryforward flag
integrationtests	58.03% <0.00%> (-6.22%)	⬇️	Carriedforward from 7bdf96a
unittests	89.19% <0.00%> (+2.70%)	⬆️

*This pull request uses carry forward flags. Click here to find out more.

Components	Coverage Δ
api	∅ <ø> (∅)
pkg_aws_library	∅ <ø> (∅)
pkg_celery_library	∅ <ø> (∅)
pkg_dask_task_models_library	∅ <ø> (∅)
pkg_models_library	∅ <ø> (∅)
pkg_notifications_library	∅ <ø> (∅)
pkg_postgres_database	∅ <ø> (∅)
pkg_service_integration	∅ <ø> (∅)
pkg_service_library	∅ <ø> (∅)
pkg_settings_library	∅ <ø> (∅)
pkg_simcore_sdk	76.98% <ø> (-8.13%)	⬇️
agent	∅ <ø> (∅)
api_server	∅ <ø> (∅)
autoscaling	∅ <ø> (∅)
catalog	∅ <ø> (∅)
clusters_keeper	∅ <ø> (∅)
dask_sidecar	∅ <ø> (∅)
datcore_adapter	∅ <ø> (∅)
director	∅ <ø> (∅)
director_v2	49.29% <ø> (-41.75%)	⬇️
dynamic_scheduler	∅ <ø> (∅)
dynamic_sidecar	90.09% <0.00%> (ø)
efs_guardian	∅ <ø> (∅)
invitations	∅ <ø> (∅)
payments	∅ <ø> (∅)
resource_usage_tracker	∅ <ø> (∅)
storage	∅ <ø> (∅)
webclient	∅ <ø> (∅)
webserver	58.91% <ø> (-28.76%)	⬇️

---

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 22374df...05235b4. Read the comment docs.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[matusdrobuliak66]

🥇 thanks

[wvangeit]

Nice find. Thanks.

[pcrespov]

🎉 thx so much!

[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 SonarQube Cloud