-
-
Couldn't load subscription status.
- Fork 446
100% test coverage #396
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
100% test coverage #396
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #396 +/- ##
===========================================
+ Coverage 99.17% 100.00% +0.82%
===========================================
Files 1 1
Lines 364 379 +15
Branches 130 136 +6
===========================================
+ Hits 361 379 +18
+ Misses 3 0 -3 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| let value = ""; | ||
| let i = 0; | ||
|
|
||
| function name(value: string) { |
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.
Pulled name function out (which adds bytes) because it's needed to avoid a crash on name being missing before the token type error.
|
Requested myself so I can find it again if I dont get to it after the tc meeting |
2a3d359 to
5de5056
Compare

Fixing up 100% test coverage cases, additionally shifting some bytes around for readability and size (mainly the
.mapto aforloop to make the recursive case more obvious).