Merge pull request #3518 from PolicyEngine/fix/lazy-redis-import

Import redis lazily for deployed-service integration tests; taking liberty of merging before tests run because:
1. This affects something that will only be testable in integ tests
2. This literally only changes conftest

Author: anth-volk

changelog.d/lazy-redis-import.fixed.md

@@ -0,0 +1 @@
+Fixed staging integration test collection by importing `redis` lazily inside the `rest_client` fixture instead of at `tests/conftest.py` module import time.

tests/conftest.py

@@ -3,7 +3,6 @@
 from contextlib import contextmanager
 from subprocess import Popen, TimeoutExpired
 import sys
-import redis
 import pytest
 
 # Add the project root directory to PYTHONPATH
@@ -31,6 +30,7 @@ def running(process_arguments, seconds_to_wait_after_launch=0):
 def client():
     """run the app for the tests to run against"""
     from policyengine_api.api import app
+    import redis
 
     app.config["TESTING"] = True
     with running(["redis-server"], 3):