[LIMS-1541] Populate firstExperimendId and dewarRegistryId in dewars

[gfrn]

JIRA ticket: LIMS-1541

Summary:

Session IDs and dewar registry IDs are now pushed to Expeye when updating/creating dewars in SCAUP.

Changes:

• Populate firstExperimendId and dewarRegistryId in dewars

To test:

• Go to https://local-oidc-test.diamond.ac.uk:9000/proposals/bi23047/sessions/100/shipments/118/dewar/new/edit, add a new dewar, delete the sample on the right (you can also create a shipment from scratch, this is just quicker), click continue, then continue to pre-session information, submit the form and check if a new dewar was created in ISPyB with firstExperimentId and dewarRegistryId populated
• Go back to https://local-oidc-test.diamond.ac.uk:9000/proposals/bi23047/sessions/100/shipments/118/dewar/new/edit, click the dewar you just created, change the dewar code, click "save" and check if the dewar registry ID has been updated in the database (the name of the dewar in SCAUP won't change)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.31%. Comparing base (ad7d213) to head (a2d23e1).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master       #8      +/-   ##
==========================================
+ Coverage   94.25%   94.31%   +0.06%     
==========================================
  Files          38       38              
  Lines        1567     1584      +17     
==========================================
+ Hits         1477     1494      +17     
  Misses         90       90              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ndg63276]

If the name of the dewar in SCAUP won't change, would it make sense to not edit the name in ISPyB as well, just edit the facility code?

[gfrn]

If the name of the dewar in SCAUP won't change, would it make sense to not edit the name in ISPyB as well, just edit the facility code?

Ultimately, the idea is to make it so that it also changes in SCAUP, so the behaviour with regards to ISPyB is the correct one. It's not a complex change, so it's definitely on the list