Skip to content

test: use SQLite sessions in unit misc#39120

Merged
asukaminato0721 merged 4 commits into
langgenius:mainfrom
asukaminato0721:split/sqlite-unit-074-unit-misc
Jul 21, 2026
Merged

test: use SQLite sessions in unit misc#39120
asukaminato0721 merged 4 commits into
langgenius:mainfrom
asukaminato0721:split/sqlite-unit-074-unit-misc

Conversation

@asukaminato0721

@asukaminato0721 asukaminato0721 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

Split 074 of 077 from #38784. This PR scopes the SQLite-session migration to unit misc.

Files

  • api/tests/unit_tests/tools/test_api_tool.py
  • api/tests/unit_tests/tools/test_mcp_tool.py
  • api/tests/unit_tests/core/app/layers/test_pause_state_persist_layer.py

Authored scope

  • 133 changed lines (85 additions, 48 deletions)
  • 3 files

Temporary upstream autofix

The upstream autofix.ci workflow adds dify-agent-runtime/README.md (6 additions, 6 deletions) to every API PR. That shared bot diff will disappear here after #38784 merges.

Validation

  • Ruff passed for every changed Python file in the split series
  • Target tests passed independently: api/tests/unit_tests/tools/test_api_tool.py, api/tests/unit_tests/tools/test_mcp_tool.py, api/tests/unit_tests/core/app/layers/test_pause_state_persist_layer.py
  • Full split-series run: 2122 passed

Part of #32454

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Pyrefly Type Coverage

Metric Base PR Delta
Type coverage 54.29% 54.31% +0.02%
Strict coverage 53.78% 53.80% +0.02%
Typed symbols 34,862 34,887 +25
Untyped symbols 29,633 29,633 0
Modules 3043 3043 0

@asukaminato0721
asukaminato0721 marked this pull request as ready for review July 16, 2026 06:06
@dosubot dosubot Bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Jul 16, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Jul 20, 2026
@asukaminato0721
asukaminato0721 added this pull request to the merge queue Jul 20, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 20, 2026
@asukaminato0721
asukaminato0721 added this pull request to the merge queue Jul 20, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Jul 20, 2026
@dosubot dosubot Bot added size:XS This PR changes 0-9 lines, ignoring generated files. and removed size:S This PR changes 10-29 lines, ignoring generated files. labels Jul 20, 2026
@asukaminato0721
asukaminato0721 added this pull request to the merge queue Jul 21, 2026
Merged via the queue into langgenius:main with commit 891b2dc Jul 21, 2026
34 checks passed
@asukaminato0721
asukaminato0721 deleted the split/sqlite-unit-074-unit-misc branch July 21, 2026 02:06
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