-
Notifications
You must be signed in to change notification settings - Fork 3
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
Workshopper testing #13
Comments
I am not sure CI is possible. The issue that usually occurs is that on windows the tty doesn't work correctly. Testing it would mean to actually have a tty tool (that has different timings) and that is executed (per hand?) to make sure that the tty works. |
Workshops can have other kinds of tests. This test ensures all the recommended solutions pass: https://github.com/workshopper/browserify-adventure/blob/master/test/solutions.js |
I should try this again, since I misunderstood the initial case that @martinheidegger meant. Is there a way to test workshoppers against new and upcoming versions of node/npm? For example, TTY no longer working in Windows. |
@substack there is workshopper-adventure-test that should do that even further. But those are automated tests. The real problem, that we can't tackle easily, are integration tests. Depending on Node-version there are various things that can fail. i.e.:
So: In short: I think the tests that need interaction with other parties are of a nature to prevent bugs like above. I am not sure how many of those can be well automated considering that this my test matrix:
|
Let's discuss setting up a CI-type solution to test workshoppers comprehensively before release.jk sorta, please see this comment below.
The text was updated successfully, but these errors were encountered: