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.

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

Jump to bottom

Playwright add foundation for search tests #6274

Merged
merged 1 commit into from
Oct 7, 2024
Merged

Playwright add foundation for search tests #6274

merged 1 commit into from
Oct 7, 2024
+464 −55

Conversation

emilghittasv
Copy link
Collaborator

  • Added nltk to the project as a dev dependency.
  • Updated the search_page.py page object by adding a couple of new locators, functions against those locators, changing the usage of super() to self and making the functions public.
  • Added search_synonym.py which lists a couple of synonyms that can be returned from SUMO when performing a search. Will reduce the list to the ones which are actually used in tests once the tests are written.
  • Updated utilities.py with adding the helper functions that will be responsible of handling the search term and evaluating if the correct search results are returned.
@emilghittasv
- This PR is the building block for the search tests.
4ab2447 
- Added nltk to the project as a dev dependency.
- Updated the search_page.py page object by adding a couple of new locators, functions against those locators, changing the usage of super() to self and making the functions public.
- Added search_synonym.py which lists a couple of synonyms that can be returned from SUMO when performing a search. Will reduce the list to the ones which are actually used in tests once the tests are written.
- Updated utilities.py with adding the helper functions that will be responsible of handling the search term and evaluating if the correct search results are returned.
@emilghittasv emilghittasv requested a review from escattone October 2, 2024 16:18
@emilghittasv emilghittasv merged commit 0c743ca into mozilla:main Oct 7, 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
2 participants
@emilghittasv @akatsoulas