Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Plugin E2E: Enable testing the variable editor #553

Merged
merged 19 commits into from
Dec 11, 2023
Merged

Plugin E2E: Enable testing the variable editor #553

merged 19 commits into from
Dec 11, 2023

Conversation

sunker
Copy link
Contributor

@sunker sunker commented Nov 22, 2023

What this PR does / why we need it:
This PR adds fixtures, models and expect matchers that enable testing a custom variable query editor and the variable query runner in a data source plugin.

Please not that the playwright tests under packages/plugin-e2e/tests are not actually part of the plugin-e2e package. They are there to test the features that are exposed by the package. They can also function as a spec of how the package can be used. We may want to move them to plugin-examples eventually.

Which issue(s) this PR fixes:

Part of #576

Fixes #

Special notes for your reviewer:

Copy link

github-actions bot commented Nov 22, 2023

Hello! 👋 This repository uses Auto for releasing packages using PR labels.

✨ This PR can be merged. It will not be considered when calculating future versions of the npm packages and will not appear in the changelogs.

@sunker sunker added the no-changelog Don't include in changelog and version calculations label Nov 22, 2023
Base automatically changed from query-editor to main November 24, 2023 18:35
@sunker sunker force-pushed the variable-editor branch 2 times, most recently from 349427d to 481446e Compare November 24, 2023 19:07
@sunker sunker changed the title Variable editor Plugin E2E: Enable testing the variable editor Nov 27, 2023
@sunker sunker marked this pull request as ready for review November 27, 2023 13:30
@sunker sunker mentioned this pull request Dec 11, 2023
6 tasks
@sunker sunker requested a review from academo December 11, 2023 14:56
Copy link
Collaborator

@jackw jackw left a comment

Choose a reason for hiding this comment

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

LGTM! 🚀

*
* Use {@link PanelEditPage.setVisualization} to change the visualization
* Use {@link PanelEditPage.datasource.set} to change the datasource
* Use {@link PanelEditPage.datasource.set} to change the
Copy link
Collaborator

Choose a reason for hiding this comment

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

nit: should this change should be reverted?

@sunker sunker merged commit 6cdd492 into main Dec 11, 2023
17 checks passed
@sunker sunker deleted the variable-editor branch December 11, 2023 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog Don't include in changelog and version calculations
Projects
Development

Successfully merging this pull request may close these issues.

3 participants