sous chef return deep copies of cached objects

[rboston628]

Description of work

It was possible for code in SNAPRed to edit the cached ingredients objects in SousChef, resulting in globally different sets of parameters.

This fixes that.

Explanation of work

Used deepcopy of the cached objects to prevent the original being overwritten.

Shallow copies cannot work here, as it is still possible to change things in a shallow copy that reflect in the original.

There are new tests to ensure the cached objects cannot be changed.

To test

Dev testing

Run diffcal, save, continue, run again same run, at tweak peak stage make sure the same peaks are there and same settings.

Run normal, make sure the tweak peak stage appears. If so, should be good.

Double check the unit tests.

CIS testing

N/A, but will be apparent in testing of PR #480

Link to EWM item

No ticket, but arose in development:

EWM#7388

Was discovered during testing of PR #480

Verification

N/A

Acceptance Criteria

N/A

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.39%. Comparing base (523b781) to head (b9d4834).
Report is 1 commits behind head on next.

Additional details and impacted files

@@           Coverage Diff           @@
##             next     #484   +/-   ##
=======================================
  Coverage   96.39%   96.39%           
=======================================
  Files          65       65           
  Lines        4740     4741    +1     
=======================================
+ Hits         4569     4570    +1     
  Misses        171      171           

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

[dlcaballero16]

Tested diffcal and parameters stay on 2nd run through. Normalization also works.