-
-
Notifications
You must be signed in to change notification settings - Fork 287
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
fix: skip useWorker mode for gossipsub e2e tests #6368
fix: skip useWorker mode for gossipsub e2e tests #6368
Conversation
Performance Report✔️ no performance regression detected Full benchmark results
|
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## unstable #6368 +/- ##
=========================================
Coverage 60.14% 60.14%
=========================================
Files 407 407
Lines 46490 46490
Branches 1545 1545
=========================================
Hits 27960 27960
Misses 18498 18498
Partials 32 32 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀 although I would guess this gives @nazarhussain heartburn from having skipped tests pile up in the eslint output. Am curious if we should discuss how to handle stuff like this long term. Opened a discussion #6376
🎉 This PR is included in v1.16.0 🎉 |
Motivation
I see a lot of failed gossipsub e2e tests since we switched to vitest, and it always happen with
useWorker=true
Description
Skipping tests with
useWorker=true
in CI, while we leave that for local tests if we find any issues with it in the futureuseWorker
flagcc @wemeetagain
part of #6358