move all redis session creation to factory

[pbugni]

This started as an attempt to mock redis for consistent testing outside of normal behavior.

That attempt failed thus far, but to capture all redis session initialization behind a single function still seems of value, for future work and easier debugging.

This PR includes a fence around the cache clearing redis communication necessary in a patient's timeline rebuild, to only hit redis when not under test. Necessary as otherwise, we experience a deadlock situation where tables can't be dropped between test runs, as postgres believes a deadlock condition exists.

[pep8speaks]

Hello @pbugni! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file portal/factories/redis.py:

Line 3:1: E302 expected 2 blank lines, found 1

• In the file portal/models/qb_timeline.py:

Line 767:80: E501 line too long (89 > 79 characters)
Line 767:90: W291 trailing whitespace
Line 768:80: E501 line too long (81 > 79 characters)

Comment last updated at 2024-03-05 21:57:21 UTC

[ivan-c]

looks good, thanks!