Amin Ghadersohi 4d80f87eb6 fix(mcp): fix E501 in update_chart.py and update_chart test mocks for column validation ...

Split an 89-char comment line and an over-limit condition in update_chart.py
to satisfy the ruff E501 rule. Also applied ruff format.

Two TestUpdateChartValidationGate tests expected CHART_VALIDATION_FAILED but
received CHART_DATASET_NOT_FOUND because _validate_update_against_dataset calls
DatasetValidator.validate_against_dataset before validate_and_compile, and the
existing mocks provided a Mock() object for chart.datasource whose .id attribute
is an auto-generated MagicMock (not a real int). Added a patch for
DatasetValidator.validate_against_dataset returning (True, None) so the
column-validation tier is bypassed and the test reaches the mocked
validate_and_compile response as intended.

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

2026-05-20 21:20:08 +00:00

..

common

…

consts

…

example_data

…

fixtures

…

integration_tests

test(charts): regression for last-modified sort order (#27500) (#40231)

2026-05-20 11:43:19 -07:00

unit_tests

fix(mcp): fix E501 in update_chart.py and update_chart test mocks for column validation

2026-05-20 21:20:08 +00:00

__init__.py

…

conftest.py

…