Skip to content
New issue

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

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

Already on GitHub? # to your account

feat: nwaku (test) #6395

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

gabrielmer
Copy link
Contributor

A short summary which serves as a squashed-commit message.

A description to understand introduced changes without reading the code.

Important changes:

  • Something worth noting for reviewers.

Closes #

Copy link

github-actions bot commented Mar 6, 2025

We require commits to follow the Conventional Commits, but with _ for non-breaking changes.
Please fix these commit messages:

fix nwaku compilation errors

@gabrielmer gabrielmer self-assigned this Mar 6, 2025
@status-im-auto
Copy link
Member

status-im-auto commented Mar 6, 2025

Jenkins Builds

Click to see older builds (102)
Commit #️⃣ Finished (UTC) Duration Platform Result
✔️ 384dbdf #1 2025-03-06 16:29:50 ~2 min ios 📦zip
✔️ 384dbdf #1 2025-03-06 16:30:18 ~3 min android 📦aar
✔️ 384dbdf #1 2025-03-06 16:31:10 ~3 min macos 📦zip
✖️ 384dbdf #1 2025-03-06 16:31:20 ~4 min tests 📄log
✔️ 384dbdf #1 2025-03-06 16:32:29 ~5 min macos 📦zip
✔️ 384dbdf #1 2025-03-06 16:33:20 ~6 min windows 📦zip
✔️ 384dbdf #1 2025-03-06 16:33:21 ~6 min linux 📦zip
✔️ 384dbdf #1 2025-03-06 16:42:27 ~15 min tests-rpc 📄log
✔️ 64c0968 #2 2025-03-06 16:37:17 ~2 min ios 📦zip
✖️ 64c0968 #2 2025-03-06 16:37:27 ~2 min tests 📄log
✔️ 64c0968 #2 2025-03-06 16:37:53 ~3 min android 📦aar
✔️ 64c0968 #2 2025-03-06 16:38:40 ~3 min macos 📦zip
✔️ 64c0968 #2 2025-03-06 16:39:01 ~4 min windows 📦zip
✔️ 64c0968 #2 2025-03-06 16:39:54 ~5 min macos 📦zip
✔️ 64c0968 #2 2025-03-06 16:40:35 ~5 min linux 📦zip
✔️ 64c0968 #2 2025-03-06 16:57:29 ~14 min tests-rpc 📄log
✔️ f0e3890 #3 2025-03-06 16:47:56 ~2 min ios 📦zip
✔️ f0e3890 #3 2025-03-06 16:48:26 ~3 min android 📦aar
✖️ f0e3890 #3 2025-03-06 16:48:33 ~3 min tests 📄log
✔️ f0e3890 #3 2025-03-06 16:49:18 ~4 min macos 📦zip
✔️ f0e3890 #3 2025-03-06 16:49:35 ~4 min windows 📦zip
✔️ f0e3890 #3 2025-03-06 16:50:30 ~5 min macos 📦zip
✔️ f0e3890 #3 2025-03-06 16:51:18 ~6 min linux 📦zip
✔️ 4df7d81 #4 2025-03-06 16:56:01 ~2 min ios 📦zip
✖️ 4df7d81 #4 2025-03-06 16:56:10 ~2 min tests 📄log
✔️ 4df7d81 #4 2025-03-06 16:56:22 ~3 min android 📦aar
✔️ 4df7d81 #4 2025-03-06 16:57:14 ~3 min macos 📦zip
✔️ 4df7d81 #4 2025-03-06 16:57:39 ~4 min windows 📦zip
✔️ 4df7d81 #4 2025-03-06 16:58:33 ~5 min macos 📦zip
✔️ 4df7d81 #4 2025-03-06 16:59:32 ~6 min linux 📦zip
✔️ 4df7d81 #3 2025-03-06 17:12:37 ~15 min tests-rpc 📄log
✔️ 33355b3 #5 2025-03-06 17:16:04 ~2 min ios 📦zip
✖️ 33355b3 #5 2025-03-06 17:16:29 ~2 min tests 📄log
✔️ 33355b3 #5 2025-03-06 17:16:41 ~3 min android 📦aar
✔️ 33355b3 #5 2025-03-06 17:18:02 ~4 min windows 📦zip
✔️ 33355b3 #5 2025-03-06 17:18:45 ~5 min macos 📦zip
✔️ 33355b3 #5 2025-03-06 17:18:57 ~5 min macos 📦zip
✔️ 33355b3 #5 2025-03-06 17:19:31 ~6 min linux 📦zip
✔️ 33355b3 #4 2025-03-06 17:28:28 ~15 min tests-rpc 📄log
✔️ f87c5e2 #6 2025-03-06 17:33:51 ~2 min ios 📦zip
✔️ f87c5e2 #6 2025-03-06 17:34:30 ~3 min android 📦aar
✔️ f87c5e2 #6 2025-03-06 17:35:13 ~4 min macos 📦zip
✔️ f87c5e2 #6 2025-03-06 17:35:36 ~4 min windows 📦zip
✔️ f87c5e2 #6 2025-03-06 17:36:29 ~5 min macos 📦zip
✔️ f87c5e2 #6 2025-03-06 17:37:23 ~6 min linux 📦zip
✔️ f87c5e2 #5 2025-03-06 17:46:19 ~15 min tests-rpc 📄log
✖️ f87c5e2 #6 2025-03-06 18:00:02 ~28 min tests 📄log
✔️ e4069de #7 2025-03-07 11:05:08 ~3 min android 📦aar
✔️ e4069de #7 2025-03-07 11:05:25 ~3 min ios 📦zip
✔️ e4069de #7 2025-03-07 11:06:08 ~3 min macos 📦zip
✔️ e4069de #7 2025-03-07 11:06:41 ~4 min windows 📦zip
✔️ e4069de #7 2025-03-07 11:07:27 ~5 min macos 📦zip
✔️ e4069de #7 2025-03-07 11:07:35 ~5 min linux 📦zip
✔️ e4069de #6 2025-03-07 11:17:11 ~14 min tests-rpc 📄log
✖️ e4069de #7 2025-03-07 11:34:03 ~31 min tests 📄log
✔️ de2aa9e #8 2025-03-07 16:43:04 ~2 min ios 📦zip
✔️ de2aa9e #8 2025-03-07 16:43:35 ~3 min android 📦aar
✔️ de2aa9e #8 2025-03-07 16:44:15 ~3 min macos 📦zip
✔️ de2aa9e #8 2025-03-07 16:44:30 ~4 min windows 📦zip
✔️ de2aa9e #8 2025-03-07 16:45:36 ~5 min macos 📦zip
✔️ de2aa9e #8 2025-03-07 16:46:27 ~6 min linux 📦zip
✔️ de2aa9e #7 2025-03-07 16:55:05 ~14 min tests-rpc 📄log
✖️ de2aa9e #8 2025-03-07 17:11:01 ~30 min tests 📄log
✔️ 6042149 #9 2025-03-10 10:53:23 ~2 min ios 📦zip
✔️ 6042149 #9 2025-03-10 10:53:53 ~3 min android 📦aar
✔️ 6042149 #9 2025-03-10 10:54:38 ~3 min macos 📦zip
✔️ 6042149 #9 2025-03-10 10:55:04 ~4 min windows 📦zip
✔️ 6042149 #9 2025-03-10 10:56:29 ~5 min macos 📦zip
✔️ 6042149 #9 2025-03-10 10:56:44 ~6 min linux 📦zip
✔️ 6042149 #8 2025-03-10 11:06:33 ~15 min tests-rpc 📄log
✖️ 6042149 #9 2025-03-10 11:22:10 ~31 min tests 📄log
✔️ 608ecd8 #10 2025-03-10 10:58:37 ~2 min ios 📦zip
✔️ 608ecd8 #10 2025-03-10 10:58:55 ~2 min android 📦aar
✔️ 608ecd8 #10 2025-03-10 10:59:57 ~3 min macos 📦zip
✔️ 608ecd8 #10 2025-03-10 11:00:16 ~4 min windows 📦zip
✔️ 608ecd8 #10 2025-03-10 11:02:37 ~5 min macos 📦zip
✔️ 608ecd8 #10 2025-03-10 11:03:06 ~6 min linux 📦zip
✔️ 608ecd8 #9 2025-03-10 11:21:39 ~14 min tests-rpc 📄log
✖️ 608ecd8 #10 2025-03-10 11:54:00 ~31 min tests 📄log
✔️ 608ecd8 #11 2025-03-10 12:36:51 ~30 min tests 📄log
✔️ a2e026b #11 2025-03-10 12:53:48 ~2 min ios 📦zip
✔️ a2e026b #11 2025-03-10 12:54:23 ~3 min android 📦aar
✔️ a2e026b #11 2025-03-10 12:55:07 ~3 min macos 📦zip
✔️ a2e026b #11 2025-03-10 12:55:31 ~4 min windows 📦zip
✔️ a2e026b #11 2025-03-10 12:56:25 ~5 min macos 📦zip
✔️ a2e026b #11 2025-03-10 12:57:30 ~6 min linux 📦zip
✔️ a2e026b #10 2025-03-10 13:08:02 ~16 min tests-rpc 📄log
✔️ a2e026b #12 2025-03-10 13:21:40 ~30 min tests 📄log
✔️ 540f374 #12 2025-03-10 13:01:18 ~2 min ios 📦zip
✔️ 540f374 #12 2025-03-10 13:01:56 ~3 min android 📦aar
✔️ 540f374 #12 2025-03-10 13:02:41 ~3 min macos 📦zip
✔️ 540f374 #12 2025-03-10 13:03:03 ~4 min windows 📦zip
✔️ 540f374 #12 2025-03-10 13:03:57 ~5 min macos 📦zip
✔️ 540f374 #12 2025-03-10 13:05:13 ~6 min linux 📦zip
✔️ 2e0a0fe #13 2025-03-10 13:04:06 ~2 min ios 📦zip
✔️ 2e0a0fe #13 2025-03-10 13:05:03 ~3 min android 📦aar
✔️ 2e0a0fe #13 2025-03-10 13:07:10 ~4 min macos 📦zip
✔️ 2e0a0fe #13 2025-03-10 13:07:34 ~4 min windows 📦zip
✔️ 2e0a0fe #13 2025-03-10 13:10:19 ~6 min macos 📦zip
✔️ 2e0a0fe #13 2025-03-10 13:10:40 ~5 min linux 📦zip
✔️ 2e0a0fe #11 2025-03-10 13:23:30 ~15 min tests-rpc 📄log
✔️ 2e0a0fe #13 2025-03-10 13:53:15 ~31 min tests 📄log
Commit #️⃣ Finished (UTC) Duration Platform Result
✖️ 66e0050 #14 2025-03-12 13:02:23 ~1 min tests 📄log
✔️ 66e0050 #14 2025-03-12 13:03:38 ~2 min ios 📦zip
✔️ 66e0050 #14 2025-03-12 13:04:12 ~3 min android 📦aar
✔️ 66e0050 #14 2025-03-12 13:05:03 ~4 min macos 📦zip
✔️ 66e0050 #14 2025-03-12 13:05:26 ~4 min windows 📦zip
✔️ 66e0050 #14 2025-03-12 13:06:50 ~5 min macos 📦zip
✔️ 66e0050 #14 2025-03-12 13:08:34 ~7 min linux 📦zip
✔️ 66e0050 #12 2025-03-12 13:20:44 ~19 min tests-rpc 📄log
✖️ 0275eb4 #15 2025-03-12 14:59:08 ~1 min tests 📄log
✔️ 0275eb4 #15 2025-03-12 15:00:17 ~2 min ios 📦zip
✔️ 0275eb4 #15 2025-03-12 15:00:41 ~3 min android 📦aar
✔️ 0275eb4 #15 2025-03-12 15:01:36 ~3 min macos 📦zip
✔️ 0275eb4 #15 2025-03-12 15:02:21 ~4 min windows 📦zip
✔️ 0275eb4 #15 2025-03-12 15:02:56 ~5 min macos 📦zip
✔️ 0275eb4 #15 2025-03-12 15:03:36 ~5 min linux 📦zip
✔️ 0275eb4 #13 2025-03-12 15:14:27 ~16 min tests-rpc 📄log

