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

✅(v2.0.0) Add tests for name removal on variety of sites #535

Open
2 of 9 tasks
arimgibson opened this issue Nov 7, 2022 · 0 comments
Open
2 of 9 tasks

✅(v2.0.0) Add tests for name removal on variety of sites #535

arimgibson opened this issue Nov 7, 2022 · 0 comments
Labels
enhancement New feature or request

Comments

@arimgibson
Copy link
Owner

arimgibson commented Nov 7, 2022

Description

In order to ensure the extension's functionality and compatibility with multiple sites and browsers, end-to-end testing should be integrated using Cypress. At least 3 initial tests should focus on the extension's functionality on sites where content is static and the Shadow DOM/Virtual DOM/Web Components are not being utilitized. Other tests should cover situations where the text on the site changes as the user interacts with it (Twitter feed, for example) and sites that utilize the Shadow DOM, React (non-SSR), Vue (non-SSR), etc. Proper test coverage requires ensuring compatibility with several of the most popular frameworks for developing front-end applications.

Problem

WIthout proper testing in place, the extension's functionality is not dependable. Updates may break certain features, and having a testing framework in place is a good way to prevent breaking changes.

Next Steps

  • Install Cypress
  • Init Cypress folders
  • Build tests for static sites
  • Build tests for interactive sites
  • Build tests for sites using the Shadow DOM and other Web Components
  • Build tests for sites using React
  • Build tests for sites using Angular
  • Build tests for sites using Vue
  • Build tests for sites using Svelte

Notes

If you are interested in completing a pull request for this issue, please tag @arimgibson to get approval first. This ensures that multiple people aren't working on the same PR at the same time, and helps to make sure you get credit for your work.

@arimgibson arimgibson added the enhancement New feature or request label Nov 7, 2022
@arimgibson arimgibson self-assigned this Nov 7, 2022
@arimgibson arimgibson removed their assignment Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

1 participant