Skip to content

Conversation

@nimat1
Copy link
Contributor

@nimat1 nimat1 commented Dec 25, 2025

Description

Type

  • Bug Fix
  • New Feature
  • Code Refactor
  • Documentation
  • Other

Related Issue

Screenshots

Testing

Pull request checklist

Please check if your PR fulfills the following requirements:

  • I checked and followed the contributor guide
  • I have tested my changes locally.
  • I have added a screenshot from the website after I tested it locally

@nimat1
Copy link
Contributor Author

nimat1 commented Dec 25, 2025

Wrong files, raising PR again
Passed test
image


test('NAV-006 Validate logoclick navigation', async ({ basePage,homePage}) => {
await basePage.navigateToPath('/');
await expect(basePage.logo).toBeVisible();
Copy link
Contributor

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.

Copy link
Contributor Author

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}) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
test('NAV-006 Validate logoclick navigation', async ({ basePage,homePage}) => {
test('NAV-006 Validate logo click navigation', async ({ basePage,homePage}) => {

@nora-weisser
Copy link
Contributor

@nimat1 , could you please update docs as well? thanks!

@nimat1
Copy link
Contributor Author

nimat1 commented Dec 26, 2025

@nimat1 , could you please update docs as well? thanks!

Done the changes, thanks !

@nimat1
Copy link
Contributor Author

nimat1 commented Dec 26, 2025

Done the all suggested changes.

@dricazenck dricazenck changed the title Navigate to Website logo test: Navigate to Website logo Dec 29, 2025
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 4, 2026

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.

3 participants