voucher can be expired after validUntilDate

bosonprotocol · Jun 21, 2023 · 2c6a218 · 2c6a218
1 parent 451dc3d
commit 2c6a218
Show file tree

Hide file tree

Showing 6 changed files with 43 additions and 20 deletions.
diff --git a/contracts/protocol/facets/ExchangeHandlerFacet.sol b/contracts/protocol/facets/ExchangeHandlerFacet.sol
@@ -382,7 +382,7 @@ contract ExchangeHandlerFacet is IBosonExchangeHandler, BuyerBase, DisputeBase {
         (Exchange storage exchange, Voucher storage voucher) = getValidExchange(_exchangeId, ExchangeState.Committed);
 
         // Make sure that the voucher has expired
-        require(block.timestamp >= voucher.validUntilDate, VOUCHER_STILL_VALID);
+        require(block.timestamp > voucher.validUntilDate, VOUCHER_STILL_VALID);
 
         // Finalize the exchange, burning the voucher
         finalizeExchange(exchange, ExchangeState.Canceled);

diff --git a/test/protocol/DisputeHandlerTest.js b/test/protocol/DisputeHandlerTest.js
@@ -487,7 +487,7 @@ describe("IBosonDisputeHandler", function () {
           block = await ethers.provider.getBlock(blockNumber);
           escalatedDate = block.timestamp.toString();
 
-          await setNextBlockTimestamp(Number(escalatedDate) + Number(escalationPeriod));
+          await setNextBlockTimestamp(Number(escalatedDate) + Number(escalationPeriod) + 1);
 
           // Attempt to retract the dispute, expecting revert
           await expect(disputeHandler.connect(buyer).retractDispute(exchangeId)).to.revertedWith(
@@ -1079,7 +1079,7 @@ describe("IBosonDisputeHandler", function () {
 
           it("Dispute has expired", async function () {
             // Set time forward to the dispute expiration date
-            await setNextBlockTimestamp(Number(timeout));
+            await setNextBlockTimestamp(Number(timeout) + 1);
 
             // Attempt to resolve the dispute, expecting revert
             await expect(
@@ -1639,7 +1639,7 @@ describe("IBosonDisputeHandler", function () {
 
           it("Dispute escalation response period has elapsed", async function () {
             // Set time past escalation period
-            await setNextBlockTimestamp(Number(escalatedDate) + Number(escalationPeriod));
+            await setNextBlockTimestamp(Number(escalatedDate) + Number(escalationPeriod) + 1);
 
             // Attempt to decide the dispute, expecting revert
             await expect(
@@ -1671,7 +1671,7 @@ describe("IBosonDisputeHandler", function () {
 
         it("should emit a EscalatedDisputeExpired event", async function () {
           // Set time forward past the dispute escalation period
-          await setNextBlockTimestamp(Number(escalatedDate) + Number(escalationPeriod));
+          await setNextBlockTimestamp(Number(escalatedDate) + Number(escalationPeriod) + 1);
 
           // Expire the escalated dispute, testing for the event
           await expect(disputeHandler.connect(rando).expireEscalatedDispute(exchangeId))
@@ -1681,7 +1681,7 @@ describe("IBosonDisputeHandler", function () {
 
         it("should update state", async function () {
           // Set time forward past the dispute escalation period
-          await setNextBlockTimestamp(Number(escalatedDate) + Number(escalationPeriod));
+          await setNextBlockTimestamp(Number(escalatedDate) + Number(escalationPeriod) + 1);
 
           // Expire the dispute
           tx = await disputeHandler.connect(rando).expireEscalatedDispute(exchangeId);
@@ -1729,7 +1729,7 @@ describe("IBosonDisputeHandler", function () {
         context("💔 Revert Reasons", async function () {
           it("The disputes region of protocol is paused", async function () {
             // Set time forward past the dispute escalation period
-            await setNextBlockTimestamp(Number(escalatedDate) + Number(escalationPeriod));
+            await setNextBlockTimestamp(Number(escalatedDate) + Number(escalationPeriod) + 1);
 
             // Pause the disputes region of the protocol
             await pauseHandler.connect(pauser).pause([PausableRegion.Disputes]);
@@ -1931,7 +1931,7 @@ describe("IBosonDisputeHandler", function () {
 
           it("Dispute escalation response period has elapsed", async function () {
             // Set time forward past the dispute escalation period
-            await setNextBlockTimestamp(Number(escalatedDate) + Number(escalationPeriod));
+            await setNextBlockTimestamp(Number(escalatedDate) + Number(escalationPeriod) + 1);
 
             // Attempt to refuse the escalated dispute, expecting revert
             await expect(disputeHandler.connect(assistantDR).refuseEscalatedDispute(exchangeId)).to.revertedWith(
@@ -2083,7 +2083,7 @@ describe("IBosonDisputeHandler", function () {
 
         it("should return the expected dispute state if exchange id is valid and dispute has expired", async function () {
           // Set time forward to the dispute's timeout
-          await setNextBlockTimestamp(Number(timeout));
+          await setNextBlockTimestamp(Number(timeout) + 1);
 
           // Anyone calls expireDispute
           await disputeHandler.connect(rando).expireDispute(exchangeId);
@@ -2336,7 +2336,7 @@ describe("IBosonDisputeHandler", function () {
             block = await ethers.provider.getBlock(blockNumber);
             escalatedDate = block.timestamp.toString();
 
-            await setNextBlockTimestamp(Number(escalatedDate) + Number(escalationPeriod));
+            await setNextBlockTimestamp(Number(escalatedDate) + Number(escalationPeriod) + 1);
 
             // Expire dispute
             await disputeHandler.connect(rando).expireEscalatedDispute(exchangeId);

diff --git a/test/protocol/ExchangeHandlerTest.js b/test/protocol/ExchangeHandlerTest.js
@@ -1979,6 +1979,14 @@ describe("IBosonExchangeHandler", function () {
           await expect(exchangeHandler.connect(rando).expireVoucher(exchange.id)).to.revertedWith(
             RevertReasons.VOUCHER_STILL_VALID
           );
+
+          // Set time forward past the last valid timestamp
+          await setNextBlockTimestamp(Number(voucherRedeemableFrom) + Number(voucherValid));
+
+          // Attempt to cancel the voucher, expecting revert
+          await expect(exchangeHandler.connect(rando).expireVoucher(exchange.id)).to.revertedWith(
+            RevertReasons.VOUCHER_STILL_VALID
+          );
         });
       });
     });
@@ -2013,6 +2021,17 @@ describe("IBosonExchangeHandler", function () {
         assert.equal(response, ExchangeState.Redeemed, "Exchange state is incorrect");
       });
 
+      it("It's possible to redeem at the the end of voucher validity period", async function () {
+        // Set time forward to the offer's validUntilDate
+        await setNextBlockTimestamp(Number(voucherRedeemableFrom) + Number(voucherValid));
+
+        // Redeem the voucher, expecting event
+        await expect(exchangeHandler.connect(buyer).redeemVoucher(exchange.id)).to.emit(
+          exchangeHandler,
+          "VoucherRedeemed"
+        );
+      });
+
       context("💔 Revert Reasons", async function () {
         it("The exchanges region of protocol is paused", async function () {
           // Pause the exchanges region of the protocol
@@ -2060,7 +2079,7 @@ describe("IBosonExchangeHandler", function () {
 
         it("current time is after to voucher's validUntilDate", async function () {
           // Set time forward past the voucher's validUntilDate
-          await setNextBlockTimestamp(Number(voucherRedeemableFrom) + Number(voucherValid) + Number(oneWeek));
+          await setNextBlockTimestamp(Number(voucherRedeemableFrom) + Number(voucherValid) + 1);
 
           // Attempt to redeem the voucher, expecting revert
           await expect(exchangeHandler.connect(buyer).redeemVoucher(exchange.id)).to.revertedWith(
@@ -3581,7 +3600,7 @@ describe("IBosonExchangeHandler", function () {
           block = await ethers.provider.getBlock(blockNumber);
           const escalatedDate = block.timestamp.toString();
 
-          await setNextBlockTimestamp(Number(escalatedDate) + Number(disputeResolver.escalationResponsePeriod));
+          await setNextBlockTimestamp(Number(escalatedDate) + Number(disputeResolver.escalationResponsePeriod) + 1);
 
           // Expire dispute
           await disputeHandler.connect(rando).expireEscalatedDispute(exchange.id);

diff --git a/test/protocol/FundsHandlerTest.js b/test/protocol/FundsHandlerTest.js
@@ -2823,7 +2823,7 @@ describe("IBosonFundsHandler", function () {
             // protocol: protocolFee
             protocolPayoff = offerTokenProtocolFee;
 
-            await setNextBlockTimestamp(Number(timeout));
+            await setNextBlockTimestamp(Number(timeout) + 1);
           });
 
           it("should emit a FundsReleased event", async function () {
@@ -2936,7 +2936,7 @@ describe("IBosonFundsHandler", function () {
               disputedDate = block.timestamp.toString();
               timeout = ethers.BigNumber.from(disputedDate).add(resolutionPeriod).toString();
 
-              await setNextBlockTimestamp(Number(timeout));
+              await setNextBlockTimestamp(Number(timeout) + 1);
             });
 
             it("should emit a FundsReleased event", async function () {
@@ -3825,7 +3825,7 @@ describe("IBosonFundsHandler", function () {
               block = await ethers.provider.getBlock(blockNumber);
               escalatedDate = block.timestamp.toString();
 
-              await setNextBlockTimestamp(Number(escalatedDate) + Number(disputeResolver.escalationResponsePeriod));
+              await setNextBlockTimestamp(Number(escalatedDate) + Number(disputeResolver.escalationResponsePeriod) + 1);
             });
 
             it("should emit a FundsReleased event", async function () {
@@ -3927,7 +3927,9 @@ describe("IBosonFundsHandler", function () {
                 block = await ethers.provider.getBlock(blockNumber);
                 escalatedDate = block.timestamp.toString();
 
-                await setNextBlockTimestamp(Number(escalatedDate) + Number(disputeResolver.escalationResponsePeriod));
+                await setNextBlockTimestamp(
+                  Number(escalatedDate) + Number(disputeResolver.escalationResponsePeriod) + 1
+                );
               });
 
               it("should update state", async function () {

diff --git a/test/protocol/clients/BosonVoucherTest.js b/test/protocol/clients/BosonVoucherTest.js
@@ -1071,8 +1071,8 @@ describe("IBosonVoucher", function () {
     });
 
     it("Should be 0 if offer is expired", async function () {
-      // Skip to after offer expiry
-      await setNextBlockTimestamp(ethers.BigNumber.from(offerDates.validUntil).add(1).toHexString());
+      // Skip to offer expiry
+      await setNextBlockTimestamp(ethers.BigNumber.from(offerDates.validUntil).toHexString(), true);
 
       // Get available premints from contract
       let availablePremints = await bosonVoucher.getAvailablePreMints(offerId);

diff --git a/test/util/utils.js b/test/util/utils.js
@@ -117,11 +117,13 @@ function compareOfferStructs(returnedOffer) {
   return true;
 }
 
-async function setNextBlockTimestamp(timestamp) {
+async function setNextBlockTimestamp(timestamp, mine = false) {
   if (typeof timestamp == "string" && timestamp.startsWith("0x0") && timestamp.length > 3)
     timestamp = "0x" + timestamp.substring(3);
   await provider.send("evm_setNextBlockTimestamp", [timestamp]);
-  await provider.send("evm_mine", []);
+
+  // when testing static call, a block must be mined to get the correct timestamp
+  if (mine) await provider.send("evm_mine", []);
 }
 
 function getSignatureParameters(signature) {