TDD: Track session state post-close access in tests

[yellowhatter]

This PR is for discussion and adds panic in tests if Session internal state is accessed after Session is being closed. I believe that to have proper design we should make this work

[github-actions]

PR missing one of the required labels: {'breaking-change', 'bug', 'documentation', 'dependencies', 'new feature', 'internal', 'enhancement'}

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.07%. Comparing base (037c546) to head (ff74d91).
Report is 23 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1905      +/-   ##
==========================================
- Coverage   71.25%   71.07%   -0.19%     
==========================================
  Files         364      364              
  Lines       65708    65714       +6     
==========================================
- Hits        46823    46707     -116     
- Misses      18885    19007     +122     

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

[wyfo]

Accessing SessionState after session close is perfectly valid. Session::close takes a shared reference, so when the session will be dropped after having been closed, it will access the state and raise an error with your PR — I guess that's not expected.