OpenZeppelin Test Helpers time

Support
1

In my contract I have a function that is set to be active after a particular block.timestamp has elapsed.
so in my test, I used

describe("do something at time stated", ()=>{

it("revert after time", async()=>{
const stillValid = await contractInstance.showAtTime() // returns timestamp 7 days from now
console.log(stilValid)
await time.increase(stillValid);

await expectRevert(await contractInstance.doAtTime(), "time over")
}

})

this above function returned error Error: Cannot increase current time (6465161954) to a moment in the past (1614832558)
after several tries I discovered on every call of increaseTo(timestamp) moves the current block time ahead.

:computer: Environment

:memo:Details

:1234: Code to reproduce

1 Like
2

Hi @makerz,

Can you share a cut down version of your contract with the showAtTime and doAtTime functions?
I assume that the time is passed in as a parameter to the constructor?

Are you using Truffle’s internal network or are you testing on ganache-cli?

1 Like
3

The time is set like so in the contract after the first deposit:

   contract TimeManager{
            uint256 openPeriod;
            uint256 showAtTime;
            uint256 value;

    function initialize(uint256 _openPeriod) external {
    openPeriod = _openPeriod;
    }

    function deposit(uint256 _value) external {
    value = _value;
    showAtTime= block.timestamp.add(openPerion);`Preformatted text`
    }

    function doAtTime() external {
    require(block.timestamp>= showAtTime,"time over");
    ///other logic...
    }
    }
1 Like
4

Hi @makerz,

In your test you are using increase but using the value of showAtTime, I think you actually want to use increaseTo

You could have a look at the tests in OpenZeppelin Contracts, such as for TokenTimelock

5

Thank you @abcoathup I found a way to run the test. I believe it would be great to have helper functions to capture the timestamp before the test and reset the timestamp to the current timestamp after each “it” block. Thank you for this reference :raised_hands:

1 Like
6

Hi @makerz,

We can’t go backwards in time onchain, so can’t reset the timestamp.

I would recommend passing in the current time (with an offset if required) in a beforeEach in the tests where the contract is created.

1 Like
7

Hi,
It seems that I cannot use any of the time.XXX functions as it keeps on throwing me with this error message.
Error: CONNECTION ERROR: Couldn't connect to node http://localhost:8545
I´ve installed npm install --save-dev @openzeppelin/test-helpers correctly.
Any ideas?