Skip to content

test: migrate standard databases to MCP integration endpoints#2712

Open
anubhav756 wants to merge 1 commit intoanubhav-frameworkfrom
anubhav-standard-db
Open

test: migrate standard databases to MCP integration endpoints#2712
anubhav756 wants to merge 1 commit intoanubhav-frameworkfrom
anubhav-standard-db

Conversation

@anubhav756
Copy link
Contributor

@anubhav756 anubhav756 commented Mar 11, 2026

Overview

This PR migrates integration tests for standard operational and analytical databases from using explicit REST API layouts to native MCP endpoints.

Changes

  • Transitioned tests for over 25+ database platforms (PostgreSQL, MySQL, AlloyDB, Spanner, Redis, etc.) to leverage the newly built MCP test framework functions.
  • Resolved a bug inside SQL tool implementations that caused the MCP tools/list protocol schema to omit required template execution parameters by correctly returning mapped parameters via GetParameters().
  • Adjusted generic HTTP assertions to support deep evaluation of MCP JSON-RPC response structures and embedded error codes (like -32603).

Note

This PR is Part 2 of 5 to migrate legacy endpoints to MCP endpoints.
(Base PR: #2711)

@anubhav756 anubhav756 self-assigned this Mar 11, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Mar 11, 2026

Link Resolution Note

Local links and directory changes work differently on GitHub than on the docsite. You must ensure fixes pass the GitHub check and also work with hugo server.
See Link Checking and Fixing with Lychee for more details.

Summary

Status Count
🔍 Total 44
✅ Successful 38
⏳ Timeouts 0
👻 Excluded 0
❓ Unknown 0
🚫 Errors 1
⛔ Unsupported 0

Errors per input

Errors in DEVELOPER.md

  • [ERROR] https://play.dgraph.io/ | Network error: Connection failed. Check network connectivity and firewall settings (error sending request for url (https://play.dgraph.io/)): Connection failed. Check network connectivity and firewall settings

Full Github Actions output

@gemini-code-assist
Copy link
Contributor

Warning

Gemini encountered an error creating the summary. You can try again by commenting /gemini summary.

@anubhav756 anubhav756 force-pushed the anubhav-standard-db branch from 895b072 to 8349245 Compare March 11, 2026 15:32
@anubhav756 anubhav756 force-pushed the anubhav-framework branch 2 times, most recently from ea98ec8 to 3240483 Compare March 11, 2026 15:55
@anubhav756 anubhav756 force-pushed the anubhav-standard-db branch from 8349245 to 37e37b4 Compare March 11, 2026 15:55
@anubhav756 anubhav756 marked this pull request as ready for review March 11, 2026 19:40
@anubhav756 anubhav756 requested review from a team as code owners March 11, 2026 19:40
@anubhav756 anubhav756 added the tests: run Label to trigger Github Action tests. label Mar 11, 2026
@anubhav756 anubhav756 force-pushed the anubhav-standard-db branch from 37e37b4 to 742e2dc Compare March 12, 2026 04:25
@anubhav756 anubhav756 force-pushed the anubhav-standard-db branch from 742e2dc to 76271e6 Compare March 12, 2026 10:54
@anubhav756 anubhav756 force-pushed the anubhav-standard-db branch from 76271e6 to 7cae1ae Compare March 12, 2026 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests: run Label to trigger Github Action tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant