Skip to content

test: Add resilience tests for MCP connection collapse and tool watcher fan-out #1270

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

test: Add resilience tests for MCP connection collapse and tool watcher fan-out #1270

wants to merge 1 commit into from

Conversation

@towseef41
Copy link

@towseef41 towseef41 commented Nov 30, 2025

Description

  • Add resilience-focused unit tests: simulate MCP connection collapse to ensure _invoke_on_background_thread raises instead of hanging, and verify the tool watcher fans out file changes to all registries with a single observer.
  • Reset shared ToolWatcher state between tests to avoid cross-test leakage.
  • Clean up minor mypy complaints in _events.py tool/use accessors to keep lint passing.

Related Issues

Documentation PR

  • Not needed (test-only change).

Type of Change

  • Test
  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

  • pre-commit run --all-files (includes lint/unit tests)
  • hatch run prepare

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/m

1 participant

@towseef41