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
Json NEP6Wallet with basic tests #847
Json NEP6Wallet with basic tests #847
Conversation
Codecov Report
@@ Coverage Diff @@
## master #847 +/- ##
=========================================
+ Coverage 38.45% 38.7% +0.24%
=========================================
Files 176 176
Lines 12474 12482 +8
=========================================
+ Hits 4797 4831 +34
+ Misses 7677 7651 -26
Continue to review full report at Codecov.
|
@erikzhang I need a small help here... do you know why second test is giving null for ScryptParameters inside NEP6Wallet? |
Nevermind, all tests passed now!! \o/ |
@vncoelho json wallet, fine for you? |
* Json NEP6Wallet with basic tests * Scrypt is null, why * testing ScryptParameters * ScryptParameters not null, then null, why * string is correct, still null * shouldnt be null * correctly using out for readonly * reading from json * format * Scrypt to lowercase
This may help you @shargon on #791