trufflesuite / truffle

:warning: The Truffle Suite is being sunset. For information on ongoing support, migration options and FAQs, visit the Consensys blog. Thank you for all the support over the years.
https://consensys.io/blog/consensys-announces-the-sunset-of-truffle-and-ganache-and-new-hardhat?utm_source=github&utm_medium=referral&utm_campaign=2023_Sep_truffle-sunset-2023_announcement_
MIT License
14.02k stars 2.31k forks source link

Remove the other slow Hardhat test #6114

Closed haltman-at closed 1 year ago

haltman-at commented 1 year ago

Increasing this timeout yet again.

If this isn't enough to stop the problems, we might want to remove this test as well... (or try increasing the Hardhat version like Nick suggested, which might allow us to add the other test back in? I dunno.)

DELETE

gnidan commented 1 year ago

Hm, I think we can probably remove this test as well. These failure conditions should probably just be tested with mocks in the from-hardhat package, rather than booting up all of Hardhat for end-to-end testing. We can probably just do this sort of end-to-end testing for the happy path.

haltman-at commented 1 year ago

Cool, I'll update this PR to remove the test, then!