🐛 Fix create soft link kwargs

[bisgaard-itis]

What do these changes do?

• This PR fixes a bug discovered by @wvangeit.
• It was caused by the changes I have recently done to how the error handling is done in the api-server
• I also add a test to ensure the function is covered by our tests. Seems that was not previously the case.

Related issue/s

How to test

Dev-ops checklist

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

[wvangeit]

🥳🥳🥳🥳

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 62.1%. Comparing base (cafbf96) to head (9bb3407).
Report is 349 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##           master   #6069      +/-   ##
=========================================
- Coverage    84.5%   62.1%   -22.5%     
=========================================
  Files          10     641     +631     
  Lines         214   31298   +31084     
  Branches       25     205     +180     
=========================================
+ Hits          181   19438   +19257     
- Misses         23   11809   +11786     
- Partials       10      51      +41     

Flag	Coverage Δ
integrationtests	59.4% <ø> (?)
unittests	89.8% <100.0%> (+5.2%)	⬆️

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

Files	Coverage Δ
...api_server/services/study_job_models_converters.py	95.6% <100.0%> (ø)

... and 650 files with indirect coverage changes

[bisgaard-itis]

thanks @pcrespov