This is an automated email from the ASF dual-hosted git repository.

xyz pushed a commit to branch branch-2.8
in repository https://gitbox.apache.org/repos/asf/pulsar.git


The following commit(s) were added to refs/heads/branch-2.8 by this push:
     new c534be7e72b [branch-2.8] Fix 
ManagedLedgerTest#testGetNumberOfEntriesInStorage
c534be7e72b is described below

commit c534be7e72b67d98777949d84527cf6047814565
Author: Yunze Xu <[email protected]>
AuthorDate: Thu Jul 28 22:46:00 2022 +0800

    [branch-2.8] Fix ManagedLedgerTest#testGetNumberOfEntriesInStorage
    
    See
    https://github.com/apache/pulsar/issues/16813#issuecomment-1198241372
    for details.
---
 .../java/org/apache/bookkeeper/mledger/impl/ManagedLedgerTest.java    | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerTest.java
 
b/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerTest.java
index 67c90d625ea..7f1ed67ce8d 100644
--- 
a/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerTest.java
+++ 
b/managed-ledger/src/test/java/org/apache/bookkeeper/mledger/impl/ManagedLedgerTest.java
@@ -2235,8 +2235,8 @@ public class ManagedLedgerTest extends 
MockedBookKeeperTestCase {
         stateUpdater.set(managedLedger, ManagedLedgerImpl.State.LedgerOpened);
         managedLedger.rollCurrentLedgerIfFull();
         Awaitility.await().untilAsserted(() -> {
-            assertEquals(managedLedger.getLedgersInfo().size(), 2);
-            assertEquals(managedLedger.getState(), 
ManagedLedgerImpl.State.ClosedLedger);
+            assertEquals(managedLedger.getLedgersInfo().size(), 3);
+            assertEquals(managedLedger.getState(), 
ManagedLedgerImpl.State.LedgerOpened);
         });
         assertEquals(5, 
managedLedger.getLedgersInfoAsList().get(0).getEntries());
         assertEquals(5, 
managedLedger.getLedgersInfoAsList().get(1).getEntries());

Reply via email to