From 2c68bfcfc650c5d93dc822548bf1d5ca7f9d1e2a Mon Sep 17 00:00:00 2001
From: Anna Shaleva <shaleva.ann@nspcc.ru>
Date: Thu, 18 Mar 2021 20:28:26 +0300
Subject: [PATCH] consensus: remove nonce from PrepareRequest

---
 pkg/consensus/payload_test.go          | 1 -
 pkg/consensus/prepare_request.go       | 7 ++-----
 pkg/consensus/prepare_request_test.go  | 5 -----
 pkg/consensus/recovery_message_test.go | 1 -
 4 files changed, 2 insertions(+), 12 deletions(-)

diff --git a/pkg/consensus/payload_test.go b/pkg/consensus/payload_test.go
index 2aad8dab5..2b24c9985 100644
--- a/pkg/consensus/payload_test.go
+++ b/pkg/consensus/payload_test.go
@@ -201,7 +201,6 @@ func randomPrepareRequest(t *testing.T) *prepareRequest {
 
 	req := &prepareRequest{
 		timestamp:         rand.Uint64(),
-		nonce:             rand.Uint64(),
 		transactionHashes: make([]util.Uint256, txCount),
 	}
 
diff --git a/pkg/consensus/prepare_request.go b/pkg/consensus/prepare_request.go
index 4cbc61f04..3ce5d2f77 100644
--- a/pkg/consensus/prepare_request.go
+++ b/pkg/consensus/prepare_request.go
@@ -12,7 +12,6 @@ type prepareRequest struct {
 	version           uint32
 	prevHash          util.Uint256
 	timestamp         uint64
-	nonce             uint64
 	transactionHashes []util.Uint256
 	stateRootEnabled  bool
 	stateRoot         util.Uint256
@@ -25,7 +24,6 @@ func (p *prepareRequest) EncodeBinary(w *io.BinWriter) {
 	w.WriteU32LE(p.version)
 	w.WriteBytes(p.prevHash[:])
 	w.WriteU64LE(p.timestamp)
-	w.WriteU64LE(p.nonce)
 	w.WriteArray(p.transactionHashes)
 	if p.stateRootEnabled {
 		w.WriteBytes(p.stateRoot[:])
@@ -37,7 +35,6 @@ func (p *prepareRequest) DecodeBinary(r *io.BinReader) {
 	p.version = r.ReadU32LE()
 	r.ReadBytes(p.prevHash[:])
 	p.timestamp = r.ReadU64LE()
-	p.nonce = r.ReadU64LE()
 	r.ReadArray(&p.transactionHashes, block.MaxTransactionsPerBlock)
 	if p.stateRootEnabled {
 		r.ReadBytes(p.stateRoot[:])
@@ -71,10 +68,10 @@ func (p *prepareRequest) Timestamp() uint64 { return p.timestamp * nsInMs }
 func (p *prepareRequest) SetTimestamp(ts uint64) { p.timestamp = ts / nsInMs }
 
 // Nonce implements payload.PrepareRequest interface.
-func (p *prepareRequest) Nonce() uint64 { return p.nonce }
+func (p *prepareRequest) Nonce() uint64 { return 0 }
 
 // SetNonce implements payload.PrepareRequest interface.
-func (p *prepareRequest) SetNonce(nonce uint64) { p.nonce = nonce }
+func (p *prepareRequest) SetNonce(nonce uint64) {}
 
 // TransactionHashes implements payload.PrepareRequest interface.
 func (p *prepareRequest) TransactionHashes() []util.Uint256 { return p.transactionHashes }
diff --git a/pkg/consensus/prepare_request_test.go b/pkg/consensus/prepare_request_test.go
index f40763c02..8a8a4e218 100644
--- a/pkg/consensus/prepare_request_test.go
+++ b/pkg/consensus/prepare_request_test.go
@@ -24,9 +24,6 @@ func TestPrepareRequest_Setters(t *testing.T) {
 	p.SetNextConsensus(util.Uint160{5, 6, 7})
 	require.Equal(t, util.Uint160{}, p.NextConsensus())
 
-	p.SetNonce(8765)
-	require.EqualValues(t, 8765, p.Nonce())
-
 	hashes := [2]util.Uint256{random.Uint256(), random.Uint256()}
 
 	p.SetTransactionHashes(hashes[:])
@@ -37,7 +34,6 @@ func TestPrepareRequest_EncodeDecodeBinary(t *testing.T) {
 	t.Run("positive", func(t *testing.T) {
 		expected := &prepareRequest{
 			timestamp: 112,
-			nonce:     1325,
 			transactionHashes: []util.Uint256{
 				random.Uint256(),
 				random.Uint256(),
@@ -53,7 +49,6 @@ func TestPrepareRequest_EncodeDecodeBinary(t *testing.T) {
 		}
 		expected := &prepareRequest{
 			timestamp:         112,
-			nonce:             1325,
 			transactionHashes: hashes,
 		}
 		data, err := testserdes.EncodeBinary(expected)
diff --git a/pkg/consensus/recovery_message_test.go b/pkg/consensus/recovery_message_test.go
index d70f0bd97..3af23db91 100644
--- a/pkg/consensus/recovery_message_test.go
+++ b/pkg/consensus/recovery_message_test.go
@@ -41,7 +41,6 @@ func testRecoveryMessageSetters(t *testing.T, enableStateRoot bool) {
 
 	req := &prepareRequest{
 		timestamp:         87,
-		nonce:             321,
 		transactionHashes: []util.Uint256{{1}},
 		stateRootEnabled:  enableStateRoot,
 	}