The scope of this task is to fix the Blockchain.getTxOut() function, and to add proper unit and integration tests for it.
The n input is not implemented correctly in the current implementation. Presumably this bug was introduced during the TypeScript conversion?
The current unit test does not catch this bug because of the way it is mocked. There really needs to be some data validation added to the function and unit tests to test each input variation.
There is no integration test for this function. One should be created. It would have caught this bug.
The scope of this task is to fix the
Blockchain.getTxOut()function, and to add proper unit and integration tests for it.The
ninput is not implemented correctly in the current implementation. Presumably this bug was introduced during the TypeScript conversion?The current unit test does not catch this bug because of the way it is mocked. There really needs to be some data validation added to the function and unit tests to test each input variation.
There is no integration test for this function. One should be created. It would have caught this bug.