vulpemventures / go-elements

Go support for Liquid/Elements transactions
MIT License
27 stars 12 forks source link

[Pset] More integration tests #109

Closed altafan closed 3 years ago

altafan commented 4 years ago

We should create more tests for pset in order to increase the number of use cases covered by this package. Just to give an idea, we miss a test that creates a CT transaction with CT inputs and CT outputs.

A more detailed list will be defined soon.

altafan commented 4 years ago

List of possible transaction:

Also, existing tests use only P2WPKH inputs and we should test also the other types of inputs:

Given that, if we do some calculations, we have a total of 30 test cases, while we actually implemented 5 yet. We should also consider to refactor pset_test.go in order to reuse as much of code as possible or it will become a real jungle for someone to read this file.

altafan commented 3 years ago

closed by #134.