Skip to content

Conversation

@AishwaryaKalloli
Copy link
Contributor

@AishwaryaKalloli AishwaryaKalloli commented May 11, 2025

TESTED=all tests passing

Motivation and Context

There was one failing tests when I ran uv run pytest on my Windows machine. So I fixed it.
There was one flaky test, so I modified verification method.

How Has This Been Tested?

Rerunning the same tests again.

Breaking Changes

No

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation update

Checklist

  • I have read the MCP Documentation
  • My code follows the repository's style guidelines
  • New and existing tests pass locally
  • I have added appropriate error handling
  • I have added or updated documentation as needed

TESTED=all tests passing
…urrently.py

Calculating number of concurrent calls by tracking start and end time.

TESTED=all existing test cases passed.
@AishwaryaKalloli AishwaryaKalloli changed the title Test fix for windows machine. Test fixes for v1.7.x May 11, 2025
Copy link
Contributor

@ihrpr ihrpr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thank you!

@ihrpr ihrpr merged commit 9bee1d9 into modelcontextprotocol:v1.7.x May 12, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants