-
Notifications
You must be signed in to change notification settings - Fork 18
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
Write unit tests for untested utilities #12
Comments
Hey, I'm trying to get some JS testing experince and this looks like a good opportunity. Please assign me this task and I'll take a look at what useful tests I can add. |
Let's keep this issue open? There are many utilities in cross-domain-utils which would benefit from unit tests, if any others are looking for a good PR idea for hacktoberfest. |
A lot of functions are left untested because they call other (tested) functions internally. It would be quite easy to add more tests if Sinon is added as a dependency. |
There are a lot of useful functions in cross-domain-utils, but they aren't all tested! Write some unit tests for untested utilities.
To find what's missing, run
npm test
, thenopen coverage/index.html
to see what files are missing tests.The text was updated successfully, but these errors were encountered: