test: add pytest coverage for SSE transport functions

[passionateSandy2004]

Added a new pytest function, test_remove_request_params, in tests/shared/test_sse.py to cover the previously untested remove_request_params utility in the SSE client module.

Motivation and Context

The remove_request_params function in src/mcp/client/sse.py was not directly tested, leaving potential edge cases unverified. By adding a focused unit test, we ensure that:

• URLs with query parameters and fragments are correctly stripped down to their base form.
• URLs without parameters remain unchanged.

This aligns the SSE client tests with the project’s coverage standards and closes gaps in our automated test suite.

How Has This Been Tested?

• Ran pytest tests/shared/test_sse.py locally, confirming:
  • URLs like http://example.com/path?foo=bar#baz are normalized to http://example.com/path.
  • Clean URLs (e.g. https://mcp.dev/stream) pass through untouched.
• Executed full test suite (pytest) to verify no regressions.

Breaking Changes

None. This update only adds tests and does not alter library behavior or public APIs.

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 (pytest conventions)
• New and existing tests pass locally
• I have added appropriate error handling
• I have added or updated documentation as needed (test coverage)

Additional context

This change brings the SSE client’s test coverage to 100% for its public utilities and contributes to overall library reliability.

[mcp-shadow]

Minor suggestions for improvement:

1. Consider using parametrized tests to make the test cases more maintainable

   @pytest.mark.parametrize("input_url,expected_output", [
       ('http://example.com/test?foo=bar', 'http://example.com/test'),
       ('http://example.com/test#section', 'http://example.com/test'),
       ('http://example.com/test', 'http://example.com/test'),
       ('/test/path?x=1', '/test/path')
   ])
   def test_remove_request_params(input_url, expected_output):
       assert remove_request_params(input_url) == expected_output

2. Add a brief docstring to the test function to explain its purpose

[ihrpr]

Thank you for working on this!
Please can you address comments here

[ihrpr]

Closing this PR, looks stale now. Please re-open if the changes are still needed and suggestions applied.