-
Notifications
You must be signed in to change notification settings - Fork 913
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
Migrate remaining VPN Selenium tests to Playwright #14904
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #14904 +/- ##
=======================================
Coverage 77.88% 77.88%
=======================================
Files 163 163
Lines 8480 8480
=======================================
Hits 6605 6605
Misses 1875 1875 ☔ View full report in Codecov by Sentry. |
7081513
to
bd698d5
Compare
bd698d5
to
12d8ea6
Compare
2f4ff6a
to
406dcd1
Compare
d8352a3
to
b685514
Compare
ff94529
to
c319715
Compare
c319715
to
a52a413
Compare
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.
Great job with this!
I just have one question and one nit highlighting that you missed a spot. It should be good to go after that. r+wc 🧮
const { test, expect } = require('@playwright/test'); | ||
const openPage = require('../../../scripts/open-page'); | ||
const url = '/en-US/products/vpn/features/'; | ||
const availableCountries = ['us', 'ca', 'gb', 'de', 'fr']; |
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.
Question: there are more countries where VPN is available in. Is this just a sample list to run the tests on some locations?
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.
The landing page experience is the same for all countries where VPN is available for, so I don't think it adds much to repeat the same test dozens of times. I think these core countries should be enough.
One-line summary
Add remaining tests for legacy VPN landing/pricing pages, as well as features page.
Issue / Bugzilla link
N/A
Testing
Successful test run: https://github.com/mozilla/bedrock/actions/runs/10159829857