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

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


The following commit(s) were added to refs/heads/branch-3.2 by this push:
     new 6255b1282e8 [improve][test][branch-3.2] Improve 
ManagedLedgerTest.testGetNumberOfEntriesInStorage
6255b1282e8 is described below

commit 6255b1282e8488ca3fc5fddcf320480af9b20255
Author: Lari Hotari <lhot...@apache.org>
AuthorDate: Thu Mar 28 12:36:44 2024 +0200

    [improve][test][branch-3.2] Improve 
ManagedLedgerTest.testGetNumberOfEntriesInStorage
    
    - improve the test case based on suggestion from @poorbarcode
---
 .../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 6782867eb50..0d9d6c0e573 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
@@ -2641,10 +2641,10 @@ public class ManagedLedgerTest extends 
MockedBookKeeperTestCase {
         Awaitility.await().untilAsserted(() -> {
            assertEquals(managedLedger.getState(), 
ManagedLedgerImpl.State.ClosedLedger);
         });
-        managedLedger.rollCurrentLedgerIfFull();
+        managedLedger.createLedgerAfterClosed();
         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