fixtureqa 0.3.4__tar.gz → 0.3.5__tar.gz

{fixtureqa-0.3.4/fixtureqa.egg-info → fixtureqa-0.3.5}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: fixtureqa
-Version: 0.3.4
+Version: 0.3.5
 Summary: FIXture — FIX Protocol Testing Tool
 Requires-Python: >=3.10
 License-File: LICENSE

{fixtureqa-0.3.4 → fixtureqa-0.3.5}/fixture/api/routers/sessions.py

@@ -71,15 +71,20 @@ def _check_ownership(sm: SessionManager, session_id: str, user: User) -> Session
     return cfg
 
 
+def _owner_store(us: UserStore, user: User) -> Optional[UserStore]:
+    """Only admin responses carry owner_username (the UI owner badge / Mine filter)."""
+    return us if user.is_admin else None
+
+
 @router.get("/", response_model=list[SessionResponse])
 def list_sessions(sm: SM, us: US, user: CurrentUser):
     configs = sm.list_sessions_for_user(user.uid, is_admin=user.is_admin)
-    user_store = us if user.is_admin else None
+    user_store = _owner_store(us, user)
     return [_session_response(sm, cfg.session_id, user_store) for cfg in configs]
 
 
 @router.post("/", response_model=SessionResponse, status_code=201)
-async def create_session(body: SessionConfigRequest, sm: SM, user: CurrentUser):
+async def create_session(body: SessionConfigRequest, sm: SM, us: US, user: CurrentUser):
     try:
         ct = ConnectionType(body.connection_type)
     except ValueError:
@@ -127,11 +132,11 @@ async def create_session(body: SessionConfigRequest, sm: SM, user: CurrentUser):
         except Exception as e:
             raise HTTPException(status_code=500, detail=f"Session added but failed to start: {e}")
 
-    return _session_response(sm, session_id)
+    return _session_response(sm, session_id, _owner_store(us, user))
 
 
 @router.post("/import", response_model=list[SessionResponse], status_code=201)
-def import_sessions(body: list[SessionConfigRequest], sm: SM, user: CurrentUser):
+def import_sessions(body: list[SessionConfigRequest], sm: SM, us: US, user: CurrentUser):
     """
     Bulk-create sessions from an exported config file.
     Each entry is processed independently; invalid entries are skipped.
@@ -169,20 +174,20 @@ def import_sessions(body: list[SessionConfigRequest], sm: SM, user: CurrentUser)
                 owner_uid=user.uid,
             )
             session_id = sm.add_session(cfg)
-            created.append(_session_response(sm, session_id))
+            created.append(_session_response(sm, session_id, _owner_store(us, user)))
         except Exception:
             pass  # skip invalid / conflicting entries
     return created
 
 
 @router.get("/{session_id}", response_model=SessionResponse)
-def get_session(session_id: str, sm: SM, user: CurrentUser):
+def get_session(session_id: str, sm: SM, us: US, user: CurrentUser):
     _check_ownership(sm, session_id, user)
-    return _session_response(sm, session_id)
+    return _session_response(sm, session_id, _owner_store(us, user))
 
 
 @router.patch("/{session_id}", response_model=SessionResponse)
-def update_session(session_id: str, body: UpdateSessionRequest, sm: SM, user: CurrentUser):
+def update_session(session_id: str, body: UpdateSessionRequest, sm: SM, us: US, user: CurrentUser):
     _check_ownership(sm, session_id, user)
     try:
         sm.update_session(session_id, body)
@@ -190,7 +195,7 @@ def update_session(session_id: str, body: UpdateSessionRequest, sm: SM, user: Cu
         raise HTTPException(status_code=409, detail=str(e))
     except (ValueError, KeyError) as e:
         raise HTTPException(status_code=422, detail=str(e))
-    return _session_response(sm, session_id)
+    return _session_response(sm, session_id, _owner_store(us, user))
 
 
 @router.delete("/{session_id}", status_code=204)
@@ -203,25 +208,26 @@ async def remove_session(session_id: str, sm: SM, user: CurrentUser):
 
 
 @router.post("/{session_id}/start", response_model=SessionResponse)
-async def start_session(session_id: str, sm: SM, user: CurrentUser):
+async def start_session(session_id: str, sm: SM, us: US, user: CurrentUser):
     _check_ownership(sm, session_id, user)
     try:
         await sm.start_session(session_id)
     except RuntimeError as e:
         raise HTTPException(status_code=409, detail=str(e))
-    return _session_response(sm, session_id)
+    return _session_response(sm, session_id, _owner_store(us, user))
 
 
 @router.post("/{session_id}/stop", response_model=SessionResponse)
 async def stop_session(
     session_id: str,
     sm: SM,
+    us: US,
     user: CurrentUser,
     force: bool = Query(default=False),
 ):
     _check_ownership(sm, session_id, user)
     await sm.stop_session(session_id, force=force)
-    return _session_response(sm, session_id)
+    return _session_response(sm, session_id, _owner_store(us, user))
 
 
 @router.post("/{session_id}/send")

{fixtureqa-0.3.4 → fixtureqa-0.3.5/fixtureqa.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: fixtureqa
-Version: 0.3.4
+Version: 0.3.5
 Summary: FIXture — FIX Protocol Testing Tool
 Requires-Python: >=3.10
 License-File: LICENSE

{fixtureqa-0.3.4 → fixtureqa-0.3.5}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "fixtureqa"
-version = "0.3.4"
+version = "0.3.5"
 description = "FIXture — FIX Protocol Testing Tool"
 requires-python = ">=3.10"
 dependencies = [

{fixtureqa-0.3.4 → fixtureqa-0.3.5}/tests/test_sessions.py

@@ -53,6 +53,23 @@ def test_create_session(authed):
     assert "session_id" in body
 
 
+def test_mutating_responses_carry_owner_username_for_admin(authed):
+    # The admin "Mine" filter matches on owner_username client-side; if any
+    # session response omits it, the upserted session vanishes from the list
+    # until a full refetch. Every endpoint must enrich it like GET /sessions/.
+    client, _ = authed
+    created = client.post("/api/sessions/", json=_ini(9876)).json()
+    assert created["owner_username"] == "admin"
+    sid = created["session_id"]
+    assert client.get(f"/api/sessions/{sid}").json()["owner_username"] == "admin"
+    patched = client.patch(f"/api/sessions/{sid}", json={"display_name": "renamed"}).json()
+    assert patched["owner_username"] == "admin"
+    stopped = client.post(f"/api/sessions/{sid}/stop").json()
+    assert stopped["owner_username"] == "admin"
+    imported = client.post("/api/sessions/import", json=[_ini(9877, "A", "B")]).json()
+    assert imported[0]["owner_username"] == "admin"
+
+
 def test_list_sessions(authed):
     client, _ = authed
     client.post("/api/sessions/", json=_ini(9001, "A", "B"))