Problem
The CONTRIBUTING.md strictly mandates testing for zero-value edge cases, but test.rs currently only tests standard positive values (e.g., 1000 and 500).
Expected behavior
New test cases are added to explicitly pass 0 and negative integers into the deposit and withdraw methods to verify the amount > 0 assertions trigger correctly.
Files to update
Project relevance
This satisfies the strict testing requirements outlined for contributors and prevents potential accounting manipulation vectors in the Vault.
Acceptance criteria
Problem
The
CONTRIBUTING.mdstrictly mandates testing for zero-value edge cases, buttest.rscurrently only tests standard positive values (e.g., 1000 and 500).Expected behavior
New test cases are added to explicitly pass
0and negative integers into thedepositandwithdrawmethods to verify theamount > 0assertions trigger correctly.Files to update
src/test.rsProject relevance
This satisfies the strict testing requirements outlined for contributors and prevents potential accounting manipulation vectors in the Vault.
Acceptance criteria
deposit(&user, &0)panics with the strictly positive error message