Skip to content

[MCP Bundle] Add test case for configuration #229

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.

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

Merged
merged 1 commit into from
Jul 30, 2025

Conversation

OskarStark
Copy link
Contributor

Q A
Bug fix? no
New feature? no
Docs? no
Issues --
License MIT

This commit adds comprehensive tests for the McpBundle configuration:

  • Tests default configuration values
  • Tests custom configuration for app name and version
  • Tests client transport configurations (stdio and sse)
  • Tests service registration based on transport settings
  • Tests autoconfiguration of MCP interfaces

@OskarStark OskarStark force-pushed the test/mcp-bundle-config branch 5 times, most recently from 3c5da57 to fa825b5 Compare July 30, 2025 10:25
@chr-hertel chr-hertel force-pushed the test/mcp-bundle-config branch from fa825b5 to fbb4a9c Compare July 30, 2025 12:10
@chr-hertel
Copy link
Member

Thank you @OskarStark.

@chr-hertel chr-hertel merged commit f1aa4e8 into symfony:main Jul 30, 2025
12 checks passed
@OskarStark OskarStark deleted the test/mcp-bundle-config branch July 30, 2025 12:19
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