Adding back default state type as per unit test

Author: davidpcls

bluesky_httpserver/database/orm.py

@@ -183,8 +183,7 @@ class Session(Timestamped, Base):
     expiration_time = Column(DateTime(timezone=False), nullable=False)
     principal_id = Column(Integer, ForeignKey("principals.id"), nullable=False)
     revoked = Column(Boolean, default=False, nullable=False)
-    # State allows for custom  authenticator information to be stored in the session.
-    state = Column(JSONVariant, nullable=False)
+    state = Column(JSONVariant, default=dict, nullable=False)
     principal: Mapped[Principal] = relationship(back_populates="sessions", lazy="joined")
 
 

bluesky_httpserver/tests/test_database.py

@@ -62,13 +62,7 @@ def test_session_state_defaults_to_empty_dict(sqlite_session):
     db.refresh(session)
     # Server default is '{}' so a session created without an explicit state
     # must not present as None to the ORM.
-    assert reloaded_state(session) == {}
-
-
-def reloaded_state(session):
-    # SQLite may return None if the server_default has not been re-selected;
-    # normalize.
-    return session.state if session.state is not None else {}
+    assert session.state == {}
 
 
 def test_get_or_create_principal_creates_when_missing(sqlite_session):