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 adding the first batch of search tests #6278

Merged
merged 1 commit into from
Oct 10, 2024
Merged

Playwright adding the first batch of search tests #6278

merged 1 commit into from
Oct 10, 2024

Conversation

emilghittasv
Copy link
Collaborator

  • Refactoring the footer.py and homepage.py page objects by marking the page functions as public, replacing super() with self and updating function calls in tests.
  • Adding a couple of synonyms inside the search_synonym.py page which are used in tests.
  • Adding a WordNetLemmatizer instance in utilities.py which is going to help extracting the singular form of words when searching for synonyms of the search term inside the search results.
  • Extending playwright coverage with the first batch of search tests inside the test_main_searchbar.py.
  • Adding searchTests marker inside pytest.ini and playwright.yml
@emilghittasv
- Refactoring the footer.py and homepage.py page objects by marking t…
0824881 
…he page functions as public, replacing super() with self and updating function calls in tests.

- Adding a couple of synonyms inside the search_synonym.py page which are used in tests.
- Adding a WordNetLemmatizer instance in utilities.py which is going to help extracting the singular form of words when searching for synonyms of the search term inside the search results.
- Extending playwright coverage with the first batch of search tests inside the test_main_searchbar.py.
- Adding searchTests marker inside pytest.ini and playwright.yml
@emilghittasv emilghittasv merged commit 76eaef1 into mozilla:main Oct 10, 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 @escattone