From fb2f90d88f540c2df051d318c8f176a643462bf0 Mon Sep 17 00:00:00 2001 From: parthiv39731 <70740707+parthiv39731@users.noreply.github.com> Date: Mon, 6 Nov 2023 19:08:47 +0530 Subject: [PATCH] BAEL-7177 review comments implemented. --- .../modifystream/ModifyStreamUnitTest.java | 14 +++----------- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/core-java-modules/core-java-streams-5/src/test/java/com/baeldung/modifystream/ModifyStreamUnitTest.java b/core-java-modules/core-java-streams-5/src/test/java/com/baeldung/modifystream/ModifyStreamUnitTest.java index f5fab408b8..9da3beb40e 100644 --- a/core-java-modules/core-java-streams-5/src/test/java/com/baeldung/modifystream/ModifyStreamUnitTest.java +++ b/core-java-modules/core-java-streams-5/src/test/java/com/baeldung/modifystream/ModifyStreamUnitTest.java @@ -95,15 +95,6 @@ public class ModifyStreamUnitTest { assertEquals(3, newPersonList.size()); } - @Test - void givenPersonList_whenRemovePersonWithRemoveIf_thenPersonRemoved() { - assertEquals(4, personList.size()); - - personList.removeIf(e -> e.getName().equals("John")); - - assertEquals(3, personList.size()); - } - @Test void givenPersonList_whenUpdatePersonEmailByInterferingWithForEach_thenPersonEmailUpdated() { personList.stream().forEach(e -> e.setEmail(e.getEmail().toUpperCase())); @@ -128,11 +119,12 @@ public class ModifyStreamUnitTest { newImmutablePersonList.forEach(e -> assertEquals(e.getEmail(), e.getEmail().toUpperCase())); } + @Test void givenPersonList_whenUpdatePersonEmailByInterferingWithPeek_thenPersonEmailUpdated() { personList.stream() - .peek(e -> e.setEmail(e.getEmail().toUpperCase())) - .collect(Collectors.toList()); + .peek(e -> e.setEmail(e.getEmail().toUpperCase())) + .collect(Collectors.toList()); personList.forEach(e -> assertEquals(e.getEmail(), e.getEmail().toUpperCase())); }