Simplest behavior: null -> exception, like the other params.

garydgregory · garydgregory · commit e454d6847ea3 · 2014-03-17T18:41:52.000Z
git-svn-id: https://svn.apache.org/repos/asf/commons/proper/csv/trunk@1578504 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/src/test/java/org/apache/commons/csv/CSVParserTest.java b/src/test/java/org/apache/commons/csv/CSVParserTest.java
@@ -719,10 +719,10 @@ public void testParseNullUrlCharsetFormat() throws Exception {
         CSVParser.parse(null, Charset.defaultCharset(), CSVFormat.DEFAULT);
     }
 
-    @Test()
+    @Test(expected = IllegalArgumentException.class)
     public void testParserUrlNullCharsetFormat() throws Exception {
         final CSVParser parser = CSVParser.parse(new URL("http://commons.apache.org"), null, CSVFormat.DEFAULT);
-        parser.close();        
+        parser.close();
     }
 
     @Test(expected = IllegalArgumentException.class)

Original file line number	Diff line number	Diff line change
`@@ -719,10 +719,10 @@ public void testParseNullUrlCharsetFormat() throws Exception {`
`719`	`719`	`CSVParser.parse(null, Charset.defaultCharset(), CSVFormat.DEFAULT);`
`720`	`720`	`}`
`721`	`721`
`722`		`- @Test()`
	`722`	`+ @Test(expected = IllegalArgumentException.class)`
`723`	`723`	`public void testParserUrlNullCharsetFormat() throws Exception {`
`724`	`724`	`final CSVParser parser = CSVParser.parse(new URL("http://commons.apache.org"), null, CSVFormat.DEFAULT);`
`725`		`- parser.close();`
	`725`	`+ parser.close();`
`726`	`726`	`}`
`727`	`727`
`728`	`728`	`@Test(expected = IllegalArgumentException.class)`