Author

Topic: x (Read 1062 times)

hero member
Activity: 518
Merit: 500
x
January 30, 2014, 06:37:45 AM
#5
You are missing tests for transaction.js, script.js and wallet.js and not taking in consideration third party modules (like jsbn and crypto-js).

You got some defined functions that arent being used (therefore tested if working), for example in index.js, rotl, rotr and endian.

You need to make tests cover exceptions being thrown (haven't found any in any file). since you are using qunit, assert.throws should do it.

On the derive of HDWallet, you are checking for this.priv, but the pub is not being covered (in case priv is false).

None of the base64 functions from convert.js are being covered.

This is what I got so far.

Btw, you should enable issues in your repo (since when you fork on github, it's not enabled by default)

Ouch - I think you should pay pocesar immediately Smiley
member
Activity: 105
Merit: 10
January 30, 2014, 05:02:43 AM
#4
You are missing tests for transaction.js, script.js and wallet.js and not taking in consideration third party modules (like jsbn and crypto-js).

You got some defined functions that arent being used (therefore tested if working), for example in index.js, rotl, rotr and endian.

You need to make tests cover exceptions being thrown (haven't found any in any file). since you are using qunit, assert.throws should do it.

On the derive of HDWallet, you are checking for this.priv, but the pub is not being covered (in case priv is false).

None of the base64 functions from convert.js are being covered.

This is what I got so far.

Btw, you should enable issues in your repo (since when you fork on github, it's not enabled by default)
hero member
Activity: 518
Merit: 500
January 26, 2014, 05:45:29 AM
#3
Will pay a 0.1 BTC bounty

How many lines of js are we talking?
hero member
Activity: 518
Merit: 500
January 23, 2014, 03:01:35 AM
#2
I re-implemented the BIP0032 wallet in vbuterin/bitcoinjs-lib and would like someone to review my work:

Source code: https://github.com/justcoin/bitcoinjs-lib/blob/master/src/hdwallet.js
Tests: https://github.com/justcoin/bitcoinjs-lib/blob/master/test/hdwallet.js

Is it customary to tip for code review?


If someone spends their time properly reviewing your code then I would certainly think a tip would be in order Smiley
full member
Activity: 232
Merit: 150
January 23, 2014, 02:54:30 AM
#1
x
Jump to: