Fix #306 and Fix #362

[melanke]

Before merging this, I have 2 questions:

• Should we check for asset.Expiration <= _blockchain.Height + 1 ??
• Neo's current implementation is not verifying Receiving Scripts, it's commented, I also didn't verify.

[aboimpinto]

Missing the test for the verification of the transaction. The test should document the behavior and make sure that is working as expected.

[melanke]

@aboimpinto What do you mean? I wrote a lot of Unit Tests on this file:
https://github.com/CityOfZion/neo-sharp/blob/8c6c5686c824e6613b92360f4595d668a529fe80/test/NeoSharp.Core.Test/Models/UtTransactionOperationsManager.cs

Is it something else?

[aboimpinto]

sorry ... my bad.
For some reason the GitHub User Interface wasn't load the file to because is too big ...

I didn't check test by test, we can see on the CodeCov how this method is cover