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

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


The following commit(s) were added to refs/heads/branch-3.0 by this push:
     new e3531e808c1 [fix][test][branch-3.0] Fix broken 
ManagedLedgerTest.testGetNumberOfEntriesInStorage
e3531e808c1 is described below

commit e3531e808c172ff588e36499c41746835d06904a
Author: Lari Hotari <lhot...@apache.org>
AuthorDate: Thu Mar 28 12:14:23 2024 +0200

    [fix][test][branch-3.0] Fix broken 
ManagedLedgerTest.testGetNumberOfEntriesInStorage
    
    - change https://github.com/apache/pulsar/pull/22034 is missing from 
branch-3.0
---
 .../java/org/apache/bookkeeper/mledger/impl/ManagedLedgerTest.java  | 6 +++---
 1 file changed, 3 insertions(+), 3 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 10bfb699780..d20bbf0d7f5 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
@@ -2638,12 +2638,12 @@ public class ManagedLedgerTest extends 
MockedBookKeeperTestCase {
 
         // trigger ledger rollover and wait for the new ledger created
         Awaitility.await().untilAsserted(() -> {
-           assertEquals("LedgerOpened", 
WhiteboxImpl.getInternalState(managedLedger, "state").toString());
+           assertEquals(managedLedger.getState(), 
ManagedLedgerImpl.State.ClosedLedger);
         });
         managedLedger.rollCurrentLedgerIfFull();
         Awaitility.await().untilAsserted(() -> {
-            assertEquals(managedLedger.getLedgersInfo().size(), 3);
-            assertEquals(managedLedger.getState(), 
ManagedLedgerImpl.State.LedgerOpened);
+            assertEquals(managedLedger.getLedgersInfo().size(), 2);
+            assertEquals(managedLedger.getState(), 
ManagedLedgerImpl.State.ClosedLedger);
         });
         assertEquals(5, 
managedLedger.getLedgersInfoAsList().get(0).getEntries());
         assertEquals(5, 
managedLedger.getLedgersInfoAsList().get(1).getEntries());

Reply via email to