Skip to content

Commit

Permalink
test: remove redundant check in refund concrete test
Browse files Browse the repository at this point in the history
  • Loading branch information
smol-ninja committed Oct 26, 2024
1 parent 9d07774 commit 72c2d34
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
3 changes: 0 additions & 3 deletions tests/integration/concrete/refund/refund.t.sol
Original file line number Diff line number Diff line change
Expand Up @@ -163,9 +163,6 @@ contract Refund_Integration_Concrete_Test is Integration_Test {
uint128 expectedStreamBalance = depositedAmount - refundAmount;
assertEq(actualStreamBalance, expectedStreamBalance, "stream balance");

// Assert that the refund amounts equal.
assertEq(refundAmount, refundAmount);

// It should decrease the aggregate amount.
assertEq(flow.aggregateBalance(token), previousAggregateAmount - refundAmount, "aggregate amount");
}
Expand Down
6 changes: 2 additions & 4 deletions tests/integration/concrete/refund/refund.tree
Original file line number Diff line number Diff line change
Expand Up @@ -28,12 +28,10 @@ Refund_Integration_Concrete_Test
β”‚ β”œβ”€β”€ it should update the stream balance
β”‚ β”œβ”€β”€ it should decrease the aggregate amount
β”‚ β”œβ”€β”€ it should perform the ERC20 transfer
β”‚ β”œβ”€β”€ it should emit 1 {Transfer}, 1 {RefundFromFlowStream}, 1 {MetadataUpdate} event
β”‚ └── it should return the transfer amount
β”‚ └── it should emit 1 {Transfer}, 1 {RefundFromFlowStream}, 1 {MetadataUpdate} event
└── given token has 18 decimals
β”œβ”€β”€ it should make the refund
β”œβ”€β”€ it should update the stream balance
β”œβ”€β”€ it should decrease the aggregate amount
β”œβ”€β”€ it should perform the ERC20 transfer
β”œβ”€β”€ it should emit 1 {Transfer}, 1 {RefundFromFlowStream}, 1 {MetadataUpdate} event
└── it should return the transfer amount
└── it should emit 1 {Transfer}, 1 {RefundFromFlowStream}, 1 {MetadataUpdate} event

0 comments on commit 72c2d34

Please sign in to comment.