Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 25, 2025

Why make this change?

  • Closes Refine Service.Tests changes for MCP #2874
  • Test for validating endpoint availability needed to be extended to cover MCP endpoints alongside existing REST and GraphQL tests. The original attempt failed due to MCP server not being ready in time.

What is this change?

  • Extended test parameters to include MCP endpoint testing with REST and GraphQL
  • Added GetMcpResponsePostConfigHydration helper method with retry logic to handle MCP server initialization timing
  • Used Task.Delay() instead of Thread.Sleep() for async-friendly waiting
  • Sourced endpoint paths from runtime options (RestRuntimeOptions, GraphQLRuntimeOptions, McpRuntimeOptions) instead of hardcoded strings
  • Fixed display names to accurately reflect test scenarios (e.g., "disabled" vs "enabled")

How was this tested?

  • Integration Tests
  • Unit Tests

Sample Request(s)

N/A - Internal test infrastructure change


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI mentioned this pull request Nov 25, 2025
2 tasks
Copilot AI changed the title [WIP] Fix missing MCP test for endpoint compliance Fix missing MCP test Nov 25, 2025
Copilot AI requested a review from RubenCerna2079 November 25, 2025 23:13
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