Skip to content

Add dedicated Graph config schema test - #3950

Open
Nice6042 wants to merge 1 commit into
plotly:devfrom
Nice6042:test/graph-config-schema-3735
Open

Add dedicated Graph config schema test#3950
Nice6042 wants to merge 1 commit into
plotly:devfrom
Nice6042:test/graph-config-schema-3735

Conversation

@Nice6042

@Nice6042 Nice6042 commented Aug 14, 2026

Copy link
Copy Markdown

Closes #3735

Adds a dedicated Graph integration test that recursively compares the live Plotly config schema with the generated dcc.Graph config metadata. Known Plotly-internal paths remain excluded, and assertion messages identify the exact config path when the schemas drift.

Tests:

  • python -m pytest components/dash-core-components/tests/integration/graph/test_graph_basics.py --headless -q — 14 passed, 1 skipped
  • python -m black --check components/dash-core-components/tests/integration/graph/test_graph_basics.py — passed

Contributor Checklist

  • I have broken down my PR scope into the following TODO tasks
    • add a dedicated Graph config-schema test
    • verify schema drift produces a path-specific failure
  • I have run the tests locally and they passed. (refer to testing section in contributing)
  • I have added tests, or extended existing tests, to cover any new features or bugs fixed in this PR

optionals

  • I have added entry in the CHANGELOG.md
  • If this PR needs a follow-up in dash docs, community thread, I have mentioned the relevant URLS as follows
    • this GitHub #PR number updates the dash docs
    • here is the show and tell thread in Plotly Dash community

@sonarqubecloud

Copy link
Copy Markdown

@T4rk1n T4rk1n left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Test looks good, but I am not sure this should run along with the rest of tests, but rather a weekly job on ci that alerts us if it fails by creating an issue.
Let me know if you want to do that or prefer if you want I can set that up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Extract graph schema validation into properly named test

2 participants