Merge pull request #32 from CSBiology/bug-noRowsFromFile-#31

#31 No FsRows when read from file
fslaborg · Jul 17, 2023 · 1b8d874 · 1b8d874
2 parents a0c11d0 + 0dfd1f0
commit 1b8d874
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/src/FsSpreadsheet.ExcelIO/FsExtensions.fs b/src/FsSpreadsheet.ExcelIO/FsExtensions.fs
@@ -199,7 +199,11 @@ module FsExtensions =
                 )
 
             sheets
-            |> Seq.fold (fun wb sheet -> FsWorkbook.addWorksheet sheet wb) (new FsWorkbook())
+            |> Seq.fold (
+                fun wb sheet -> 
+                    sheet.RescanRows()      // we need this to have all FsRows present in the final FsWorksheet
+                    FsWorkbook.addWorksheet sheet wb
+            ) (new FsWorkbook())
 
         /// <summary>
         /// Creates an FsWorkbook from a given Stream to an XlsxFile.

diff --git a/tests/FsSpreadsheet.ExcelIO.Tests/FsExtensions.fs b/tests/FsSpreadsheet.ExcelIO.Tests/FsExtensions.fs
@@ -159,6 +159,9 @@ let fsExtensionTests =
                 testCase "Worksheet SwateTemplateMetadata from 2EXT02_Protein has correct value" <| fun _ ->
                     let v = tf2Worksheet.Value.CellCollection.TryGetCell(1,1).Value.Value
                     Expect.equal v "Id" "value is not equal"
+                testCase "Worksheet SwateTemplateMetadata from 2EXT02_Protein has FsRows" <| fun _ ->
+                    let rows = tf2Worksheet.Value.Rows
+                    Expect.isGreaterThan rows.Length 0 "Worksheet SwateTemplateMetadata from 2EXT02_Protein has no FsRows"
             ]
         ]
     ]