Skip to content

test: strengthen workflow collaboration assertions - #38608

Open
ojasarora77 wants to merge 2 commits into
langgenius:mainfrom
ojasarora77:test/clean-assert-once-workflow-collaboration
Open

test: strengthen workflow collaboration assertions#38608
ojasarora77 wants to merge 2 commits into
langgenius:mainfrom
ojasarora77:test/clean-assert-once-workflow-collaboration

Conversation

@ojasarora77

@ojasarora77 ojasarora77 commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

#37826

Summary

  • replace a bare assert_called_once() on set_session_info with assertions on workflow id and persisted session payload fields
  • replace a bare session.scalar.assert_called_once() with compiled query assertions that validate workflow/tenant filtering intent
  • keep scope to test_workflow_collaboration_service.py for a small, reviewable change

From Cursor

Test plan

  • uv run pytest tests/unit_tests/services/test_workflow_collaboration_service.py -q -o addopts=
  • uv run --dev ruff check tests/unit_tests/services/test_workflow_collaboration_service.py

Replace low-value bare call-count checks with payload and query assertions so tests verify behavior and filtering intent for workflow collaboration flows.
@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jul 9, 2026
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Pyrefly Type Coverage

Metric Base PR Delta
Type coverage 52.64% 52.64% 0.00%
Strict coverage 52.16% 52.16% 0.00%
Typed symbols 32,618 32,618 0
Untyped symbols 29,619 29,619 0
Modules 2986 2986 0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant