Add sample Starknet P2P tests utilizing synthetic node and get header request #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Can't use the package directly right now due to dependencies issue in Juno. Refer to: NethermindEth/juno#2084
wojciechos
force-pushed
the
go-p2p-tests
branch
2 times, most recently
from
4144fc1
to
921bbdb
Compare
wojciechos
force-pushed
the
go-p2p-tests
branch
from
921bbdb
to
97d8f27
Compare
derrix060
reviewed
Aug 29, 2024
|
|
||
| message Event { | ||
| Hash transaction_hash = 1; | ||
| Felt252 from_address = 3; // looks like mistake? |
There was a problem hiding this comment.
the fact that it doesn't have a value for 2?
wojciechos
force-pushed
the
go-p2p-tests
branch
from
0b49a74
to
8acd4a0
Compare
wojciechos
force-pushed
the
go-p2p-tests
branch
from
8acd4a0
to
ec2f145
Compare
wojciechos
force-pushed
the
go-p2p-tests
branch
from
51cddb0
to
f735714
Compare
rianhughes
reviewed
Sep 10, 2024
rianhughes
reviewed
Sep 10, 2024
rianhughes
reviewed
Sep 10, 2024
rianhughes
reviewed
Sep 10, 2024
rianhughes
reviewed
Sep 10, 2024
rianhughes
reviewed
Sep 10, 2024
rianhughes
reviewed
Sep 10, 2024
rianhughes
reviewed
Sep 10, 2024
rianhughes
reviewed
Sep 13, 2024
rianhughes
reviewed
Sep 13, 2024
rianhughes
approved these changes
Sep 13, 2024
wojciechos
force-pushed
the
go-p2p-tests
branch
from
a3453c9
to
d3a238e
Compare
* Add p2p discover test * Fix discovery test * Clean up tests * Use go's benchmarking for perf tests * Make perf test framework Discovery test (#3) * Add p2p discover test * Fix discovery test * Clean up tests * Use go's benchmarking for perf tests * dsf * Make perf test framework * use external target node for pipeline run --------- Discovery test (#4) * Add p2p discover test * Fix discovery test * Clean up tests * Use go's benchmarking for perf tests
wojciechos
force-pushed
the
go-p2p-tests
branch
from
d3a238e
to
98503d2
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please ignore the /protocols/ directory. It's copied directly from Juno with minor changes. Temporarly I couldn't import the package because of many not needed, heavy Rust deps. For more details, see: NethermindEth/juno#2084.
PR added demo p2p tests for starkness network, including: