-
Notifications
You must be signed in to change notification settings - Fork 26
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: Use new e2e selectors #833
Conversation
@@ -37,6 +37,7 @@ nx-cloud.env | |||
|
|||
# End to End tests | |||
playwright-report/ | |||
packages/plugin-e2e/test-results/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All of a sudden Playwright started generating zip files related to reports. Anyway, this dir should definitely be ignored.
annotationsChoosePanelInput: string; | ||
}; | ||
Tooltip: { | ||
container: string; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
These selectors were not used in plugin-e2e.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Lots of new things in G11.
🚀 PR was released in |
What this PR does / why we need it:
This PR replaces a few unsafe locators with proper selectors defined grafana/e2e-selectors (and in the versioned selectors in plugin-e2e). The new selectors where recently added to the main branch of Grafana, hence we'll have to use the unsafe locator for versions before 11. This should be fine though as no one will ever backport selector updates to older version of Grafana.
Also updating related docs.
Which issue(s) this PR fixes:
Fixes #724
Special notes for your reviewer:
The selectors were added to Grafana in the following PRs.
grafana/grafana#83248
grafana/grafana#83246
grafana/grafana#83240
grafana/grafana#83230
📦 Published PR as canary version:
Canary Versions
✨ Test out this PR locally via:
npm install @grafana/create-plugin@4.3.0-canary.833.13a3f6f.0 npm install @grafana/plugin-e2e@0.23.0-canary.833.13a3f6f.0 # or yarn add @grafana/create-plugin@4.3.0-canary.833.13a3f6f.0 yarn add @grafana/plugin-e2e@0.23.0-canary.833.13a3f6f.0