[dev] Add tests for coinbase maturity

keep-starknet-strange / raito

Bitcoin ZK client written in Cairo.

https://raito.wtf

MIT License

40 stars 34 forks source link

Closed m-kus closed 1 month ago

m-kus commented 1 month ago

Outputs of coinbase transaction cannot be spent sooner than 100 blocks.
There is a validation rule for that implemented, but it lacks unit tests.

Extend validation/transaction.cairo with at least two scenarios that cover happy and unhappy paths.

JoE11-y commented 1 month ago

Can I take a look at this? @m-kus

m-kus commented 1 month ago

@JoE11-y I have specified incorrect file in the issue, pls check

JoE11-y commented 1 month ago

@m-kus I've updated the PR, and fixed the merge issue.