Skip to content

Conversation

edgurgel
Copy link
Member

What kind of change does this PR introduce?

  • Run more tests in async mode
  • Use less Process.sleep whenever possible

What is the current behavior?

Some time waiting for tests. ~ 300 seconds before my changes

What is the new behavior?

Less time waiting for tests. Finished in 230.6 seconds (11.5s async, 219.0s sync) after my changes

Only change I made on lib/* was to help with tests assertion so not really any feature changes

Additional context

Might have one or two flaky tests but I'm sure I can solve them

* Run more tests in async mode
* Use less Process.sleep whenever possible
Copy link

vercel bot commented May 19, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
realtime-demo ⬜️ Ignored (Inspect) Visit Preview May 19, 2025 10:41am

@coveralls
Copy link

coveralls commented May 19, 2025

Coverage Status

coverage: 83.443% (+1.0%) from 82.469%
when pulling 2bdb787 on chore/faster-tests-part-1
into 386aeb3 on main.

@edgurgel edgurgel merged commit e27b587 into main May 19, 2025
6 of 7 checks passed
@edgurgel edgurgel deleted the chore/faster-tests-part-1 branch May 19, 2025 21:27
@kiwicopple
Copy link
Member

🎉 This PR is included in version 2.35.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants