PoiExcelUtil.getExcelDataAsMap() returns NullPointerException when there is no content in cell

[WendyZ123]

QAF Version

3.0.1b

Steps To Reproduce

1. prepare an excel sheet called "test" with a table like :
| Name | Age |
| Lilly | 18 |
| Jane | |
| Tom | 19 |
2. call PoiExcelUtil.getExcelDataAsMap(filePath, test); (test is sheetName)
3. run it

Expected behavior

read data from an excel file and return it as an object[][].

Actual behavior

return NullPointerException

Is the issue reproducible on runner?

Yes

• QAS
• Maven
• Gradle
• Ant
• Eclipse
• Intellij IDEA

Test case sample

Read data from the file and verify its size is not zero.

[cjayswal]

It worked as Expected for me. No Key-value / null for column, where col is empty, which is expected behavior.

[WendyZ123]

I think exception comes from this method:
+++++++++++++++++++++++++++++++
public static Object getCellContent(Cell cell) {
if (cell != null) {
FormulaEvaluator evaluator = cell.getSheet().getWorkbook().getCreationHelper().createFormulaEvaluator();
CellValue cellValue = evaluator.evaluate(cell);
...}}
+++++++++++++++++++++++++++++++
cellValue will be null if there is no content in cell. When we call cellValue.getCellType() later, It will throw NullPointerException.

But getCellContentAsString() method has handled it.
+++++++++++++++++++++++++++++++
public static String getCellContentAsString(Cell cell) {
if (cell != null) {
FormulaEvaluator evaluator = cell.getSheet().getWorkbook().getCreationHelper().createFormulaEvaluator();
CellValue cellValue = evaluator.evaluate(cell);
if (cellValue == null) {
return "";
} else {
...}..}
+++++++++++++++++++++++++++++++

I'm thinking if there is no value in cell, shall we handle it and return null or empty string?