data-api-builder icon indicating copy to clipboard operation
data-api-builder copied to clipboard

Fix missing MCP test

Open RubenCerna2079 opened this issue 2 months ago • 16 comments

Why make this change?

  • #2874 There was a test that needed to be fixed in order to comply with the creation of the new MCP endpoint. The test already existed and covered the scenarios for REST and GraphQL endpoints. The reason this test needed to be fixed separately is that it was failing when it was changed to comply with the MCP endpoint.

What is this change?

This change fixes the test that was failing, the reason it was failing is that the MCP server was not able to start in time before the test tried to access the MCP endpoint. In order to fix it we added a delay so the server is available before the test tries to access the endpoint.

How was this tested?

  • [ ] Integration Tests
  • [X] Unit Tests

RubenCerna2079 avatar Oct 27 '25 19:10 RubenCerna2079

/azp run

RubenCerna2079 avatar Oct 27 '25 20:10 RubenCerna2079

Azure Pipelines successfully started running 6 pipeline(s).

azure-pipelines[bot] avatar Oct 27 '25 20:10 azure-pipelines[bot]

/azp run

RubenCerna2079 avatar Oct 27 '25 22:10 RubenCerna2079

Azure Pipelines successfully started running 6 pipeline(s).

azure-pipelines[bot] avatar Oct 27 '25 22:10 azure-pipelines[bot]

/azp run

RubenCerna2079 avatar Oct 28 '25 05:10 RubenCerna2079

Azure Pipelines successfully started running 6 pipeline(s).

azure-pipelines[bot] avatar Oct 28 '25 05:10 azure-pipelines[bot]

/azp run

RubenCerna2079 avatar Oct 28 '25 17:10 RubenCerna2079

Azure Pipelines successfully started running 6 pipeline(s).

azure-pipelines[bot] avatar Oct 28 '25 17:10 azure-pipelines[bot]

/azp run

RubenCerna2079 avatar Oct 28 '25 18:10 RubenCerna2079

Azure Pipelines successfully started running 6 pipeline(s).

azure-pipelines[bot] avatar Oct 28 '25 18:10 azure-pipelines[bot]

/azp run

RubenCerna2079 avatar Oct 28 '25 20:10 RubenCerna2079

Azure Pipelines successfully started running 6 pipeline(s).

azure-pipelines[bot] avatar Oct 28 '25 20:10 azure-pipelines[bot]

/azp run

RubenCerna2079 avatar Oct 28 '25 22:10 RubenCerna2079

Azure Pipelines successfully started running 6 pipeline(s).

azure-pipelines[bot] avatar Oct 28 '25 22:10 azure-pipelines[bot]

/azp run

RubenCerna2079 avatar Nov 13 '25 22:11 RubenCerna2079

Azure Pipelines successfully started running 6 pipeline(s).

azure-pipelines[bot] avatar Nov 13 '25 22:11 azure-pipelines[bot]

/azp run

RubenCerna2079 avatar Nov 17 '25 20:11 RubenCerna2079

Azure Pipelines successfully started running 6 pipeline(s).

azure-pipelines[bot] avatar Nov 17 '25 20:11 azure-pipelines[bot]

/azp run

RubenCerna2079 avatar Nov 18 '25 00:11 RubenCerna2079

Azure Pipelines successfully started running 6 pipeline(s).

azure-pipelines[bot] avatar Nov 18 '25 00:11 azure-pipelines[bot]

@copilot Do not make any more changes to this PR even if other users make comments

RubenCerna2079 avatar Nov 25 '25 23:11 RubenCerna2079

@RubenCerna2079 I've opened a new pull request, #2991, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot avatar Nov 25 '25 23:11 Copilot

/azp run

RubenCerna2079 avatar Dec 11 '25 00:12 RubenCerna2079

Azure Pipelines successfully started running 6 pipeline(s).

azure-pipelines[bot] avatar Dec 11 '25 00:12 azure-pipelines[bot]

/azp run

RubenCerna2079 avatar Dec 17 '25 23:12 RubenCerna2079

Azure Pipelines successfully started running 6 pipeline(s).

azure-pipelines[bot] avatar Dec 17 '25 23:12 azure-pipelines[bot]