question about the root chain test code

omgnetwork / plasma-mvp

OmiseGO's research implementation of Minimal Viable Plasma

MIT License

561 stars 158 forks source link

Closed geekchu closed 5 years ago

geekchu commented 6 years ago

I had a question when studying the test code in line 189 and line 190 in

plasma-mvp/tests/root_chain/contracts/root_chain/test_root_chain.py

assert_tx_failed(lambda: root_chain.challengeExit(utxo_pos4, utxo_pos1, tx_bytes4, proof, sigs, confirmSig[::-1]))`

I think the second param of challengeExit should be oindex1 rather than utxo_pos1, am i right?

smartcontracts commented 5 years ago

Yep! You're right. Thanks for opening this issue.