Skip to content

Commit cc4ec8a

Browse files
committed
fix: address lint errors after rebase
1 parent 0ef6906 commit cc4ec8a

4 files changed

Lines changed: 9 additions & 10 deletions

File tree

testing/simulated/blob_fetcher_integration_test.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -56,9 +56,10 @@ func (s *SimulatedSuite) TestBlobFetcher_MultiNodeFetch() {
5656
s.InitializeChain(s.T())
5757

5858
// Move chain forward one block
59-
blsSigner := simulated.GetBlsSigner(s.HomeDir)
59+
nodeAddress, err := s.SimComet.GetNodeAddress()
60+
s.Require().NoError(err)
6061
startTime := time.Now()
61-
_, _, _ = s.MoveChainToHeight(s.T(), 1, 1, blsSigner, startTime)
62+
_, _, _ = s.MoveChainToHeight(s.T(), 1, 1, nodeAddress, startTime)
6263

6364
// Create test blobs
6465
testSlot := math.Slot(100)
@@ -125,7 +126,7 @@ func (s *SimulatedSuite) TestBlobFetcher_MultiNodeFetch() {
125126

126127
// Queue blob request, wait for it to be downloaded and validate
127128
s.Require().NoError(node1Fetcher.QueueBlobRequest(block))
128-
time.Sleep(200 * time.Millisecond)
129+
time.Sleep(500 * time.Millisecond)
129130
storedSidecars, err := node1Store.GetBlobSidecars(testSlot)
130131
s.Require().NoError(err)
131132
s.Require().Len(storedSidecars, 2)

testing/simulated/malicious_proposer_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -557,7 +557,7 @@ func testBuildInvalidBlock(
557557
pubkey, err := blsSigner.GetPubKey()
558558
r.NoError(err)
559559

560-
signedBlk, sidecars, err := builder.SimComet.Comet.Blockchain.ParseBeaconBlock(
560+
signedBlk, sidecars, err := builder.SimComet.Comet.Blockchain.ParseProcessProposalRequest(
561561
&types.ProcessProposalRequest{
562562
Txs: PrepReq.Txs,
563563
Height: PrepReq.Height,

testing/simulated/pectra_fork_test.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -221,10 +221,9 @@ func (s *PectraForkSuite) TestTimestampFork_ELAndCLInSync_IsSuccessful() {
221221
// set consensus time for the next block to match
222222
// the timestamp of the payload built optimistically.
223223
forkVersion := s.Geth.TestNode.ChainSpec.ActiveForkVersionForTimestamp(math.U64(consensusTime.Unix())) //#nosec: G115
224-
blk, _, err := encoding.ExtractBlobsAndBlockFromRequest(
225-
processRequest,
224+
blk, err := encoding.UnmarshalBeaconBlockFromABCIRequest(
225+
processRequest.GetTxs(),
226226
blockchain.BeaconBlockTxIndex,
227-
blockchain.BlobSidecarsTxIndex,
228227
forkVersion,
229228
)
230229
s.Require().NoError(err)

testing/simulated/utils.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -256,10 +256,9 @@ func (s *SharedAccessors) MoveChainToHeight(
256256
// set consensus time for the next block to match
257257
// the timestamp of the payload built optimistically.
258258
forkVersion := s.TestNode.ChainSpec.ActiveForkVersionForTimestamp(math.U64(proposalTime.Unix())) //#nosec: G115
259-
blk, _, err := encoding.ExtractBlobsAndBlockFromRequest(
260-
processReq,
259+
blk, err := encoding.UnmarshalBeaconBlockFromABCIRequest(
260+
processReq.GetTxs(),
261261
blockchain.BeaconBlockTxIndex,
262-
blockchain.BlobSidecarsTxIndex,
263262
forkVersion,
264263
)
265264
require.NoError(t, err)

0 commit comments

Comments
 (0)