yuqi1129 opened a new pull request, #13121: URL: https://github.com/apache/gravitino/pull/13121
### What changes were proposed in this pull request? Extract `tags` from the detailed metadata tag list response instead of `names`. Add regression tests for complete tag details, empty results, server errors, and the MCP tool response. ### Why are the changes needed? The request uses `details=true`, so the REST API returns `tags`. Reading the absent `names` field silently returns `[]` for objects with associated tags. Fix: #13120 ### Does this PR introduce _any_ user-facing change? `list_tags_for_metadata` returns the documented tag objects, including comments, properties, audit information, and inheritance flags. ### How was this patch tested? - The REST client and MCP tool regression tests fail before the fix and pass afterward. - MCP unit suite: 272 tests and 9 subtests passed. - `./gradlew :mcp-server:spotlessApply`, targeted Pylint (10/10), and `git diff --check` passed. - HTTP responses are simulated with `httpx.MockTransport`; no live Gravitino server is required for these tests. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
