Skip to content

test: use sqlite3 session in test_workspaces_members#38775

Merged
41tair merged 1 commit into
langgenius:mainfrom
asukaminato0721:sqlite3-controllers-openapi-test_workspaces_members
Jul 14, 2026
Merged

test: use sqlite3 session in test_workspaces_members#38775
41tair merged 1 commit into
langgenius:mainfrom
asukaminato0721:sqlite3-controllers-openapi-test_workspaces_members

Conversation

@asukaminato0721

Copy link
Copy Markdown
Contributor

Replace mocked db.session lookups with real SQLite-backed tenants, accounts, and memberships. Exercise real read services and verify the workspace-switch transaction commits current membership state while retaining external boundaries. Part of #32454.

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jul 10, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Pyrefly Diff

base → PR
--- /tmp/pyrefly_base.txt	2026-07-10 16:42:07.404259697 +0000
+++ /tmp/pyrefly_pr.txt	2026-07-10 16:41:52.068154694 +0000
@@ -2231,21 +2231,21 @@
 ERROR `in` is not supported between `Literal['GET']` and `None` [not-iterable]
   --> tests/unit_tests/controllers/openapi/test_workspaces.py:50:12
 ERROR Object of class `FunctionType` has no attribute `view_class` [missing-attribute]
-   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:184:12
+   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:200:12
 ERROR `in` is not supported between `Literal['POST']` and `None` [not-iterable]
-   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:185:12
+   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:201:12
 ERROR Object of class `FunctionType` has no attribute `view_class` [missing-attribute]
-   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:190:12
+   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:206:12
 ERROR `in` is not supported between `Literal['GET']` and `None` [not-iterable]
-   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:191:12
+   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:207:12
 ERROR `in` is not supported between `Literal['POST']` and `None` [not-iterable]
-   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:192:12
+   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:208:12
 ERROR Object of class `FunctionType` has no attribute `view_class` [missing-attribute]
-   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:197:12
+   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:213:12
 ERROR `in` is not supported between `Literal['DELETE']` and `None` [not-iterable]
-   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:198:12
+   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:214:12
 ERROR `in` is not supported between `Literal['PATCH']` and `None` [not-iterable]
-   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:199:12
+   --> tests/unit_tests/controllers/openapi/test_workspaces_members.py:215:12
 ERROR Cannot index into `Iterable[bytes]` [bad-index]
    --> tests/unit_tests/controllers/service_api/app/test_audio.py:186:16
 ERROR Cannot index into `Response` [bad-index]

@github-actions

Copy link
Copy Markdown
Contributor

Pyrefly Type Coverage

Metric Base PR Delta
Type coverage 52.85% 52.86% +0.01%
Strict coverage 52.36% 52.37% +0.01%
Typed symbols 32,803 32,814 +11
Untyped symbols 29,539 29,540 +1
Modules 2988 2988 0

@asukaminato0721
asukaminato0721 requested a review from 41tair July 14, 2026 05:27
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jul 14, 2026
@41tair
41tair added this pull request to the merge queue Jul 14, 2026
Merged via the queue into langgenius:main with commit 995de49 Jul 14, 2026
31 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants