Skip to content
This repository was archived by the owner on Dec 5, 2019. It is now read-only.

Conversation

@tudorelu
Copy link

No description provided.

@coveralls
Copy link

coveralls commented May 14, 2019

Pull Request Test Coverage Report for Build 87

  • 25 of 39 (64.1%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.08%) to 53.954%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/packages/core/protocol/account.ts 6 9 66.67%
src/packages/core/api/account.ts 3 7 42.86%
src/packages/core/utils/crypto.ts 16 23 69.57%
Totals Coverage Status
Change from base Build 81: 0.08%
Covered Lines: 1244
Relevant Lines: 2046

💛 - Coveralls

@amalcaraz amalcaraz changed the base branch from master to feature/mobile-dapp-integration May 14, 2019 13:23
Copy link
Collaborator

@amalcaraz amalcaraz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributing, but please check the comments and try to fix it :)

 + added tests and cleaned up code
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants