Improve closing study workflow

[odeimaiz]

What do these changes do?

There was a bit a of a race condition when setting the lock to a study-card after closing the study, specially when running in not so powerful machines. This PR wants to fix that. Let's see.

e2e: Check study state before deleting

Related issue number

related to #1987

How to test

Checklist

• Did you change any service's API? Then make sure to bundle document and upgrade version (make openapi-specs, git commit ... and then make version-*)
• Unit tests for the changes exist
• Runs in the swarm
• Documentation reflects the changes
• New module? Add your github username to .github/CODEOWNERS

[codecov]

Codecov Report

Merging #2004 (8807f25) into master (9ff3024) will decrease coverage by 0.0%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           master   #2004     +/-   ##
========================================
- Coverage    73.7%   73.7%   -0.1%     
========================================
  Files         385     385             
  Lines       14210   14210             
  Branches     1434    1434             
========================================
- Hits        10480   10479      -1     
  Misses       3351    3351             
- Partials      379     380      +1     

Flag	Coverage Δ
integrationtests	63.0% <ø> (-0.1%)	⬇️
unittests	67.3% <ø> (-0.1%)	⬇️

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

Impacted Files	Coverage Δ
...webserver/computation_comp_tasks_listening_task.py	88.7% <0.0%> (-2.3%)	⬇️
.../director/src/simcore_service_director/producer.py	61.1% <0.0%> (+0.2%)	⬆️

[pcrespov]

Much better

[sanderegg]

great!