Skip to content

Commit 0564c22

Browse files
committed
Fix issues from rebase
1 parent 72879c5 commit 0564c22

2 files changed

Lines changed: 12 additions & 9 deletions

File tree

src/herder/HerderImpl.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -944,7 +944,7 @@ HerderImpl::recvSCPEnvelope(SCPEnvelope const& envelope)
944944
auto maybeQSet = mApp.getHerder().getQSet(qSetHash);
945945
if (maybeQSet)
946946
{
947-
processSCPQueue();
947+
processSCPQueue(true);
948948
}
949949
}
950950
else if (status == Herder::ENVELOPE_STATUS_PROCESSED)

src/herder/test/HerderTests.cpp

Lines changed: 11 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -5373,8 +5373,8 @@ TEST_CASE("processing of next slot happens after apply", "[herder]")
53735373
StellarValue sv;
53745374
toStellarValue(env.statement.pledges.externalize().commit.value,
53755375
sv);
5376-
auto txSet =
5377-
herder.getPendingEnvelopes().getTxSet(sv.txSetHash);
5376+
auto txSet = std::get<TxSetXDRFrameConstPtr>(
5377+
herder.getPendingEnvelopes().getTxSet(sv.txSetHash));
53785378
REQUIRE(txSet);
53795379
return std::make_pair(env, txSet->toStellarMessage());
53805380
}
@@ -5410,11 +5410,14 @@ TEST_CASE("processing of next slot happens after apply", "[herder]")
54105410

54115411
SCPEnvelope invalidEnv{};
54125412
invalidEnv.statement.slotIndex = invalidSlot;
5413-
invalidEnv.statement.pledges.type(SCP_ST_PREPARE);
5414-
invalidEnv.statement.pledges.prepare().ballot.counter = 1;
5415-
invalidEnv.statement.pledges.prepare().ballot.value =
5416-
xdr::xdr_to_opaque(invalidValue);
5417-
invalidEnv.statement.pledges.prepare().quorumSetHash = qsetHash;
5413+
invalidEnv.statement.pledges.type(SCP_ST_CONFIRM);
5414+
auto& invalidConfirm = invalidEnv.statement.pledges.confirm();
5415+
invalidConfirm.ballot.counter = 1;
5416+
invalidConfirm.ballot.value = xdr::xdr_to_opaque(invalidValue);
5417+
invalidConfirm.nPrepared = 1;
5418+
invalidConfirm.nCommit = 1;
5419+
invalidConfirm.nH = 1;
5420+
invalidConfirm.quorumSetHash = qsetHash;
54185421
invalidEnv.statement.nodeID = keyA.getPublicKey();
54195422
herderC.signEnvelope(keyA, invalidEnv);
54205423

@@ -5438,7 +5441,7 @@ TEST_CASE("processing of next slot happens after apply", "[herder]")
54385441
Herder::ENVELOPE_STATUS_READY);
54395442
REQUIRE(herderC.trackingConsensusLedgerIndex() == target - 1);
54405443

5441-
// Inject future invalid PREPARE for slot target+1. Herder should accept.
5444+
// Inject future invalid CONFIRM for slot target+1. Herder should accept.
54425445
REQUIRE(herderC.recvSCPEnvelope(invalidEnv) ==
54435446
Herder::ENVELOPE_STATUS_FETCHING);
54445447

0 commit comments

Comments
 (0)