You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by mi...@apache.org on 2026/05/07 16:42:28 UTC

(superset) branch fix/mcp-list-datasets-schema-request-wrapper updated: test(mcp): add -> None return types and remove redundant ToolError import

This is an automated email from the ASF dual-hosted git repository.

aminghadersohi pushed a commit to branch fix/mcp-list-datasets-schema-request-wrapper
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/fix/mcp-list-datasets-schema-request-wrapper by this push:
     new 70d811aa4e3 test(mcp): add -> None return types and remove redundant ToolError import
70d811aa4e3 is described below

commit 70d811aa4e392f04ed236f046b6f70c2a279b56c
Author: Amin Ghadersohi <am...@gmail.com>
AuthorDate: Thu May 7 16:42:07 2026 +0000

    test(mcp): add -> None return types and remove redundant ToolError import
---
 tests/unit_tests/mcp_service/dataset/tool/test_dataset_tools.py | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/tests/unit_tests/mcp_service/dataset/tool/test_dataset_tools.py b/tests/unit_tests/mcp_service/dataset/tool/test_dataset_tools.py
index 66a7f3fba80..25a8a4c0fc6 100644
--- a/tests/unit_tests/mcp_service/dataset/tool/test_dataset_tools.py
+++ b/tests/unit_tests/mcp_service/dataset/tool/test_dataset_tools.py
@@ -2128,7 +2128,9 @@ class TestListDatasetsRequestWrapper:
 
     @patch("superset.daos.dataset.DatasetDAO.list")
     @pytest.mark.asyncio
-    async def test_request_wrapper_enforced_by_tool(self, mock_list, mcp_server) -> None:
+    async def test_request_wrapper_enforced_by_tool(
+        self, mock_list, mcp_server
+    ) -> None:
         """The MCP tool layer accepts the request wrapper and returns results.
 
         Verifies end-to-end that wrapping params in ``request={}`` works through