Skip to content

test: Become a mentee Section #143 #202

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.

Sign up for GitHub

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

Open
joanaBrit wants to merge 6 commits into
base: main
Choose a base branch
from
Open

test: Become a mentee Section #143 #202

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
50547c4
Remove mentee registration option from mentorship dropdown
joanaBrit Nov 23, 2025
8b589e5
Create file for playright test
joanaBrit Dec 21, 2025
1e372c0
test: add title, description tests
joanaBrit Jan 11, 2026
8032165
playwright
joanaBrit Jan 11, 2026
832ae3e
Merge branch 'main' into test-143-become-a-mentee-playwright
joanaBrit Jan 11, 2026
c58ff9a
test: add button testing
joanaBrit Jan 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -59,5 +59,6 @@
"engines": {
"node": ">=20",
"pnpm": ">=9"
}
},
"packageManager": "pnpm@9.15.4+sha512.b2dc20e2fc72b3e18848459b37359a32064663e5627a51e4c74b2c29dd8e8e0491483c3abb40789cfd578bf362fb6ba8261b05f0387d76792ed6e23ea3b1b6a0"
}
50 changes: 50 additions & 0 deletions playwright-tests/tests/becomeAMentee.page.spec.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,50 @@
import { expect } from '@playwright/test';
import { test } from '@utils/fixtures';

test('Validate "Become a Mentee" section and Find a Mentor button', async ({
page,
}) => {
// Navigate to Mentorship page
await page.goto('/mentorship');

// Scroll to "Become a Mentee" section
const sectionTitle = page.getByRole('heading', {
Copy link
Contributor

@nora-weisser nora-weisser Jan 12, 2026

Choose a reason for hiding this comment

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

Could you please move this locator to mentorship.page.ts POM?

level: 4,
name: /Become a Mentee/i,
});
await sectionTitle.scrollIntoViewIfNeeded();
Copy link
Contributor

@nora-weisser nora-weisser Jan 12, 2026

Choose a reason for hiding this comment

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

There is no need to scroll to the section to validate it.

Copy link
Contributor Author

@joanaBrit joanaBrit Jan 18, 2026

Choose a reason for hiding this comment

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

Okay, why is it not necessary? Because we don’t have any interactions, apart from the button to go to a different page? Thank you! I’ll remove it. @nora-weisser


// Validate title
await expect(sectionTitle).toBeVisible();

// Validate description text
const description = page.getByRole('heading', {
Copy link
Contributor

@nora-weisser nora-weisser Jan 12, 2026

Choose a reason for hiding this comment

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

Could you move this to the mentorship page definition?

level: 5,
name: /You should become a mentee if you:/i,
});
await expect(description).toBeVisible();

// List items
const items = [
'Want to start a career in software engineering',
'Want to find a better job',
'Want to be promoted at work',
'Want to apply for a leadership position',
'Need support in advancing your career',
];

for (const item of items) {
await expect(page.getByText(item)).toBeVisible();
}
Comment on lines +36 to +38
Copy link
Contributor

@nora-weisser nora-weisser Jan 12, 2026

Choose a reason for hiding this comment

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

Since those items are list items in the DOM, I would locate list of items and validate against array you mentioned above.
https://playwright.dev/docs/api/class-locatorassertions#locator-assertions-to-have-text


// Validate "Find a mentor" button
const findMentorButton = page.getByRole('button', {
Copy link
Contributor

@nora-weisser nora-weisser Jan 12, 2026

Choose a reason for hiding this comment

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

Could you replace locator with the existing one on homePage: findMentorButton?

name: 'Find a mentor',
});
await expect(findMentorButton).toBeVisible();
await expect(findMentorButton).toBeEnabled();
await findMentorButton.click();
Comment on lines +44 to +46
Copy link
Contributor

@nora-weisser nora-weisser Jan 12, 2026

Choose a reason for hiding this comment

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

Visibility and enabled checks (44, 45 lines) are redundant before a click.
Playwright automatically ensures that:

  • the locator resolves to exactly one element
  • the element is visible
  • the element is stable (not animating)
  • the element is enabled

See the docs:
https://playwright.dev/docs/actionability


// Verify redirection
await expect(page).toHaveURL(/\/mentorship\/mentors/);
});