-
Notifications
You must be signed in to change notification settings - Fork 31
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
Tape vs Tap issue #34 #35
Conversation
Codecov Report
@@ Coverage Diff @@
## master #35 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 1 2 +1
Lines 13 28 +15
Branches 1 2 +1
=====================================
+ Hits 13 28 +15
Continue to review full report at Codecov.
|
…eathing room") around code examples and images
…h Testing image in README.md #34
…de in vending-machine.js
…chitecture-example from the end of both "advanced" tutorials
@iteles not "urgent" but good to get reviewed/merged this week. thank you! ✨ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This. Is. Quite. The. Update.
I've updated for some minor typos, flow and removed duplicate words, totally ready to merge.
Finally understand the difference between tape and tap very clearly 👍
|
||
If you found this _taster_ to front-end testing with Tape and JSDOM | ||
helpful, consider reading our more _comprehensive_ example: | ||
https://github.com/dwyl/todomvc-vanilla-javascript-elm-architecture-example |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
we _either_ need to "reset" the state of `COINS` _inside_ each test, | ||
***or*** we need to reset the state of `COINS` _before_ each test. | ||
|
||
Here is the difference: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
@iteles in preparation for releasing the "TodoMVC" example I have given this tutorial an update.
learn-tape
tutorial to coverTap
Migrate form Tape to Tap (Run Tests in PARALLEL!) ? #34 and explain the reasoning why someone would want to useTap
instead ofTape
with a clear use-case example. ✅Please review and merge if/when happy. thanks.