test: restore the previous hub-user override on teardown rather than popping it

Other modules install theirs at import time; popping left later modules
running against the real dependency. Surfaced by the same pattern in #21.

Co-authored-by: Cursor <cursoragent@cursor.com>

Author: sumerjohal

app/tests/test_internal_secret.py

@@ -41,9 +41,16 @@
 @pytest.fixture(autouse=True)
 def _env(monkeypatch):
     monkeypatch.setenv("TEST_STATUS_LIST_DIR", TESTKIT)
+    # Other modules install their hub-user override at import time, which is
+    # before this fixture runs and after it would have popped. Restore rather
+    # than pop, or every later module runs against the real dependency.
+    previous = app.dependency_overrides.get(require_hub_user)
     app.dependency_overrides[require_hub_user] = lambda: {"sub": "pancake-forwarded@demo.com"}
     yield
-    app.dependency_overrides.pop(require_hub_user, None)
+    if previous is None:
+        app.dependency_overrides.pop(require_hub_user, None)
+    else:
+        app.dependency_overrides[require_hub_user] = previous
 
 
 @pytest.fixture