-
Notifications
You must be signed in to change notification settings - Fork 48
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 tests to the extension #89
Labels
Comments
Some tests are added for low level functionality, namely some parsing functionality. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The extension starts having enough features that a set of tests is appropriate.
I think some tests will need user interaction to be verified, but many modules could be tested automatically.
It would still be a significant amount of work, though worth it to prevent regression (which we have suffered from a couple of times in the past)
The text was updated successfully, but these errors were encountered: