Merge pull request #137 from Kotlin/fix-read-excel

Consider skipRows when obtaining column indexes #132

Author: koperagen

dataframe-excel/src/main/kotlin/org/jetbrains/kotlinx/dataframe/io/xlsx.kt

@@ -158,7 +158,7 @@ public fun DataFrame.Companion.readExcel(
             }
         }
     } else {
-        sheet.getRow(0).map { it.columnIndex }
+        sheet.getRow(skipRows).map { it.columnIndex }
     }
 
     val headerRow = sheet.getRow(skipRows)

dataframe-excel/src/test/kotlin/org/jetbrains/kotlinx/dataframe/io/XlsxTest.kt

@@ -84,4 +84,10 @@ class XlsxTest {
         val df = DataFrame.readExcel(testResource("custom_header_position.xlsx"), skipRows = 1)
         df.columnNames() shouldBe listOf("header1", "header2")
     }
+
+    @Test
+    fun `consider skipRows when obtaining column indexes`() {
+        val df = DataFrame.readExcel(testResource("header.xlsx"), skipRows = 6)
+        df.columnNames() shouldBe listOf("Well", "Well Position", "Omit", "Sample Name", "Target Name", "Task", "Reporter", "Quencher")
+    }
 }