Skip to content

Add Savings Vault deposit boundary tests #555

Description

@El-swaggerito

Description

Difficulty: Expert

Type: Test

Summary

Add boundary tests for Savings Vault deposit behaviour.

Current Behaviour

Deposit logic may not be tested for zero values, very large values, repeated deposits, and invalid initialisation states.

Expected Behaviour

The test suite should verify deposit boundaries and ensure state remains consistent.

Suggested Implementation

Create boundary tests, invalid input tests, repeated deposit tests, and documentation updates.

Files or Areas Likely Affected

  • contracts/
  • src/
  • tests/
  • docs/

Acceptance Criteria

  • Zero deposit tests are added.
  • Large value deposit tests are added where practical.
  • Repeated deposit behaviour is tested.
  • Invalid initialisation state is tested.
  • State consistency is verified.
  • Docs explain deposit assumptions.

Additional Notes

This improves contract confidence.

Metadata

Metadata

Assignees

No one assigned

    Labels

    GrantFox OSSIssue tracked for GrantFox OSS contributionMaybe RewardedIssue may be eligible for a GrantFox rewardOfficial Campaign | FWC26Official GrantFox campaign labelcontractSmart contract related workexpertExpert difficultysorobanSoroban smart contract worktestTesting workvaultVault related work

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions