chore: fix JRS tests failing because of PCES (#17508)

Signed-off-by: Lazar Petrovic <lpetrovic05@gmail.com>
hiero-ledger · Jan 23, 2025 · 4da0822 · 4da0822
1 parent 9701de2
commit 4da0822
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 10 deletions.
diff --git a/...platform-core/src/main/java/com/swirlds/platform/event/preconsensus/PcesFileIterator.java b/...platform-core/src/main/java/com/swirlds/platform/event/preconsensus/PcesFileIterator.java
@@ -94,7 +94,6 @@ private void findNext() throws IOException {
                 // This is possible (if not likely) when a node is shut down abruptly.
                 hasPartialEvent = true;
                 closeFile();
-                throw e;
             } catch (final NullPointerException e) {
                 // The PlatformEvent constructor can throw this if the event is malformed.
                 hasPartialEvent = true;

diff --git a/...m-test/src/test/java/com/swirlds/platform/test/event/preconsensus/PcesReadWriteTests.java b/...m-test/src/test/java/com/swirlds/platform/test/event/preconsensus/PcesReadWriteTests.java
@@ -284,14 +284,7 @@ void truncatedEventTest(@NonNull final AncientMode ancientMode, final boolean tr
         final PcesFileIterator iterator = file.iterator(Long.MIN_VALUE);
         final List<PlatformEvent> deserializedEvents = new ArrayList<>();
 
-        if (truncateOnBoundary) {
-            iterator.forEachRemaining(deserializedEvents::add);
-        } else {
-            assertThrows(
-                    IOException.class,
-                    () -> iterator.forEachRemaining(deserializedEvents::add),
-                    "A partial event should have been detected and an IOException should have been thrown");
-        }
+        iterator.forEachRemaining(deserializedEvents::add);
 
         assertEquals(truncateOnBoundary, !iterator.hasPartialEvent());
 
@@ -363,7 +356,7 @@ void corruptedEventsTest(@NonNull final AncientMode ancientMode) throws IOExcept
             assertEquals(events.get(i), iterator.next());
         }
 
-        assertThrows(IOException.class, iterator::next);
+        assertThrows(Exception.class, iterator::next);
     }
 
     @ParameterizedTest