[tests-only] add e2e test for progress bars #132
Conversation
Signed-off-by: nabim777 <[email protected]>
|
|
nabim777
force-pushed
the
tests/app-progress-bar
branch
10 times, most recently
from
6375fd9 to
012bb0e
Compare
nabim777
requested review from
anon-pradip,
amrita-shrestha,
saw-jan,
S-Panta,
nirajacharya2,
prashant-gurung899,
PrajwolAmatya and
individual-it
nabim777
force-pushed
the
tests/app-progress-bar
branch
2 times, most recently
from
5a70aa9 to
2b7121c
Compare
Signed-off-by: nabim777 <[email protected]>
nabim777
force-pushed
the
tests/app-progress-bar
branch
from
2b7121c to
59e6483
Compare
|
i can see multiple extension in one pipeline just want to know how they run, parallel or turn-by-turn ? Could you describe about hooks and running process in PR description (like hooks run separately for each extension or all extension have same hooks used). So that it will be easy for everyone to review your PR. Add link to doc to web-extension . Please keep in mind, while working on QA task, about your team members involvement? If you are working on new task in which most of the staff has not worked before, adding background in description is useful for everyone. |
| await logout(adminPage) | ||
| }) | ||
|
|
||
| test('select the progressBarOption from the web-app-progress-bars extension', async () => { |
There was a problem hiding this comment.
| test('select the progressBarOption from the web-app-progress-bars extension', async () => { | |
| test('select the progressBarOption on the account page', async () => { |
this only tests if the dropdown shows the Nyan Cat option and if its selectable, nothing more
Signed-off-by: nabim777 <[email protected]>
nabim777
force-pushed
the
tests/app-progress-bar
branch
from
1c12e36 to
9bfe0fb
Compare
| await logout(adminPage) | ||
| }) | ||
|
|
||
| test('select the progressBarOption on the account page', async () => { |
There was a problem hiding this comment.
| test('select the progressBarOption on the account page', async () => { | |
| test('select NyanCat progressBarOption on the account page', async () => { |
| await this.accountManageBtn.click() | ||
| } | ||
|
|
||
| async selectProgressBarExtension(){ |
There was a problem hiding this comment.
| async selectProgressBarExtension(){ | |
| async selectNyanCatProgressBarExtension(){ |
|
|
||
| this.progressBarSelector = this.page.locator('.extension-preference .vs__search') | ||
| this.progressBarCurrent = this.page.locator('.extension-preference .vs__selected span') | ||
| this.progressBarOption = this.page.getByText('Nyan Cat progress bar') |
There was a problem hiding this comment.
| this.progressBarOption = this.page.getByText('Nyan Cat progress bar') | |
| this.nyanCatProgressBarOption = this.page.getByText('Nyan Cat progress bar') |
| await this.progressBarSelector.waitFor() | ||
| await this.progressBarSelector.click() |
There was a problem hiding this comment.
IMO click wait for selector to be visibile https://playwright.dev/docs/api/class-locator#locator-click why to add expicit waitfor ?
| await this.progressBarSelector.waitFor() | |
| await this.progressBarSelector.click() | |
| await this.progressBarSelector.click() |
Description
In this PR, an e2e test is added for the
web-app-progress-barextension.And currently there is only one web-app-progress-bar named
Nyan Cat progress bar, so there is only a test for it.steps added:
account managepageNyan Cat progress barTest run command
Related Issue
Screenshots (if appropriate):
Following video shows what's automated in the E2E test for
web-app-progress-barextension (added in this PR)Screencast.from.01-22-2025.04.50.54.PM.webm
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist: