Skip to content

Conversation

@tiginamaria
Copy link
Contributor

Split ServerTest per feature test classes

Motivation and Context

How Has This Been Tested?

Breaking Changes

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

Additional context

@tiginamaria tiginamaria added the refactoring Making things better label Nov 3, 2025
@tiginamaria tiginamaria requested a review from kpavlov November 3, 2025 18:01
@kpavlov kpavlov added the tests label Nov 3, 2025
kpavlov
kpavlov previously approved these changes Nov 3, 2025
Copy link
Contributor

@kpavlov kpavlov left a comment

Choose a reason for hiding this comment

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

Peer-reviewed, LGTM

Split ServerTest per feature test classes
@kpavlov kpavlov merged commit 2fc0b40 into main Nov 3, 2025
4 checks passed
@kpavlov kpavlov deleted the tigina/server-features-test branch November 3, 2025 18:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

refactoring Making things better tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants