Integration tests: replace cb-helloblock with cb-insight and cb-blockr

[weilu]

Blockr flips out on tx propagation sometimes, so using cb-insight for that. Why not use cb-insight for everything? Well, it doesn't support getting multiple transactions, yet. Feature wise, cb-blockr is still more complete. I'm also using a patched version of cb-insights, because latest version in npm does not 1) support transaction.get and 2) include unconfirmed amount in address.summary.

https://coconut-macaroon.herokuapp.com/ is just a faucet I spawn up with @jprichardson's neat bitcoin-faucet (patched as well :P)

[weilu]

This closes #399

[dcousens]

Does this just go on forever?

[coveralls]

Coverage remained the same at 99.42% when pulling 713f038 on replace-cb-helloblock into 3532fce on master.

[dcousens]

ACK, LGTM.

Unless @rubensayshi can give us a way to integrate with https://www.blocktrail.com/ that doesn't require an API token, I'm happy with this.

@rubensayshi perhaps allow non-authenticated requests but just have them severely rate limited?

edit: That said, I don't like that that we have two libraries, but that will change in time.

[jprichardson]

Looks good, other than @weilu butchering my precious faucet :p

[weilu]

@jprichardson FML haha your faucet needs more patching butchering -- doesn't handle p2sh address man.

[coveralls]

Coverage remained the same at 99.42% when pulling 70d423f on replace-cb-helloblock into 3532fce on master.

[dcousens]

Tests still failing?

[coveralls]

Coverage remained the same at 99.42% when pulling 9a921ad on replace-cb-helloblock into 3532fce on master.

[coveralls]

Coverage remained the same at 99.42% when pulling 4028bd5 on replace-cb-helloblock into 3532fce on master.

[dcousens]

Tests pass. Looks good. Can we hold out on a response from @rubensayshi before merge?

[weilu]

sure.

[dcousens]

Merging, @rubensayshi feel free to comment on the above.