Skip to content
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

Playwright expanding group coverage for group leaders testing #6356

Merged

Conversation

emilghittasv
Copy link
Collaborator

  • Adding TestGroup3 to playwright.
  • Expanding the user_groups_messages.py to include locale leader removal or addition page messages.
  • Modifying the flows in user_group_flow.py to handle actions against group leaders.
  • Adding locators and functions for group leaders in groups_page.py page object. Adding a new test in test_groups.py which verifies:
  • That admin users can add and remove locale leaders from groups.
  • That the correct banner is displayed when adding or removing a group leader.
  • That adding a group leader to a group successfully adds the leader in both the leaders & members group lists.

- Expanding the user_groups_messages.py to include locale leader removal or addition page messages.
- Modifying the flows in user_group_flow.py to handle actions against group leaders.
- Adding locators and functions for group leaders in groups_page.py page object.
Adding a new test in test_groups.py which verifies:
- That admin users can add and remove locale leaders from groups.
- That the correct banner is displayed when adding or removing a group leader.
- That adding a group leader to a group successfully adds the leader in both the leaders & members group lists.
@emilghittasv emilghittasv force-pushed the playwright-add-more-group-tests branch from 1164d3b to 9a7a3c0 Compare November 13, 2024 19:20
@emilghittasv emilghittasv merged commit f6a3e99 into mozilla:main Nov 14, 2024
2 checks passed
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.

2 participants