Adding test case

Author: lsergio

fastexcel-reader/src/main/java/org/dhatim/fastexcel/reader/RowSpliterator.java

@@ -92,7 +92,7 @@ private Row next() throws XMLStreamException {
         }
 
         int trackedColIndex = 0;
-        int rowIndex = getRowIndexWithFallback(++trackedRowIndex);
+        int rowIndex = getRowIndexWithFallback(trackedRowIndex);
         boolean isHidden = "1".equals(r.getAttribute("hidden"));
 
         List<Cell> cells = new ArrayList<>(rowCapacity);
@@ -103,7 +103,7 @@ private Row next() throws XMLStreamException {
                 break;
             }
 
-            Cell cell = parseCell(trackedColIndex++, rowIndex);
+            Cell cell = parseCell(trackedColIndex++);
             CellAddress addr = cell.getAddress();
             // we may have to adjust because we may have skipped blanks
             trackedColIndex = addr.getColumn() + 1;
@@ -112,6 +112,7 @@ private Row next() throws XMLStreamException {
             cells.set(addr.getColumn(), cell);
             physicalCellCount++;
         }
+        trackedRowIndex++;
         rowCapacity = Math.max(rowCapacity, cells.size());
         return new Row(rowIndex, physicalCellCount, cells, isHidden);
     }
@@ -128,7 +129,7 @@ private CellAddress getCellAddressWithFallback(int trackedColIndex, int trackedR
                 new CellAddress(trackedRowIndex, trackedColIndex);
     }
 
-    private Cell parseCell(int trackedColIndex, int trackedRowIndex) throws XMLStreamException {
+    private Cell parseCell(int trackedColIndex) throws XMLStreamException {
         CellAddress addr = getCellAddressWithFallback(trackedColIndex, trackedRowIndex);
         String type = r.getOptionalAttribute("t").orElse("n");
         String styleString = r.getAttribute("s");

fastexcel-reader/src/test/java/org/dhatim/fastexcel/reader/RowSpliteratorTest.java

@@ -0,0 +1,28 @@
+package org.dhatim.fastexcel.reader;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+
+import java.io.InputStream;
+
+import org.junit.jupiter.api.Test;
+
+public class RowSpliteratorTest {
+
+	@Test
+	void testBlankCells() throws Exception {
+		InputStream is = Resources.open("/xml/blank_cells.xml");
+		RowSpliterator it = new RowSpliterator(null, is);
+		it.tryAdvance(row -> {
+			assertEquals(8, row.getCellCount());
+			Cell cell = row.getCell(7);
+			assertEquals("H1", cell.getAddress().toString());
+		});
+		it.tryAdvance(row -> {
+			assertEquals(8, row.getCellCount());
+			Cell cell = row.getCell(7);
+			assertEquals("H2", cell.getAddress().toString());
+		});
+	}
+	
+}
+

fastexcel-reader/src/test/resources/xml/blank_cells.xml

@@ -0,0 +1,48 @@
+<sheetData>
+	<row>
+		<c t="b">
+			<v>1</v>
+		</c>
+		<c r="C1" t="b">
+			<v>1</v>
+		</c>
+		<c t="b">
+			<v>1</v>
+		</c>
+		<c t="b">
+			<v>1</v>
+		</c>
+		<c t="b">
+			<v>1</v>
+		</c>
+		<c t="b">
+			<v>1</v>
+		</c>
+		<c t="b">
+			<v>1</v>
+		</c>
+	</row>
+	<row>
+		<c t="b">
+			<v>1</v>
+		</c>
+		<c r="C2" t="b">
+			<v>1</v>
+		</c>
+		<c t="b">
+			<v>1</v>
+		</c>
+		<c t="b">
+			<v>1</v>
+		</c>
+		<c t="b">
+			<v>1</v>
+		</c>
+		<c t="b">
+			<v>1</v>
+		</c>
+		<c t="b">
+			<v>1</v>
+		</c>
+	</row>
+</sheetData>