Skip to content

Fix test 32 find 004#21

Merged
fondation451 merged 2 commits intomainfrom
fix-TEST-32-FIND-004
Sep 22, 2025
Merged

Fix test 32 find 004#21
fondation451 merged 2 commits intomainfrom
fix-TEST-32-FIND-004

Conversation

@fondation451
Copy link
Copy Markdown
Collaborator

No description provided.

@fondation451 fondation451 self-assigned this Sep 22, 2025
Copilot AI review requested due to automatic review settings September 22, 2025 12:03
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds updated test snapshots for token contract tests that validate failure conditions for various operations when amounts are not positive or operations are empty.

  • Updates snapshot for safe transfer test to ensure it captures the correct failure state for non-positive amounts
  • Updates snapshot for redeem test to reflect proper handling of non-positive amounts
  • Adds new snapshots for mint operations that validate failure handling for non-positive amounts and empty operations

Reviewed Changes

Copilot reviewed 11 out of 11 changed files in this pull request and generated no comments.

Show a summary per file
File Description
test_safe_transfer_should_fail_if_amount_is_not_strictly_positive.1.json New test snapshot file capturing the expected state when safe transfer fails due to non-positive amount
test_redeem_should_fail_if_amount_is_not_positive.1.json Updates amount values from 0 to 100 in balance and supply tracking
test_mint_should_fail_if_amount_is_not_strictly_positive.1.json New test snapshot file for mint operation failure with non-positive amounts
test_mint_batch_should_fail_if_operations_is_empty.1.json New test snapshot file for mint batch operation failure when operations array is empty
test_mint_batch_should_fail_if_one_of_the_operations_is_not_strictly_positive.1.json New test snapshot file for mint batch operation failure when one operation has non-positive amount

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@fondation451 fondation451 merged commit a65a715 into main Sep 22, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants