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

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

commit b3daff2323172cac47f91683c04b0edf8f14a048
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Fri Jul 9 11:22:15 2021 -0400

    Better resource management.
---
 .../java/org/apache/commons/csv/issues/JiraCsv247Test.java     | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/src/test/java/org/apache/commons/csv/issues/JiraCsv247Test.java 
b/src/test/java/org/apache/commons/csv/issues/JiraCsv247Test.java
index a3a7f8f..4dc18a0 100644
--- a/src/test/java/org/apache/commons/csv/issues/JiraCsv247Test.java
+++ b/src/test/java/org/apache/commons/csv/issues/JiraCsv247Test.java
@@ -67,14 +67,16 @@ public class JiraCsv247Test {
         assertFalse(format.getAllowMissingColumnNames(), "By default we should 
not allow missing column names");
 
         assertThrows(IllegalArgumentException.class, () -> {
-            try (final Reader in = new 
StringReader("a,,c,d,e\n1,2,3,4,5\nv,w,x,y,z")) {
-                format.parse(in);
+            try (final Reader reader = new 
StringReader("a,,c,d,e\n1,2,3,4,5\nv,w,x,y,z");
+                CSVParser parser = format.parse(reader);) {
+                // should fail
             }
         }, "1 missing column header is not allowed");
 
         assertThrows(IllegalArgumentException.class, () -> {
-            try (final Reader in = new 
StringReader("a,,c,d,\n1,2,3,4,5\nv,w,x,y,z")) {
-                format.parse(in);
+            try (final Reader reader = new 
StringReader("a,,c,d,\n1,2,3,4,5\nv,w,x,y,z");
+                CSVParser parser = format.parse(reader);) {
+                // should fail
             }
         }, "2+ missing column headers is not allowed!");
     }

Reply via email to