process only if the worksheetname is '*' for all

Set Validate Range for rows to max rows available

Public/Import-Excel.ps1

@@ -23,7 +23,7 @@
         [Parameter(ParameterSetName = 'PackageC', Mandatory)]
         [Switch]$NoHeader     ,
         [Alias('HeaderRow', 'TopRow')]
-        [ValidateRange(1, 9999)]
+        [ValidateRange(1, 1048576)]
         [Int]$StartRow = 1,
         [Alias('StopRow', 'BottomRow')]
         [Int]$EndRow ,
@@ -235,8 +235,14 @@
                                     #    Write-Verbose "Import cell '$($Worksheet.Cells[$R, $P.Column].Address)' with property name '$($p.Value)' and value '$($Worksheet.Cells[$R, $P.Column].Value)'."
                                 }
                             }
+
+                            if ($WorksheetName -eq '*') {
                                 $xlBook["$targetSheetname"] += [PSCustomObject]$NewRow
                             }
+                            else {
+                                [PSCustomObject]$NewRow
+                            }
+                        }
                         #endregion
                     }
                 }
@@ -247,6 +253,8 @@
                 # $EndColumn = 0
                 if ($Path) { $stream.close(); $ExcelPackage.Dispose() }
 
+                if ($WorksheetName -eq '*') {
+
                     if ($Raw) {
                         foreach ($entry in $xlbook.GetEnumerator()) {
                             $entry.Value
@@ -262,3 +270,4 @@
             }
         }
     }
+}

__tests__/ImportExcelTests/ImportExcelEndRowEndColumn.tests.ps1

@@ -32,6 +32,12 @@ Describe 'Test' -Tag ImportExcelEndRowAndCols {
             $colNames[0] | Should -Be 'Units'
             $colNames[1] | Should -Be 'Price'
         }
+
+        It 'Should read any row up to maximum allowed row' {
+            $xlMaxRows = "$PSScriptRoot\MaxRows.xlsx"
+            $actual = Import-Excel $xlMaxRows -StartRow 1048576 -EndRow 1048576 -NoHeader
+            $actual.P1 | Should -Be 1048576
+        }
     }
 
     Context 'Test reading multiple sheets with data in differnt rows and columns' {

__tests__/ImportExcelTests/ImportExcelTiming.tests.ps1

@@ -0,0 +1,12 @@
+Import-Module $PSScriptRoot\..\..\ImportExcel.psd1 -Force
+
+Describe "Tests Import-Excel Timings" -Tag Timing {
+    It "Should read the 20k xlsx in -le 2100 milliseconds" {
+        $timer = Measure-Command {
+            $data = Import-Excel $PSScriptRoot\TimingRows20k.xlsx
+        }
+
+        $timer.TotalMilliseconds | Should -BeLessOrEqual 2100        
+        $data.Count | Should -Be 19999
+    }
+}

changelog.md

@@ -3,6 +3,7 @@
 - Fix docs [#1254](https://github.com/dfinke/ImportExcel/pull/1251)`Add-Worksheet` warning. Thank you [Wilson Stewart](https://github.com/WilsonStewart)
 - Fix docs [#1251](https://github.com/dfinke/ImportExcel/pull/1251)`Add-Worksheet` warning. Thank you [Jeremiah Adams](https://github.com/JeremiahTheFirst)
 - Fix docs [#1253](https://github.com/dfinke/ImportExcel/pull/1253) `convertfrom-exceltosqlinsert`. Thank you [Wes Stahler](https://github.com/stahler)
+- Set Validate Range for rows to max rows available [#1273](https://github.com/dfinke/ImportExcel/pull/1273). Thank you [Stephen Brown](https://github.com/steve-daedilus)
 - Extended Get-ExcelFileSummary to include more Visible -eq $true|$false
 
 # 7.8.1