Amin Ghadersohi a5d0205568 fix(mcp): fix MCPPermissionDeniedError handler order and visibility test patch targets ...

MCPPermissionDeniedError(PermissionError) was being caught by the generic
PermissionError branch in _handle_error before reaching its own handler,
so the ToolError message was sanitized to "You don't have access to this
resource." instead of the structured permission message.

Move the MCPPermissionDeniedError branch above PermissionError so the
subclass is matched first.

Also fix four visibility test patch targets: auth.py imports
security_manager at module level via `from superset import security_manager`,
so tests must patch `superset.mcp_service.auth.security_manager` (not
`superset.security_manager`) to intercept calls inside auth.py.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

2026-05-27 21:03:18 +00:00

..

common

…

consts

…

example_data

…

fixtures

…

integration_tests

test(model): roll back uncommitted ds_col mutations in timestamp-expression tests (#40451)

2026-05-26 21:17:08 -03:00

unit_tests

fix(mcp): fix MCPPermissionDeniedError handler order and visibility test patch targets

2026-05-27 21:03:18 +00:00

__init__.py

…

conftest.py

…