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

fix(tests)_: prevent crash when creating wakuv2 from multiple goroutines #5902

Merged
merged 1 commit into from
Oct 1, 2024

Conversation

osmaczko
Copy link
Contributor

@osmaczko osmaczko commented Oct 1, 2024

Creating wakuv2 from multiple goroutines results in a race condition where multiple in-memory databases with the same name are attempted to be opened

Creating wakuv2 from multiple goroutines results in a race condition
where multiple in-memory databases with the same name are attempted to
be opened.
@osmaczko osmaczko requested review from igor-sirotin and a team October 1, 2024 14:36
@status-im-auto
Copy link
Member

status-im-auto commented Oct 1, 2024

Jenkins Builds

Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 399f588 #1 2024-10-01 14:40:20 ~3 min tests-rpc 📄log
✔️ 399f588 #1 2024-10-01 14:40:42 ~3 min linux 📦zip
✔️ 399f588 #1 2024-10-01 14:41:18 ~4 min ios 📦zip
✔️ 399f588 #1 2024-10-01 14:41:43 ~5 min android 📦aar
✔️ 399f588 #1 2024-10-01 15:09:40 ~32 min tests 📄log

@codecov-commenter
Copy link

codecov-commenter commented Oct 1, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Please upload report for BASE (develop@107e2cb). Learn more about missing BASE report.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             develop    #5902   +/-   ##
==========================================
  Coverage           ?   46.10%           
==========================================
  Files              ?      891           
  Lines              ?   158064           
  Branches           ?        0           
==========================================
  Hits               ?    72882           
  Misses             ?    76822           
  Partials           ?     8360           
Flag Coverage Δ
functional 11.85% <ø> (?)
unit 45.51% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Copy link
Collaborator

@igor-sirotin igor-sirotin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

noice, you found it! 💪

@osmaczko osmaczko merged commit 2c0178c into develop Oct 1, 2024
12 checks passed
@osmaczko osmaczko deleted the fix/crash-TestFetchRealCommunity branch October 1, 2024 18:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants