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

Feat/AN-4120 playwright e2e tests #93

Merged
merged 40 commits into from
Jul 30, 2024
Merged

Conversation

MGJamJam
Copy link
Contributor

@MGJamJam MGJamJam commented Jul 19, 2024

Issue: https://bitmovin.atlassian.net/browse/AN-4120

Work:

  • Adds Playwright e2e tests for plugin.
  • Adds default value for percentile value.
  • Increases the MinuteThreshold for the AUTO interval calculation to 3h1m

@MGJamJam MGJamJam requested a review from Svarozic July 23, 2024 18:24
@@ -60,34 +60,6 @@ jobs:
version: latest
args: buildAll

- name: Check for E2E
Copy link
Contributor

Choose a reason for hiding this comment

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

why changes in github workflow files ? Have they not been generated by grafana plugin generator tool ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

See answer below.
But good point, instead of having a seperate e2e workflow I will include this here. I will check how the ci file that is generated today with the tool looks like and copy it over.

Comment on lines -27 to -29
e2e-results/
**/cypress/videos
**/cypress/report.json
Copy link
Contributor

Choose a reason for hiding this comment

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

Why is "cypress" mentioned in project files ? I thought new plugin has only playwright ? is this just left over from angular grafana plugin ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Cypress is mentioned because grafana just recently deprecated cypress in their plugin development (https://grafana.com/docs/grafana/latest/breaking-changes/breaking-changes-v11-0/#the-grafanae2e-testing-tool-is-deprecated) and back in March when I created the plugin with the generator tool, this generator tool was still using cypress.

Comment on lines -3 to -10
e2e.scenario({
describeName: 'Smoke test',
itName: 'Smoke test',
scenario: () => {
e2e.pages.Home.visit();
e2e().contains('Welcome to Grafana').should('be.visible');
},
});
Copy link
Contributor

Choose a reason for hiding this comment

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

what created this files ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

The generation tool, see answer above 😊

e2e/configurationEditor.spec.ts Outdated Show resolved Hide resolved
package.json Show resolved Hide resolved
src/components/ConfigEditor.tsx Outdated Show resolved Hide resolved
src/components/GroupByRow.tsx Outdated Show resolved Hide resolved
src/types/grafanaTypes.ts Outdated Show resolved Hide resolved
src/utils/intervalUtils.ts Outdated Show resolved Hide resolved
@MGJamJam MGJamJam requested a review from Svarozic July 29, 2024 22:05
@MGJamJam MGJamJam merged commit 642d140 into main Jul 30, 2024
10 checks passed
@MGJamJam MGJamJam deleted the feat/AN-4120_playwright-e2e-tests branch July 30, 2024 10:07
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.

2 participants