More address testing

[kyledrake]

This completes the testing for Address.

The one case I didn't test is for this line: https://github.com/bitcoinjs/bitcoinjs-lib/blob/master/src/address.js#L8

Can someone explain the purpose of this line of code to me?

[kyledrake]

I'm fixing the spacing and adding another commit, ignore this pull

[abrkn]

Again I think the constructor is too flexible. Drop the:

 if (!(this instanceof Address)) { return new Address(bytes, version); } if (arguments[0] instanceof Address) { this.hash = arguments[0].hash; this.version = arguments[0].version; }

All these cloning functions are unnecessary