app: wire and test simnet core workflow #206
Conversation
Codecov Report
@@ Coverage Diff @@
## main #206 +/- ##
=======================================
Coverage ? 51.32%
=======================================
Files ? 48
Lines ? 3236
Branches ? 0
=======================================
Hits ? 1661
Misses ? 1369
Partials ? 206 Continue to review full report at Codecov.
|
corverroos
force-pushed
the
corver/simnettest
branch
from
133d766
to
5f22e9b
Compare
tbls/tss.go
Outdated
| // where Verifiers(coefficients of public polynomial) are used to compute sum of products of public polynomial with | ||
| // identifier as x coordinate. | ||
| // TODO(corver): Remove this once kryptology concurrency issues have been addressed. | ||
| var mu sync.Mutex |
There was a problem hiding this comment.
this can be defined above blsScheme
| @@ -116,9 +117,15 @@ func Attest(ctx context.Context, eth2Cl Eth2AttProvider, signFunc SignFunc, | |||
| return eth2Cl.SubmitAttestations(ctx, atts) | |||
| } | |||
|
|
|||
| // TODO(corver): Remove this once kryptology concurrency issues have been addressed. | |||
| var mu sync.Mutex | |||
| @@ -116,9 +117,15 @@ func Attest(ctx context.Context, eth2Cl Eth2AttProvider, signFunc SignFunc, | |||
| return eth2Cl.SubmitAttestations(ctx, atts) | |||
| } | |||
|
|
|||
| // TODO(corver): Remove this once kryptology concurrency issues have been addressed. | |||
There was a problem hiding this comment.
kryptology concurrency issues are resolved but still there's one small issue that is blocking our migration to coinbase/kryptology
There was a problem hiding this comment.
cool, can remove when reverted to kryptology
Wires the simnet core workflow using beaconmock and validaormock as well as a test that 2 slots are attested.
category: feature
ticket: #203