-
Notifications
You must be signed in to change notification settings - Fork 80
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
compose: add vouch to compose #1814
Conversation
Codecov ReportBase: 55.20% // Head: 55.14% // Decreases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## main #1814 +/- ##
==========================================
- Coverage 55.20% 55.14% -0.06%
==========================================
Files 169 170 +1
Lines 22140 22293 +153
==========================================
+ Hits 12222 12294 +72
- Misses 8355 8419 +64
- Partials 1563 1580 +17
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
84f683d
to
e2e1368
Compare
@@ -173,9 +209,33 @@ func (p *headProducer) handleGetBlockRoot(w http.ResponseWriter, r *http.Request | |||
|
|||
// handleEvents is a http handler to handle "/eth/v1/events". | |||
func (p *headProducer) handleEvents(w http.ResponseWriter, r *http.Request) { |
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.
should probably add a test for this headProducer, it is pretty complex...
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.
Done!
Adds vouch to compose and also made some changes to beaconmock which were needed by vouch specifically.
Note: Vouch doesn't support block proposals because of RANDAO check in go-eth2-client.
category: feature
ticket: #1403