Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Increase timeout for evm hardhat tests #1827

Merged
merged 1 commit into from
Aug 23, 2024
Merged

Conversation

udpatil
Copy link
Collaborator

@udpatil udpatil commented Aug 21, 2024

Describe your changes and provide context

There has been a high occurrence of EVM integration tests flaking recently due to timeouts being exceeded during tests, this increases timeouts such that we no longer have this flakiness during the first run of integration tests where all of them are running at the same time.

Testing performed to validate your change

CI

Copy link

codecov bot commented Aug 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 60.66%. Comparing base (da9d59c) to head (7535e36).
Report is 3 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1827      +/-   ##
==========================================
- Coverage   60.75%   60.66%   -0.09%     
==========================================
  Files         257      257              
  Lines       22558    22558              
==========================================
- Hits        13705    13685      -20     
- Misses       7883     7905      +22     
+ Partials      970      968       -2     

see 4 files with indirect coverage changes

@udpatil udpatil merged commit 83fd512 into main Aug 23, 2024
46 of 48 checks passed
@udpatil udpatil deleted the increase-hardhat-timeout branch August 23, 2024 13:47
yzang2019 added a commit that referenced this pull request Aug 27, 2024
* main:
  Increase timeout for evm hardhat tests (#1827)
  Fork event manager when creating EVM snapshots (#1831)
  Gas mispricing in cw vm patch (#1824)
  Remove dex actually (#1816)
  Bump seidb to fix receipt missing on restart issue (#1817)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants