Merge branch 'main' of github.com:codeforpdx/tenantfirstaid into feat/chat-refactor

Author: apkostka

.github/workflows/pr-check.yml

@@ -2,12 +2,15 @@ name: PR Checks
 
 on:
   pull_request:
-    types: [opened, synchronize, reopened, closed]
+    types: [opened, synchronize, reopened]
+
+  push:
+    branches:
+      - main
+      - 'releases/*'
 
 jobs:
   backend-test:
-    # skip this check for closed PRs that are not merged
-    if: ${{ github.event.action != 'closed' || github.event.pull_request.merged == true }}
     runs-on: ubuntu-latest
     defaults:
       run:
@@ -22,7 +25,7 @@ jobs:
           enable-cache: true
           cache-dependency-glob: "backend/uv.lock"
           # Install a specific version of uv.
-          version: "0.7.12"
+          version: "0.7.13"
 
       - name: Set up Python
         run: uv python install
@@ -46,8 +49,6 @@ jobs:
         run: uv run pytest -v -s
 
   frontend-build:
-    # skip this check for closed PRs that are not merged
-    if: ${{ github.event.action != 'closed' || github.event.pull_request.merged == true }}
     runs-on: ubuntu-latest
     defaults:
       run:

backend/pyproject.toml

@@ -6,10 +6,9 @@ dependencies = [
   "flask",
   "valkey",
   "gunicorn",
-  "openai",
+  "openai==1.89",
   "jsonlines",
   "simplejson",
-  "ipdb>=0.13.13",
   "python-dotenv",
   "pandas>=2.3.0",
 ]
@@ -24,11 +23,12 @@ build-backend = "setuptools.build_meta"
 
 [dependency-groups]
 dev = [
-    "pytest>=8.3.5",
+    "ipdb>=0.13.13",
+    "pytest>=8.4.0",
     "pytest-cov>=6.1.1",
     "pytest-mock>=3.14.1",
-    "ruff>=0.11.13",
-    "ty>=0.0.1a8",
+    "ruff>=0.12.0",
+    "ty>=0.0.1a11",
     "types-Flask>=1.1.6",
 ]
 

backend/tests/test_session.py

@@ -15,18 +15,13 @@ def mock_valkey_ping_nop(mocker):
 
 @pytest.fixture
 def mock_valkey(mock_valkey_ping_nop, mocker):
-    # mock_valkey_dbcon_client = mocker.Mock()
-    # mocker.patch("tenantfirstaid.session.Valkey", mock_valkey_dbcon_client)
-
-    _data: Dict[str, Any] = {}
+    _data: Dict[str, str] = {}
 
     mock_valkey_ping_nop.set = mocker.Mock(
         side_effect=lambda key, value: _data.update({key: value})
     )
 
-    mock_valkey_ping_nop.get = mocker.Mock(
-        return_value=lambda key: _data.get(key, None)
-    )
+    mock_valkey_ping_nop.get = mocker.Mock(side_effect=lambda key: _data[key])
 
     return mock_valkey_ping_nop
 
@@ -81,10 +76,7 @@ def test_session_init_ping_exception(mocker, capsys):
 
 
 def test_session_get_unknown_session_id(mocker, mock_environ):
-    test_data = {
-        "city": "Test City",
-        "state": "Test State",
-    }
+    test_data = {"city": "Test City", "state": "Test State", "messages": []}
 
     mock_valkey_client = mocker.Mock()
     mocker.patch("tenantfirstaid.session.Valkey", return_value=mock_valkey_client)
@@ -111,30 +103,62 @@ def test_session_get_unknown_session_id(mocker, mock_environ):
         }
 
 
-# def test_session_set_and_get(mocker, mock_environ, mock_valkey):
-#     test_data: Dict[str, Any] = {
-#         "city": "Test City",
-#         "state": "Test State",
-#         "messages": ["this is message 1", "this is message 2"],
-#     }
-
-#     mock_valkey_client = mocker.Mock()
-#     mocker.patch("tenantfirstaid.session.Valkey", return_value=mock_valkey_client)
-#     mock_valkey_client.ping = mocker.Mock()
-
-#     tenant_session = TenantSession()
-#     app = Flask(__name__)
-#     app.add_url_rule(
-#         "/api/init",
-#         view_func=InitSessionView.as_view("init", tenant_session),
-#         methods=["POST"],
-#     )
-#     app.secret_key = "test_secret_key"  # Set a secret key for session management
-
-#     with app.test_request_context("/api/init", method="POST", json=test_data):
-#         response = app.full_dispatch_request()
-#         assert response.status_code == 200  # Ensure the response is successful
-#         session_id = response.json["session_id"]
-
-#         tenant_session.set(session_id, test_data)
-#         assert tenant_session.get() == test_data
+def test_session_set_and_get(mocker, mock_environ, mock_valkey):
+    test_data_obj: Dict[str, Any] = {
+        "city": "Test City",
+        "state": "Test State",
+        "messages": ["this is message 1", "this is message 2"],
+    }
+
+    tenant_session = TenantSession()
+    app = Flask(__name__)
+    app.add_url_rule(
+        "/api/init",
+        view_func=InitSessionView.as_view("init", tenant_session),
+        methods=["POST"],
+    )
+    app.secret_key = "test_secret_key"  # Set a secret key for session management
+
+    with app.test_request_context("/api/init", method="POST", json=test_data_obj):
+        response = app.full_dispatch_request()
+        assert response.status_code == 200  # Ensure the response is successful
+        session_id = response.json["session_id"]
+        assert session_id is not None  # Ensure session_id is set
+        assert isinstance(session_id, str)  # Ensure session_id is a string
+
+        tenant_session.set(session_id, test_data_obj)
+        assert tenant_session.get() == test_data_obj
+
+
+def test_session_set_some_and_get_none(mocker, mock_environ, mock_valkey):
+    test_data_obj: Dict[str, Any] = {
+        "city": "Test City",
+        "state": "Test State",
+        "messages": ["this is message 1", "this is message 2"],
+    }
+
+    tenant_session = TenantSession()
+    app = Flask(__name__)
+    app.add_url_rule(
+        "/api/init",
+        view_func=InitSessionView.as_view("init", tenant_session),
+        methods=["POST"],
+    )
+    app.secret_key = "test_secret_key"  # Set a secret key for session management
+
+    # Simulate no data for the session (i.e. network error or similar)
+    mock_valkey.get.side_effect = lambda key: None
+
+    with app.test_request_context("/api/init", method="POST", json=test_data_obj):
+        response = app.full_dispatch_request()
+        assert response.status_code == 200  # Ensure the response is successful
+        session_id = response.json["session_id"]
+        assert session_id is not None  # Ensure session_id is set
+        assert isinstance(session_id, str)  # Ensure session_id is a string
+
+        tenant_session.set(session_id, test_data_obj)
+        assert tenant_session.get() == {
+            "city": "",
+            "state": "",
+            "messages": [],
+        }