Merge pull request #74 from waxxxd/master

monitorjbl · web-flow · commit d14367ecb0ea · 2017-01-26T15:28:37.000-05:00
-Additional check to ensure the cell is null before the new implemention
diff --git a/src/test/java/com/monitorjbl/xlsx/StreamingReaderTest.java b/src/test/java/com/monitorjbl/xlsx/StreamingReaderTest.java
@@ -603,6 +603,7 @@ public void testShouldReturnBlankForMissingCellPolicy_CREATE_NULL_AS_BLANK() thr
 		) {
   	  	  Row row = reader.iterator().next();
 		  assertEquals("B1 is Null ->", row.getCell(0, Row.CREATE_NULL_AS_BLANK).getStringCellValue()); //Remain unchanged
+		  assertThat(row.getCell(1), is(nullValue()));
 		  assertNotNull(row.getCell(1, Row.CREATE_NULL_AS_BLANK));
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -603,6 +603,7 @@ public void testShouldReturnBlankForMissingCellPolicy_CREATE_NULL_AS_BLANK() thr`
`603`	`603`	`) {`
`604`	`604`	`Row row = reader.iterator().next();`
`605`	`605`	`assertEquals("B1 is Null ->", row.getCell(0, Row.CREATE_NULL_AS_BLANK).getStringCellValue()); //Remain unchanged`
	`606`	`+ assertThat(row.getCell(1), is(nullValue()));`
`606`	`607`	`assertNotNull(row.getCell(1, Row.CREATE_NULL_AS_BLANK));`
`607`	`608`	`}`
`608`	`609`	`}`