@gabrielmer gabrielmer force-pushed the feat-integrating-waku-go-bindings-rebased branch from de2aa9e to 6042149 Compare March 10, 2025 10:50
@igor-sirotin
Copy link
Collaborator

@gabrielmer can you please extract the ./vendor updates into a separate PR, just to make this PR lighter and simpler for review?

@gabrielmer
Copy link
Contributor Author

@gabrielmer can you please extract the ./vendor updates into a separate PR, just to make this PR lighter and simpler for review?

yeah for sure! so adding the vendors in a separate PR so when merged they don't appear in the diff? I assume that if I just remove the changes of the vendors from this PR it won't compile, although I might be missing something

@gabrielmer
Copy link
Contributor Author

gabrielmer commented Mar 10, 2025

Although now that I think, the go-waku changes need this PR, otherwise it won't compile either

Copy link

codecov bot commented Mar 10, 2025

Codecov Report

Attention: Patch coverage is 23.73737% with 151 lines in your changes missing coverage. Please review.

Project coverage is 21.45%. Comparing base (9e769ce) to head (0275eb4).

Files with missing lines Patch % Lines
wakuv2/nwaku_test_utils.go 0.00% 36 Missing ⚠️
wakuv2/common/envelope.go 0.00% 29 Missing ⚠️
timesource/timesource.go 0.00% 21 Missing ⚠️
protocol/messenger_communities.go 6.66% 14 Missing ⚠️
protocol/messenger_mailserver.go 48.14% 13 Missing and 1 partial ⚠️
protocol/communities/manager.go 0.00% 8 Missing ⚠️
wakuv2/gowaku.go 58.33% 5 Missing ⚠️
protocol/communities/community.go 0.00% 3 Missing ⚠️
protocol/communities/persistence.go 0.00% 3 Missing ⚠️
wakuv2/config.go 0.00% 3 Missing ⚠️
... and 10 more

