boltlabs-inc / zeekoe

Zero-knowledge layer-2 payment channels
MIT License
24 stars 1 forks source link

Add `test-all` to test-zeekoe.py #317

Closed DariusParvin closed 3 years ago

DariusParvin commented 3 years ago

add test-allcommand to test-zeekoe.py to run all scenario tests closes #304

A couple of things to note:

  1. This only tests the happy paths (including dispute). It doesn't test for cases where zeekoe should fail, e.g. making a payment with insufficient funds.
  2. The tests don't verify that the customer or merchant daemons respond appropriately. What the test does check (using Pytezos) is that the operations it broadcasts are valid, and that the operations reach min_confirmations before continuing. What is missing from the test is making sure that zeekoe responds appropriately, e.g. that custClaim gets broadcasted after custClose, or that the merchant calls merchDispute on a revoked state. Currently you'd have to manually check on chain.

I'm thinking if this PR looks good then the above can be filed as issues.