Fix timing issue in tests

Author: marta-lokhova

src/herder/test/HerderTests.cpp

@@ -5068,7 +5068,9 @@ herderExternalizesValuesWithProtocol(uint32_t version,
             REQUIRE(lcl == currentCLedger());
 
             waitForAB(fewLedgers, false);
-            REQUIRE(currentALedger() == nextLedger);
+            // waitForAB stops once A reaches nextLedger, but A may close one
+            // more ledger before crankUntil's periodic check observes it
+            REQUIRE(currentALedger() >= nextLedger);
             // C is at most a ledger behind
             REQUIRE(currentCLedger() >= nextLedger - 1);
         }

src/overlay/test/TCPPeerTests.cpp

@@ -159,6 +159,13 @@ TEST_CASE("TCPPeer read malformed messages", "[overlay][tcppeer]")
 
     auto const& msgRead =
         n1->getOverlayManager().getOverlayMetrics().mMessageRead;
+
+    // crankUntilAuthenticated returns as soon as both peers reach GOT_AUTH,
+    // but n0 still sends SEND_MORE_EXTENDED and GET_SCP_STATE right after
+    // auth. Wait for p1 to read those (HELLO, AUTH, plus the two control
+    // messages = 4) so the baseline snapshot below is stable.
+    s->crankUntil([&]() { return msgRead.count() >= 4; },
+                  std::chrono::seconds(10), false);
     auto msgReadPrev = msgRead.count();
 
     auto msg = makeStellarMessage(1);