Support for bitcoin transactions/address generation

[Midren]

Changes:

• Added coinType for address generation
• Fixed address generation tests to support coinType
• Add bitcoin_transaction_sign method to REST API

Does this change need to mentioned in CHANGELOG.md?
yes

Requires testing
yes

[Midren]

@therealssj Could you explain, why this test should have Failure msg, because I cannot find a reason?

[therealssj]

@RomanMilishchuk the test is just that when the gateway method TransactionSign returns a Failure msg then the httpResponse is 409 "failure msg"
so the body cannot have any errors

[therealssj]

while you are at it can you also fix these test names
https://github.com/SkycoinProject/hardware-wallet-daemon/blob/master/src/api/transaction_sign_test.go#L159

https://github.com/SkycoinProject/hardware-wallet-daemon/blob/master/src/api/transaction_sign_test.go#L177

[Midren]

@therealssj but why on the given inputs TransactionSign should fail with Failure msg?

@RomanMilishchuk the test is just that when the gateway method TransactionSign returns a Failure msg then the httpResponse is 409 "failure msg"
so the body cannot have any errors

[therealssj]

@RomanMilishchuk the test is a check for what happens when TransactionSign method returns 409 with some error msg

the test is basically checking that this part of code https://github.com/SkycoinProject/hardware-wallet-daemon/blob/master/src/api/transaction_sign.go#L110

is working as it shud

[Senyoret1]

@Midren do you have any test params which work with the new bitcoin_transaction_sign transaction endpoint? I tried to call it with this data:

{
	"transaction_inputs": [{
		"index": 0,
		"prev_hash": "0e3e2357e806b6cdb1f70b54c3a3a17b6714ee1f0e68bebb44a74b1efd512098"
	}],
	"transaction_outputs": [{
		"address_index": null,
		"address": "12c6DSiU4Rq3P4ZxziKxzrL5LmMBrzjrJX",
		"coins": "1"
	}]
}

Usign that data the response was protocol error: unexpected TXINPUT on every odd try and Failed with message: Signing transaction failed on every even try.