-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Other Things To Do #50
Comments
Hi @jlord -- I would love to get involved with this project and could start by setting up testing infrastructure and writing tests. Do you have any preferences for test frameworks? I've been experimenting with writing some unit tests for the lib modules using jasmine but I'm flexible. |
@LizzHale HI! Getting tests going would be amazing! In general I prefer the lightest weight infrastructure as possible and have used tap/tape. I'm sure that some of my code may need to be restructured—and that's OK, I want it to be better—so that each thing to be tested can be just be required in the test file, run with The first, most important things, in my mind, are the verify scripts themselves. This is where the most problems occur. This can also be tricky because many require requests but I'd been thinking it would be nice to have sample data for some of the GitHub requests to use so that tests can still work offline and without needing access to an account. I'd love to hear your thoughts! |
Thanks for the guidance! I've used TAP implementations for other languages but didn't realize one existed for JavaScript.
Agreed! I'll start experimenting and see if I can't come up with something that'll work for the verify scripts. |
Closing this out because these other things can just be their own issue 🍕 |
Not critical, but would be ✨
helper.js
andchallenge-completed.js
The text was updated successfully, but these errors were encountered: