-
Notifications
You must be signed in to change notification settings - Fork 31
test: Navigate to Website logo #191
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
base: main
Are you sure you want to change the base?
Conversation
|
|
||
| test('NAV-006 Validate logoclick navigation', async ({ basePage,homePage}) => { | ||
| await basePage.navigateToPath('/'); | ||
| await expect(basePage.logo).toBeVisible(); |
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.
It is unnecessary to verify if the logo is visible, click action is enough. If logo is not present, it will throw an error.
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.
Thanks. Done the suggested changes.
| await basePage.verifyURL('/mentorship/mentors'); | ||
| }); | ||
|
|
||
| test('NAV-006 Validate logoclick navigation', async ({ basePage,homePage}) => { |
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.
| test('NAV-006 Validate logoclick navigation', async ({ basePage,homePage}) => { | |
| test('NAV-006 Validate logo click navigation', async ({ basePage,homePage}) => { |
|
@nimat1 , could you please update docs as well? thanks! |
Done the changes, thanks ! |
|
Done the all suggested changes. |
|




Description
Type
Related Issue
Screenshots
Testing
Pull request checklist
Please check if your PR fulfills the following requirements: