Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Invalid parent hash test #3927

Merged
merged 1 commit into from
Apr 3, 2022
Merged

Invalid parent hash test #3927

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
47 changes: 47 additions & 0 deletions src/Nethermind/Nethermind.Merge.Plugin.Test/EngineModuleTests.V1.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1075,6 +1075,53 @@ private async Task<BlockRequestResult> BuildAndGetPayloadOnBranch(
return getPayloadResult.Data!;
}

[Test]
public async Task payloadV1_invalid_parent_hash()
{
using MergeTestBlockchain chain =
await CreateBlockChain(new MergeConfig() {Enabled = true, TerminalTotalDifficulty = "0"});
IEngineRpcModule rpc = CreateEngineModule(chain);

// Correct new payload
BlockRequestResult blockRequestResult1 = CreateBlockRequest(
CreateParentBlockRequestOnHead(chain.BlockTree),
TestItem.AddressA);
var newPayloadResult1 = await rpc.engine_newPayloadV1(blockRequestResult1);
newPayloadResult1.Data.Status.Should().Be(PayloadStatus.Valid);

// Fork choice updated with first np hash
var forkChoiceState1 = new ForkchoiceStateV1(blockRequestResult1.BlockHash, blockRequestResult1.BlockHash,
blockRequestResult1.BlockHash);
var forkchoiceUpdatedResult1 = await rpc.engine_forkchoiceUpdatedV1(forkChoiceState1);
forkchoiceUpdatedResult1.Data.PayloadStatus.Status.Should().Be(PayloadStatus.Valid);

// New payload unknown parent hash
BlockRequestResult blockRequestResult2A = CreateBlockRequest(blockRequestResult1, TestItem.AddressA);
blockRequestResult2A.ParentHash.Bytes[0] = (byte)(255 - blockRequestResult2A.ParentHash.Bytes[0]);
TryCalculateHash(blockRequestResult2A, out Keccak? hash);
blockRequestResult2A.BlockHash = hash;
var newPayloadResult2A = await rpc.engine_newPayloadV1(blockRequestResult2A);
newPayloadResult2A.Data.Status.Should().Be(PayloadStatus.Accepted);

// Fork choice updated with unknown parent hash
var forkChoiceState2A = new ForkchoiceStateV1(blockRequestResult2A.BlockHash,
blockRequestResult2A.BlockHash,
blockRequestResult2A.BlockHash);
var forkchoiceUpdatedResult2A = await rpc.engine_forkchoiceUpdatedV1(forkChoiceState2A);
forkchoiceUpdatedResult2A.Data.PayloadStatus.Status.Should().Be(PayloadStatus.Syncing);

// New payload with correct parent hash
BlockRequestResult blockRequestResult2B = CreateBlockRequest(blockRequestResult1, TestItem.AddressA);
var newPayloadResult2B = await rpc.engine_newPayloadV1(blockRequestResult2B);
newPayloadResult2B.Data.Status.Should().Be(PayloadStatus.Valid);

// Fork choice updated with correct parent hash
var forkChoiceState2B = new ForkchoiceStateV1(blockRequestResult2B.BlockHash, blockRequestResult2B.BlockHash,
blockRequestResult2B.BlockHash);
var forkchoiceUpdatedResult2B = await rpc.engine_forkchoiceUpdatedV1(forkChoiceState2B);
forkchoiceUpdatedResult2B.Data.PayloadStatus.Status.Should().Be(PayloadStatus.Valid);
}

private async Task<BlockRequestResult> BuildAndGetPayloadResult(
IEngineRpcModule rpc, MergeTestBlockchain chain, Keccak headBlockHash, Keccak finalizedBlockHash,
Keccak safeBlockHash,
Expand Down