❌ Your patch check has failed because the patch coverage (23.73%) is below the target coverage (50.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #6395      +/-   ##
===========================================
- Coverage    23.84%   21.45%   -2.39%     
===========================================
  Files          801      802       +1     
  Lines       104698   128378   +23680     
===========================================
+ Hits         24961    27540    +2579     
- Misses       76284    97531   +21247     
+ Partials      3453     3307     -146     
Flag Coverage Δ
functional 21.45% <23.73%> (-2.39%) ⬇️

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

Files with missing lines Coverage Δ
node/status_node_services.go 72.13% <100.00%> (-1.34%) ⬇️
protocol/messenger.go 26.02% <100.00%> (-3.14%) ⬇️
protocol/messenger_mailserver_cycle.go 54.81% <ø> (+0.57%) ⬆️
protocol/messenger_peers.go 10.00% <ø> (+0.90%) ⬆️
protocol/messenger_store_node_request_manager.go 53.16% <100.00%> (-17.27%) ⬇️
signal/events_shhext.go 28.78% <100.00%> (-12.88%) ⬇️
waku/types/waku.go 66.66% <ø> (-6.67%) ⬇️
wakuv2/common/message.go 88.63% <100.00%> (-1.78%) ⬇️
wakuv2/history_processor_wrapper.go 66.66% <100.00%> (-33.34%) ⬇️
wakuv2/message_publishing.go 56.16% <ø> (+3.85%) ⬆️
... and 21 more

... and 698 files with indirect coverage changes

@gabrielmer gabrielmer force-pushed the feat-integrating-waku-go-bindings-rebased branch from 2e0a0fe to 66e0050 Compare March 12, 2025 13:00
richard-ramos and others added 2 commits March 12, 2025 16:56
adding waku-go-bindings and some changes to nwaku.go

cleanup

fix compilation errors

updating makefile

updating type

always building libwaku

always build libwaku

remove nwaku submodule

Revert "remove nwaku submodule"

This reverts commit e68749b.

removing submodule

removing libwaku auto libwaku build

updating vendors

workaround for using pb types

adding nwaku_utils.go

fixing store requests

converting store responses

converting from envelope types on received message

improving store types conversion

fixing cursor conversion

fixing crash

first rebase attempt

improvements

more fixes

adding unimplemented methods + fixes

fixing compilatioj

fix crash

fix gowaku error

fixing store message parsing

updating waku-go-bindings

fixing waku-go-bindings integration

updaring gitignore

fixing gowaku compilation

Revert "updating vendors"

This reverts commit 64edef5.

adding missing vendor file

fix for bindings vendor

temporarily fixing test target

fixing statusgo-library target

fixing status-cli

updating waku-go-bindings

improving compilation for non-nwaku cases

temporarily moving to waku-go-bindings version that allows go compilation

updating waku-go-bindings

fixing compilation

chore: updating waku-go-bindings and improving makefile

fixing makefile
@gabrielmer gabrielmer force-pushed the feat-integrating-waku-go-bindings-rebased branch from 66e0050 to 0275eb4 Compare March 12, 2025 14:57
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants