Refactor testClientConnectWithInvalidJson

[devcrocod]

simplification

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
• Refactoring

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

[Copilot]

Pull Request Overview

This PR refactors the testClientConnectWithInvalidJson test to simplify error handling by replacing a verbose runCatching pattern with a more direct try-finally block using assertFailsWith.

Key Changes:

• Replaced runCatching().onSuccess().onFailure() chain with try-finally block
• Used assertFailsWith<IllegalStateException> for cleaner assertion of expected exception
• Removed unused imports (TimeoutCancellationException, fail)

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}