Skip to content

escrow::contribute() rejecting a call after the escrow has been refunded (AlreadyRefunded) has no test #174

Description

@chonilius

contribute() in contracts/escrow/src/lib.rs rejects both AlreadyPaid and AlreadyRefunded via the same match escrow.status block. contracts/escrow/src/test.rs has test_contribute_rejects_after_already_paid, which funds an escrow, releases it, then asserts a subsequent contribute returns Error::AlreadyPaid — but there's no equivalent test funding an escrow, letting it lapse into Refunded (either via admin-triggered early refund or the permissionless post-deadline path), and then asserting contribute returns Error::AlreadyRefunded. Add the missing sibling test, mirroring test_contribute_rejects_after_already_paid's structure.

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workinggood first issueGood for newcomershelp wantedExtra attention is neededtestingTesting/QA infrastructure

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions