Fix EventHandlingSimulator::get_simulation_state

[dweindl]

Previously, get_simulation_state not only created a SimulationState but also updated the current workspace. This is unexpected and causes problems when handling nested events when the workspace is updated but the solver was not reinitialized yet (kind of reverting the primary event assignment). Thus, decouple snapshotting of the current state from pulling Solver state.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.72%. Comparing base (e728c2a) to head (e485009).
⚠️ Report is 63 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2901      +/-   ##
==========================================
- Coverage   80.31%   79.72%   -0.59%     
==========================================
  Files         330      330              
  Lines       22641    22634       -7     
  Branches     1532     1530       -2     
==========================================
- Hits        18184    18046     -138     
- Misses       4447     4578     +131     
  Partials       10       10              

Flag	Coverage Δ
cpp	75.50% <100.00%> (-0.01%)	⬇️
cpp_python	33.77% <100.00%> (-0.02%)	⬇️
petab	39.19% <100.00%> (-0.02%)	⬇️
python	74.48% <100.00%> (-0.01%)	⬇️
sbmlsuite-jax	?

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

Files with missing lines	Coverage Δ
src/forwardproblem.cpp	89.55% <100.00%> (+0.04%)	⬆️

... and 6 files with indirect coverage changes

🚀 New features to boost your workflow:

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