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

Add automated browser testing #377

Merged
merged 5 commits into from
Sep 27, 2023

Conversation

duggalsu
Copy link
Contributor

@duggalsu duggalsu commented Sep 20, 2023

  • Moved automated-testing to browser-extension

  • Added requirements.txt to reduce setup steps

  • Updated slur detection scripts for chrome, firefox

  • Added slur detection scripts for brave, chromium, edge

  • Updated gitignore

  • This addresses issue Automate testing of Uli #374

Edit 1: requirements.txt has pinned versions for reproducibility

- Moved automated-testing to browser-extension
- Added requirements.txt to reduce setup steps
- Updated slur detection scripts for chrome, firefox
- Added slur detection scripts for brave, chromium, edge
- Updated gitignore
@ghost
Copy link

ghost commented Sep 20, 2023

👇 Click on the image for a new way to code review

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map legend

@duggalsu duggalsu mentioned this pull request Sep 20, 2023
@dennyabrain
Copy link
Contributor

is there substantial difference in the chromium based browsers to warrant distinct test files? I'm thinking that an extension that we test on chrome works just fine in brave. So would automating chrome tests be good enough?
Wanted to know if you read somewhere or learnt about differences.

- Updated firefox automated testing
- Fixed firefox automation
- Added workaround for unknown bug for all browsers
@duggalsu
Copy link
Contributor Author

I have not read up on specific differences in chromium based browsers. However, there is a ChromeType flag controlling the webdriver to use for testing.

@duggalsu duggalsu changed the title Add automated browser testing [WiP] Add automated browser testing Sep 26, 2023
@dennyabrain dennyabrain merged commit d3cda62 into tattle-made:main Sep 27, 2023
3 checks passed
duggalsu added a commit that referenced this pull request Oct 24, 2023
* Add automated browser testing
- Moved automated-testing to browser-extension
- Added requirements.txt to reduce setup steps
- Updated slur detection scripts for chrome, firefox
- Added slur detection scripts for brave, chromium, edge
- Updated gitignore

* - Updated readme
- Updated firefox automated testing

* - Migrated to testing wiki pages than tweets
- Fixed firefox automation
- Added workaround for unknown bug for all browsers

* - Fixed sonarlint suggestion in firefox test
@duggalsu duggalsu deleted the automate_testing branch November 29, 2023 10:33
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