Skip to content
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

67 unit tests #152

Merged
merged 69 commits into from
Jan 10, 2017
Merged

67 unit tests #152

merged 69 commits into from
Jan 10, 2017

Conversation

JohnDanz
Copy link
Member

Unit tests so far... still have some to complete.

…st folder, found issue with tools resetting augur, commented out account.js unit tests until bug is fixed so unit tests will pass for now
…ccount to avoid errors with tests. I dont think these reset statements appear to be required
…g, filters, contracts, branches, batch, and modified accounts to no longer run tests that require connecting to a server since that is not a unit test. will need to rework tests so that they only test the code in augur.js, orignial implementations will be left in other test folders for integration testing
…he tests, still need a bit of work to get the same format
…rectly or im misunderstanding something. Added transact unit tests.
…ion to correctly pass onFailed to this.transact
…an error without failing gracefully when adding a transaction to batch without a contract defined.
…h an error key but never return it to the callback, added more unit tests to logs
…ompleteSetsLogs when options was passed as undefined
@coveralls
Copy link

coveralls commented Jan 10, 2017

Coverage Status

Coverage increased (+16.7%) to 49.898% when pulling 56d176b on 67-UnitTests into 789d0ac on master.

@tinybike tinybike merged commit 56d176b into master Jan 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants