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

garydgregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-csv.git


The following commit(s) were added to refs/heads/master by this push:
     new 82aef023 Remove extra layer of assertAll()
82aef023 is described below

commit 82aef023f95154329416753eeba87aa93292d62b
Author: Gary Gregory <[email protected]>
AuthorDate: Mon Oct 19 18:44:14 2026 -0400

    Remove extra layer of assertAll()
---
 src/test/java/org/apache/commons/csv/CSVRecordTest.java | 13 ++++---------
 1 file changed, 4 insertions(+), 9 deletions(-)

diff --git a/src/test/java/org/apache/commons/csv/CSVRecordTest.java 
b/src/test/java/org/apache/commons/csv/CSVRecordTest.java
index 82153297..6fedd65b 100644
--- a/src/test/java/org/apache/commons/csv/CSVRecordTest.java
+++ b/src/test/java/org/apache/commons/csv/CSVRecordTest.java
@@ -18,7 +18,6 @@
  */
 package org.apache.commons.csv;
 
-import static org.junit.jupiter.api.Assertions.assertAll;
 import static org.junit.jupiter.api.Assertions.assertArrayEquals;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertFalse;
@@ -104,10 +103,8 @@ class CSVRecordTest {
         try (CSVParser parser = CSVParser.parse(csv, format)) {
             final CSVRecord record = parser.nextRecord();
 
-            assertAll("Test that it gets the last instance of a column when 
there are duplicate headings",
-                () -> assertEquals("2", record.get("A")),
-                () -> assertEquals("6", record.get("B"))
-            );
+            assertEquals("2", record.get("A"));
+            assertEquals("6", record.get("B"));
         }
     }
 
@@ -120,10 +117,8 @@ class CSVRecordTest {
             final CSVRecord record = parser.nextRecord();
             final Map<String, String> map = record.toMap();
 
-            assertAll("Test that it gets the last instance of a column when 
there are duplicate headings",
-                () -> assertEquals("2", map.get("A")),
-                () -> assertEquals("6", map.get("B"))
-            );
+            assertEquals("2", map.get("A"));
+            assertEquals("6", map.get("B"));
         }
     }
 

Reply via email to