-
-
Notifications
You must be signed in to change notification settings - Fork 156
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
Testing #34
Comments
I totally agree, but not sure how to test it. Suggestions and pull requests welcome. |
I think unit tests would get it most of the way covered without too much hassle. Integration tests with Electron is a different story. I could set up unit tests with mocha if that would interest you. |
Agreed, some unit test for the non-Electron specific behavior would be great. I added some initial test skeleton. There are two possible solutions, either extract the non-Electron functionality out into separate functions and export those for testing, or mock some of the Electron APIs. A pull request would be very much welcome :) |
@IssueHunt has funded $40.00 to this issue.
|
I'm disinclined to use libraries with tests. We should add them!
The text was updated successfully, but these errors were encountered: