From a942f2133dfc0ee480e5dcd8208311a3a4dce37d Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Fri, 29 Oct 2021 13:21:52 -0400
Subject: [PATCH 01/24] Add basic function, tests, and sample file

---
 Public/Read-OleDbData.TestA.sql |   4 ++
 Public/Read-OleDbData.TestB.sql |   7 ++
 Public/Read-OleDbData.TestC.sql |   4 ++
 Public/Read-OleDbData.TestD.sql |  10 +++
 Public/Read-OleDbData.TestE.sql |  31 +++++++++
 Public/Read-OleDbData.Tests.ps1 | 113 ++++++++++++++++++++++++++++++++
 Public/Read-OleDbData.ps1       |  68 +++++++++++++++++++
 Public/Read-OleDbData.xlsx      | Bin 0 -> 16588 bytes
 8 files changed, 237 insertions(+)
 create mode 100644 Public/Read-OleDbData.TestA.sql
 create mode 100644 Public/Read-OleDbData.TestB.sql
 create mode 100644 Public/Read-OleDbData.TestC.sql
 create mode 100644 Public/Read-OleDbData.TestD.sql
 create mode 100644 Public/Read-OleDbData.TestE.sql
 create mode 100644 Public/Read-OleDbData.Tests.ps1
 create mode 100644 Public/Read-OleDbData.ps1
 create mode 100644 Public/Read-OleDbData.xlsx

diff --git a/Public/Read-OleDbData.TestA.sql b/Public/Read-OleDbData.TestA.sql
new file mode 100644
index 0000000..576c6af
--- /dev/null
+++ b/Public/Read-OleDbData.TestA.sql
@@ -0,0 +1,4 @@
+select
+    ROUND(F1) as [A1]
+from
+    [sheet3$A1:A1]
\ No newline at end of file
diff --git a/Public/Read-OleDbData.TestB.sql b/Public/Read-OleDbData.TestB.sql
new file mode 100644
index 0000000..9334b90
--- /dev/null
+++ b/Public/Read-OleDbData.TestB.sql
@@ -0,0 +1,7 @@
+select ROUND(F1) as [A1] from [sheet1$A1:A1]
+union all select ROUND(F1) as [A1] from [sheet2$A1:A1]
+union all select ROUND(F1) as [A1] from [sheet3$A1:A1]
+union all select ROUND(F1) as [A1] from [sheet4$A1:A1]
+union all select ROUND(F1) as [A1] from [sheet5$A1:A1]
+union all select ROUND(F1) as [A1] from [sheet6$A1:A1]
+union all select ROUND(F1) as [A1] from [sheet7$A1:A1]
\ No newline at end of file
diff --git a/Public/Read-OleDbData.TestC.sql b/Public/Read-OleDbData.TestC.sql
new file mode 100644
index 0000000..9e3f5f7
--- /dev/null
+++ b/Public/Read-OleDbData.TestC.sql
@@ -0,0 +1,4 @@
+select
+  *
+from
+  [sheet1$A1:E10]
diff --git a/Public/Read-OleDbData.TestD.sql b/Public/Read-OleDbData.TestD.sql
new file mode 100644
index 0000000..efb169c
--- /dev/null
+++ b/Public/Read-OleDbData.TestD.sql
@@ -0,0 +1,10 @@
+select top 1
+    'All A1s' as [A1],
+    F1 as [Sheet1],
+    (select F1 FROM [sheet2$a1:a1]) as [Sheet2],
+    (select F1 FROM [sheet3$a1:a1]) as [Sheet3],
+    (select F1 FROM [sheet4$a1:a1]) as [Sheet4],
+    (select F1 FROM [sheet5$a1:a1]) as [Sheet5],
+    (select F1 FROM [sheet6$a1:a1]) as [Sheet6],
+    (select F1 FROM [sheet7$a1:a1]) as [Sheet7]
+FROM [sheet1$a1:a1]
\ No newline at end of file
diff --git a/Public/Read-OleDbData.TestE.sql b/Public/Read-OleDbData.TestE.sql
new file mode 100644
index 0000000..bd98cfa
--- /dev/null
+++ b/Public/Read-OleDbData.TestE.sql
@@ -0,0 +1,31 @@
+select top 1
+    'All A1s Start from Sheet1' as [A1],
+    F1 as [Sheet1],
+    (select F1 FROM [sheet2$a1:a1]) as [Sheet2],
+    (select F1 FROM [sheet3$a1:a1]) as [Sheet3],
+    (select F1 FROM [sheet4$a1:a1]) as [Sheet4]
+FROM [sheet1$a1:a1]
+UNION ALL
+select top 1
+    'All A1s Start from Sheet2' as [A1],
+    (select F1 FROM [sheet1$a1:a1]) as [Sheet1],
+    F1 as [Sheet2],
+    (select F1 FROM [sheet3$a1:a1]) as [Sheet3],
+    (select F1 FROM [sheet4$a1:a1]) as [Sheet4]
+FROM [sheet2$a1:a1]
+UNION ALL
+select top 1
+    'All A1s Start from Sheet3' as [A1],
+    (select F1 FROM [sheet1$a1:a1]) as [Sheet1],
+    (select F1 FROM [sheet2$a1:a1]) as [Sheet2],
+    F1 as [Sheet3],
+    (select F1 FROM [sheet4$a1:a1]) as [Sheet4]
+FROM [sheet3$a1:a1]
+UNION ALL
+select top 1
+    'All A1s Start from Sheet4' as [A1],
+    (select F1 FROM [sheet1$a1:a1]) as [Sheet1],
+    (select F1 FROM [sheet2$a1:a1]) as [Sheet2],
+    (select F1 FROM [sheet3$a1:a1]) as [Sheet3],
+    F1 as [Sheet4]
+FROM [sheet4$a1:a1]
diff --git a/Public/Read-OleDbData.Tests.ps1 b/Public/Read-OleDbData.Tests.ps1
new file mode 100644
index 0000000..c2cf65b
--- /dev/null
+++ b/Public/Read-OleDbData.Tests.ps1
@@ -0,0 +1,113 @@
+. .\Read-OleDbData.ps1
+Describe "Read-OleDbData" {
+    BeforeAll{
+        $tfp = (Get-ChildItem Read-OleDbData.xlsx).fullname # test file path
+        $cs = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$tfp;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
+    }
+    Context "When Read-OleDbData.xlsx and we want sheet1 a1" {
+        BeforeAll{
+            $sql = "select ROUND(F1) as [A1] from [sheet1`$A1:A1]"
+            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
+        }
+        It "should be PSCustomObject" {
+            $Results.GetType().Name | Should -Be 'PSCustomObject'
+        }
+        It "should have length of 1" {
+            @($Results).length | Should -Be 1
+        }
+        It "should be value of 1" {
+            $Results.A1 | Should -Be 1
+        }
+    }
+    Context "When Read-OleDbData.xlsx and we want sheet2 a1" {
+        BeforeAll{
+            $sql = "select ROUND(F1) as [A1] from [sheet2`$A1:A1]"
+            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
+        }
+        It "should be PSCustomObject" {
+            $Results.GetType().Name | Should -Be 'PSCustomObject'
+        }
+        It "should have length of 1" {
+            @($Results).length | Should -Be 1
+        }
+        It "should be value of 2" {
+            $Results.A1 | Should -Be 2
+        }
+    }
+    Context "When Read-OleDbData.xlsx and we want a1 on sheet3 and sql is in a file" {
+        BeforeAll{
+            $sql = Get-Content .\Read-OleDbData.TestA.sql -raw
+            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
+        }
+        It "should be PSCustomObject" {
+            $Results.GetType().Name | Should -Be 'PSCustomObject'
+        }
+        It "should have length of 1" {
+            @($Results).length | Should -Be 1
+        }
+        It "should be value of 2" {
+            $Results.A1 | Should -Be 3
+        }
+    }
+    Context "When Read-OleDbData.xlsx, we want a1 on sheets1-7, want to validate the values match properly, and sql is in a file" {
+        BeforeAll{
+            $sql = Get-Content .\Read-OleDbData.TestB.sql -raw
+            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
+        }
+        It "should be PSCustomObject" {
+            $Results[0].GetType().Name | Should -Be 'PSCustomObject'
+        }
+        It "should have length of 7" {
+            @($Results).length | Should -Be 7
+        }
+        It "should have data where sum of all initial records match the value of the last record" {
+            $a = $Results.A1
+            ($a[0..5] | Measure-Object -sum).sum | Should -Be $a[6]
+        }
+    }
+    Context "When Read-OleDbData.xlsx, select range sheet1 A1:E10, and sql is in a file" {
+        #note, this spreadsheet doesn't have the fields populated other than A1, so it will, correctly, return only one value
+        BeforeAll{
+            $sql = Get-Content .\Read-OleDbData.TestC.sql -raw
+            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
+        }
+        It "should be PSCustomObject" {
+            $Results.GetType().Name | Should -Be 'PSCustomObject'
+        }
+        It "should have length of 1" {
+            @($Results).length | Should -Be 1
+        }
+    }
+    Context "When Read-OleDbData.xlsx, select a1 from all sheets as a single record, and sql is in a file" {
+        #note, this spreadsheet doesn't have the fields populated other than A1, so it will, correctly, return only one value
+        BeforeAll{
+            $sql = Get-Content .\Read-OleDbData.TestD.sql -raw
+            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
+        }
+        It "should be PSCustomObject" {
+            $Results.GetType().Name | Should -Be 'PSCustomObject'
+        }
+        It "should have length of 1" {
+            @($Results).length | Should -Be 1
+        }
+        It "should have 8 properties" {
+            @($Results.psobject.Properties).length | Should -Be 8
+        }
+    }
+    Context "When Read-OleDbData.xlsx, select a1 from all sheets as a single record multiple times to create a range, and sql is in a file" {
+        #note, this spreadsheet doesn't have the fields populated other than A1, so it will, correctly, return only one value
+        BeforeAll{
+            $sql = Get-Content .\Read-OleDbData.TestE.sql -raw
+            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
+        }
+        It "should be Object[]" {
+            $Results.GetType().Name | Should -Be 'Object[]'
+        }
+        It "should have length of 4" {
+            @($Results).length | Should -Be 4
+        }
+        It "should have 5 properties on first record" {
+            @($Results[0].psobject.Properties).length | Should -Be 5
+        }
+    }
+}
diff --git a/Public/Read-OleDbData.ps1 b/Public/Read-OleDbData.ps1
new file mode 100644
index 0000000..579d904
--- /dev/null
+++ b/Public/Read-OleDbData.ps1
@@ -0,0 +1,68 @@
+#Requires -Version 5
+function Read-OleDbData {
+    <#
+        .SYNOPSIS
+        Read data from an OleDb source using dotnet classes. This allows for OleDb queries against excel spreadsheets. Examples will only be for querying xlsx files.
+
+        For additional documentation, see Microsoft's documentation on the System.Data OleDb namespace here:
+        https://docs.microsoft.com/en-us/dotnet/api/system.data.oledb
+
+        .DESCRIPTION
+        Read data from an OleDb source using dotnet classes. This allows for OleDb queries against excel spreadsheets. Examples will only be for querying xlsx files using ACE.
+
+        .EXAMPLE
+        Read-OleDbData `
+            -ConnectionString "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=file.xlsx;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'" `
+            -SqlStatement "select ROUND(F1) as [A] from [sheet1$A1:A1]"
+
+        .EXAMPLE
+        $ConnectionString = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=file.xlsx;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
+        $SqlStatement = "select ROUND(F1) as [A] from [sheet1$A1:A1]"
+        Read-OleDbData -ConnectionString $ConnectionString -SqlStatement $SqlStatement
+
+        .EXAMPLE
+        $ReadDataArgs = @{
+            SqlStatement = Get-Content query.sql -Raw
+            ConnectionString = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=file.xlsx;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
+        }
+        $Results = Read-OleDbData @ReadDataArgs
+        
+    #>
+    param(
+        [Parameter(Mandatory)]
+        [ValidateNotNullOrEmpty()]
+        [String] $ConnectionString,
+        [Parameter(Mandatory)]
+        [ValidateNotNullOrEmpty()]
+        [String] $SqlStatement   
+    )
+    
+    if ($IsLinux -or $IsMacOS) {
+        #todo: possibly add support for linux/mac somehow. i haven't researched this at all as i 
+        #      don't have a need for that and i'm not sure anyone else would in this context, but it does
+        #      appear that once upon a time mono had support for oledb, so maybe it is (or was) supported.
+        #      mono link here: https://www.mono-project.com/archived/ole_db/ 
+        Write-Error "Read-OleDbData only runs on Windows"
+        return
+    }
+
+    #todo: add checks for dotnet libs
+    #todo: possibly add checks for ace drivers, but maybe only needed if we want to restrict usage.
+    #      i currently just pass through the query and connection string so user is only limited by
+    #      their own machine setup, but they have to check for those dependencies themselves.
+    #todo: possibly try/catch. i personally do not do this, as i let them throw naturally and catch
+    #      them/handle them outside of this function.
+    #todo: possibly allow for DataSets instead of just DataTable. I used to use a similar method before
+    #      switching to the sqlcmd module and use datasets as I had multiple tables coming back from
+    #      sql sometimes. i think in this case, it's best to just keep it simple, but maybe someone
+    #      out there would prefer to be able to get multiple tables back. i have not tested that
+    #      with the OleDbDataAdapter.
+    #todo: possibly just return the datatable, i do it as below because i prefer to simplify the output
+    #      and get rid of the extra fields that come back with the datatable or rows.
+    
+    $DataTable = new-object System.Data.DataTable
+    $DataAdapter = new-object System.Data.OleDb.OleDbDataAdapter $SqlStatement,$ConnectionString
+    $null = $DataAdapter.Fill($DataTable)
+    $null = $DataAdapter.Dispose()
+    $DataTable.Rows | Select-Object $DataTable.Columns.ColumnName
+}
\ No newline at end of file
diff --git a/Public/Read-OleDbData.xlsx b/Public/Read-OleDbData.xlsx
new file mode 100644
index 0000000000000000000000000000000000000000..5458ccfb403ef5417928af40c4dff8fe476cbc0c
GIT binary patch
literal 16588
zcmeHuWl)^kwk;Ceogl&8-AT~k8r<F8J-9S3!7V@_Sa6r%8iGTR;O-v$HSB$E&SvMF
zmmlv{om=-d)!o%}uTiUKk7;AgPfim289E3g2s8)?2oVUsLL5T~C<q8G@Ff}uG?<2v
zjkTkZwWH1(H(MhIEqYfgOTzchz$mjoz<{6s-`oFS3k)U<Shq7Gi(DjJA;r{UFb%&`
z#_)x(N~BZRg2FZ-#Ei6ZgAZ&y=;X!3Kq87xk$&Hq;qjHTf6m%RZQBs^MFQJ%i#+Lf
zZ)4b1j8n=FMm$!M$*=?Z0&5_dYrqU|zwb%ARN23UQNkTmMe*}k!ZQNR`#jFgSjAl4
zg6{j`HlBybL?5<k-qN|Qiu^51gq&HK_5}vHeyPxLP41LMY&kRS^WqC7<>zQaW@I9S
zaxILP1aU#>Qt<(E{chr;RNP3!Jz#|K+)mmtIro0Z1x$vXpYow1Hj=FyBtA0x>1E@6
zm*0WHD@^FnvfAd~a&S!$Vn*lp42gB$ko@9PoEcG)p4THX#FyeKY(^FT&7R;WDmA7@
zW2c``$w&T&RHU;s=e|YzksMlNPl~OhdHd0+ES|41^Py|=O=1Kyg|{3$4T9?)6NZQ^
zr)2h8R+hZ8(MkF~lTTPib}b^+JZ~COSww7M!;%=Zr(79aoR^T>hS-R!U5}iR4dUes
zn-`TW7jPgT_xIo+a({8)Y9&UJQy?p)fc^;wbYLBOBTEMc`iIB=b=?2K-uh2py*O4{
zwu2Eq=s@fyxc6dWJ{nn2(nUb5k?4*0Tk$!h>WCat-1$}tJmfbx{@|~Dn!RuOXXkk%
zwtI=rmsyJesOUT-H7+GViC1<`P}Jo1??mm2mpW0LC(b6$lV3}?Q93tAQ5V$}WJ~rf
zkcf^Q36~>(rB}g$LCeDn!sJW#RqvKo`>y|^6m&vRajzt(td=8dJML?$&(ymQTL>>h
zxTJPRlQDW7^i95%diGipon7K6DVlPbmFZ{La}&Gi7+QWl5l(GIyY-}#N$XW0W<~wM
z{PlGYS=O14dKK%@K)PEe2TVs%?_r-GfS}+zFqi$^No*U%P4vJ)K>UDN4hd)(S4##L
z8+!|V8yky<3|FkAA-lqf?WJROFSL4r%oFN>wr4s@s!ocXoedY&OG;nen`b`<CwQ`O
zQ8@;gq+ArH^~RHri+|xu%T0}ixkZN&LxZzlFN7$j%|I#Y-pR9DK89UrwPt;*uI<t_
zw9|f%Q>&{Dg2Y*w7G9@$0K_w^mjaS`%AqT#Yq1=~t((eCf$1U?xF2+>Gt<4vxQedG
zHVQxDW37BPOEMk#9Np`{+0Z2z&%4v_)S<x^EvC%^aeSSpdInYVfj78R@7SohVxUwF
zv@5P4(48%}*SvpqyMB+<qp)okKYLQ9o#CyIU$z|*Mcd?vd<3P@lna~_==c!fRj;=n
zz94_SuFJ0fNs{O5<A`A<Jf*%xK%rj@jVlGokC&AwVsvpdFq{VfOULo`_gia%<2T-7
zT7VMem#R4=N+d<b&HXglu51ECQjK)?RfJPTj&$bJPR-L1F)a9Fm&)p|it3sY^gH@h
z3SbPCJliI#Q-sV35ChH#L$nH8=lfO{?=;Bhuyu^42$v$QULhxG?VX&#%8-6H(C81~
zZGk|!+lLyj?7%jdjcS^c5s#l$6DF2gVWsg~#`+-zDgLdzT7BPYVjOLCXY;1Tslz!K
z-!HDZ;JZ=N(rsqR^+HO=cLuA-(ru5n+S^8z)A1R;keclpv{e+WaQ9o@`IIkE@*m0S
z5=5~HJ7#n=c^L$lvgz}B_4811`AuZe45e(VhhW@u^ukP77%ISD0<3&6MV)s?2I=in
zg54HhtD)m~4MF*?vx8=Ts8x>xb-4=iEl6ftu{Z1!&LsLo{hp3jP>3nY6g$fFf^_Ek
zZO~E3xy<`pq=$dplj7j9Qu`i-u!lP5;^J|>*W-QfA2%llOM=r)!aZ)fvDhCwLH5RX
zKm0`BFBo-+$7#N3i|lk#9Txie(a{BRPfLVDlpthJDgI@K#n+p`FJ?(5ZW2zAJ6Ob*
zFZaa@*aYJ`#&VSClixV>^<%k~c^`}PyHj$Z8A3UFqo<BicFFD6=tQ@li-+&AB6y}6
z2fh}H{bFz4Z-rd+MaT2#T^Qa8n~6EyQvI_L<BZLk=}(aWfSz7vV`k~~s)=!F;?j-}
z_;=qkwwGv{bD1z{w{^)2X`#(S8&ODs;<<5?h$F)#eMB3`9a}GfzV>%-l(La;zytaz
zC?p674hS?T&>R2ATL0{m|HxpVz|s@=+5g$MruYH#j)&4K@Xmjv<=q~sj5@K&h(d2o
zE&<`{k^gk;NCDElS8xC~!ZRm>sx*{_3&zp8z5T|6_p)WvICQEw6trL^Qs1`K%Pu;0
zI)X$tbLovtSfSa$Dzxl;YQ6krxElhbd31%TaUd+xa5FexPrPDaVh1<n{usWOmoqx)
z8NE$uv8WciI(Ijr+m&Ek#Q;%x9eabc@@nYbBDK&QqY3+L#NU~NwvL3JP{7Q>)-S!^
zfh7Va{%95+!1cT(oKGKoksePG?sCg;$zKgBNbOp;;xl715;+3=$}NkVZqgh_vXX96
z!bLKr!McVS8PC~Vud>dhnfWE<JKxt8r)|7LF#guNlzI8_JX5XbH`H|if?slLK_x2X
z2aqq=&wzK}C;p7|4yHy%jt&fuPfQOXKP^tfVwMrTX94OOQShLKc4KsMnqCI2k{7<e
zW?w7DNTahNJa-QJdY{K$j9ZLAuni1f@+$qlKJV-6iR@-Z<!$~B)^J8tzNv)WRQT3g
zYX8^a#Rgto-17kV%z&=1ms1Q|0;wfdfws`l&0lPH=Ax1H-DuD&vtI}!QB<>saWP5@
z?dm=&>V92FVu$`ERKKFfL{NWUZ?oZ@CN0|n)kq^tXGkR1u#E=SG@SI2hizE$o;7rx
zGB_c7Mch8CAd>>G8b>p`<(R<SYi=Vvd<0LX;=ZmgKkmEZV&_bYEU>_m)7Tb*?u9d;
zS5!u=Zd~*><FU&VTizZN{BVff-0crr(2J98Usp+zO%!*o=s(O<k{|9eFNY2LHXzAJ
zw;>`o+`~+n4~QL7fk^KJv~tSQz+jxq6OnAgW1+|K;-_xZqC;ITytve!Gi^SDvN%M!
zELvT+<~m<45=bccR_gHbbSX7jdME41Bx^X4^P`NU9@i<RZo8|qQ-fjT+Q$*7XHM@K
z(ZNr&WYMO5ZkT23TjAPI95ct_cLmUcA)*0IEKa%+KZc}8JOJGCQKbZo%qCXy&>s0v
z44a^i)>RJbTE&>65^qx}14yUT8v&qHhjW?jdyu&i0ny!5eTP{QEK=Nu(LZ97oY=?{
zHBGqhp?(#KptNxZ&;LDJnEznQK;bMSa?nL!i~sQVn7z4(Js|}SRzQ1pZ8PyeO~@Kk
ztbNYuJdi6gL|Oshn5MK_sXo?Q18TLalo3k9Myxo2@x3yyJ~nXYy!5g3gCLtcDqW>>
zWX{1d+mgIH5^@Kttc{3|B+RaSZk4u!)?I9xyYBp~dG|0EET>**Y<%!s^w{1tR|7M6
zDcr1-+ntPb{W;{D1k?%;yWt84m5>-Zmx2ZM!di|L%&&{A9M?hw(8GMh&V@?*%&(|M
zh`&wZW4Bl^?K`k8`K2LvGnF2F`Eq#Q7N^|)Hn2Y;dayKwphKsEY-L%-J^Y?Zid9ea
zMFPVjF12@NuBp38mrdCbk6tCuVEzCpl#=o3#np-a2~|e8zg<-_M4VbVB4+9(ttuVB
zh>8APQJ48`@VeQ@vUYxIeZzziC1JG;#4jh3eQ~!YvYrh4Am?|w0DeKA1ira>_~swx
zVpKHVrt=(}UNME%c<guR7+0tiYkASix;8|{=&}bgaQm(qIGiZOiUC3+4hv`Ue%JS{
zgNjQB?+<zT*6sQuDJ2#Kd!@f8%$%ruuYFKXCo3t2oN|yDRMOyVZ}$HVmy$Dt&k1#9
z%*LsJIv+qHWe>4v4f)}-L#w3ACJc5U5rSgu$cu0mO|H`iu2^xACe8!7^2<i+c?+z>
z#lPnY%O6~k1OU0xa}ekamEA};>t?BM$uPj3&4u27Zl_}S4YQnAR6#r5`yvuAmM%BA
zOxL$N*83z=<=k@=pG(XD<CH6+BLwvuT)~@#)zghAr?RY0?(Zs1{@UxOv~*td{677W
zy3RETqbQ`O?D-`QLj6e?CF65JjJ+>p%^6d8Sfx95bxA*Uziu!PNB@$%UYL(1X#PU)
zl@<+=B(evchl?-vH@n~~t0G8`JhLNc8HdVQBwr9dLj5A{wIPytRJu{=G5@7m#1;N7
zMf%>p#l*!F&H3>A170&Sc+~=}5JTq?PlCHxy60FNqh2>iddpNS$621B3-WzEq+OM`
zG;YDt{LrZjy8K$?^F`G8tpihC`qdK30?%8E{kK?CAOQJJ=*1gxQ>1F|PSjBEvqhAt
zip!$Hu+=tB$vzbKcCTONt&ZGPQnX;zg7Em12@u4z_V4eP_GsT2i;-Eh9L(Y3wTewt
zA>#T3!Fq*`ZEH%<l+_}mkyoaU#qwwyLPkE@L0^%BpL6j_TfVzz07(64q-^R%>>@RQ
zLMO8&TBSY^#CIiWwI3_^9xF4MvqK><D58PB{Jm`vf-ifB4oDXB>Ibq26Oq?L+Nyvn
zF4iyP%_2PuZ+{3ZTOi%fOYz*ryu=4T;KU7BvuNEw{3@^{l;9nL13Plcf1QR{{|qdV
zsf_4BKura)^g@@gk5u5Zsj=xeEwxg5jYki5nIBiLvvuLsJ|DTn2Ze?Z(f0PKgPV<a
zd@r0{pO30Sc*(3Vz85|Rqn_a4y1g{OprtI2yudifO61}bmm~m05mbODm00CDK7(og
zVg}4Y%*G-#Y_&x26+Lk{;p>QrAM~6VV^vm2=F3ULUL*m@Vj^h1|0h{^fMf~!FS2Bt
z*&zYRGLP^`mK^RinMblX0m+hzc+1}nB+CepEJ>P!;ny7Rs!$<m$LWQcc`RC-eEQyn
z18BA;?;9D9;-A;6@cY^nta`;Re@jScY9l`g;M3IO*Ce0+2qcS&czuY<8nJ8WOIbg&
zQ%y(!X)?r`ITw`<=GuuT`>hb5SRqO80E4MwXiHH_EheH17+L&pO=LXjtU+q-bR*b_
zeG=y9v?8j1e2JEr^qi_cu>C=-G{#MP^PKvWN~TDWs^s&Gpj6Uesi@?>Ys_;_<Uz%7
zX(n5pGrO>p`&L@!hsZ))1W;L^Im*xHG;oU9gS$D9&;9H>44A7-@Z~tB;fUC#+cM;n
zNLJJzQqMu&K!jwljcnjp#M>Mw?81yQWrt<jWP964%Sf7WO+9Z-sEgjf=`1Y+#W-ZU
zHY~WUXgE26ZRxysw9N3!`h`t-&Kn;{9QA)q9JWW|#7jq|0yEM<!WDt$f<F<F4zwy+
zs_2(&g$XMUm%&%XARlT@@(S;p4Ul&wwusbW1`NV8-FjGrZX8P_>>;gk3(;^~`N`zq
z68aj%Hop?gj`*P(xTPN{OhgakVqe93-yrM6^QVBnB@-)<7DSq4M==n^%UbSML>RA&
zqtUVhNNO>TnJ%h%6i|PgzA!0XuK0jYG#bVDCPNYwmrMp@yvE;@vR>qNoe+0yjQR+=
zbot8FTvSjAf%v>4{(Mr9>Z5QKjd|W>OK06DfnJDubI)#0^5)d@+w^=1*pl%6l3>G0
zvlgUB?)W+q+zTZU8sZF%k{x2O0ke`WcUkwRvOZu|0$nead`*!|b}Gx6D$P;k#0Wce
zXNW#DmHB#yOnq$6vs!hqR^#67B=LhCIa3Ec^?M@OP3kP#h}q>!vL*fR)14E`+8=Xf
zr?Zg=qHrk%>Iw<*@>zI=)QBNEm|J-kr$N8H^0c@(wbE!_VC^H^$RG7qNn{`d_;)EM
z>r0hVzTd3@B`L!hDd)&QWagMVW}Vy?WUK_evZ~vOBNV*8rx;VMvtb%%9U8-$(pt(_
zLayYE_)0Zof=*>iSd}>IX66d2Q%`SbB?jY0V<Z#0Uv>3jMwv}!5zpx$p}KHe1<F$5
zRc%YKS${&sE!d|XJDtxNIPa@Ek!SfOO>?RyG}TX$>vOlYCO-rt_ynH*x+Jb0c-~L=
z?<vLp3rhKtral8LlCF?<D0t&E`yHQ!RGIOCQ|9*`J;G%)dBLV+w3eU0HhnS)Iz1|;
zwtr@jSIU9|K?LNKR<C>Nh|WtDe=1<1)3BION|b|DcFJ^rL&axSS<3_RI$CZvd_T<s
zwYzxQcbR<{Vf@F3aZ$Tq$x5Rdrq2N9xI$IDAS5X}wJ{nZrVB<#H3?8jIz(7=mq6@-
zI-y%<3f$R|_&TWkx^Dz2zQctAoubO4TI!T!5~Mp;B?t>|Hm~s_>IhBzRl*IX1I;oQ
zgcTI9t(>;NNGprhHLh9M4Kz=4HSS&X-lMSC#y^Ck9|p1S?Y@RB=yo8rzi>}_Ngs!2
z7&FoeFy)VW9p}o38~MS1#IPQ3Fr#Ngm)SEz1`BD$wk4%?l$Vawc)F1ovOk<(B!10I
zgSV9|ZMTu@sJd>%{XnOfNi<}Z?hT=*5^`iM32jzkv|r6`CJp2Ht4XwluRK?UCEpK%
z(B+PW*1R{zRHA3oJCvX2Q0SnG+o}0ss77FOhV(QKlDY88M)0EyS_atR?q8Ei*xucz
zyHVfNnzG3b+;q829S$hQS1`*)T{0TIeq>dgN5tqMo#%=%jfEsNr3Mumuo|+!UYy(N
zhD*!Zk8qYLvaas<@C+UoGU<bY#T4D4EAIuwTsF~+Sk?5onkcSS+R~_AKqtjSF@T+~
z_?~9<{+EGj1O0rDAIPfr|2j}{{J|=Dn;Ay*zy*rEm#*G3xxOpU+CyUBR6$1sON^;r
zSf}BZi7gMZ%e~oaJ>U+hLiKo+fV#zL*WwW6m43QkU)l>AuGG*+NdOuDio_{9MOrk|
zn+|e;vRsBWu)q=gIiGKy%ViJJmV9dZJQ$~6K)12uj%F&O@fxPNUT{Mo<U;(+c>B_@
zWs(&_$ouPpQdR6{Ohtj_o5|5h8rTiA2?6!*u5oaHd9EO!<7!X2`dqMRjjm2TX}$$t
zOsh6TwMe2~5h^eDU7!7!Daj<py7xK>QD0>nO(##owBy%LY35`RH4TnJq?Ha|rjGJ0
z`vZ2xr!H;U4}r{*^9I+EU%Hl9Udg~{X@Bq1zWic6ae&lA$?((&rHfbdu#3<YTf~3F
zwH<0qjgdfV<(ji4YT331`!>%_opMeuGYp%Og?k0at0~@A47@#D=`__(Szucr2%Kas
z98>BCH=Shsu*fEjFfS-_6`@?;779{#SNRM<W^?E^cW8LoX*kZ`bVP$mlz*C&^}0bc
zuXHly=?-rZk6mVkqABMYam9|juGnY~<z>4Pv&}D;n1%JRcJ;fu*IEB~>cHfl@$Pba
z>*%CElk?83yVaR`S82RR{!@2zn7o&`T5^7H+DSY<V_GLJpuUq=WhxAT7q(J9f(#>G
zD*%RUJ&g(by>r7^8+npTN&48CQB_w5f~qxR=N^5vb0R2x(MIX#J&pGf?Ri^67NQzf
zX(;{KLd&Tgv;gZ=wR@Modtjmccj2fF3%NxG*vNGgJ{(p)HusLEMpi})kB^Ui{jS;<
z04@i5E7sLZcn6mowv{Nd?~7k8<K{^X)8p}~Yj)+8S(sy+aM8fXxXu(PbMq1fZTR9x
zUx6bupGLuuR&5cCP<)vtDOn>;wUkhIMgY9NYtGNV$nbVQaXcDrPI7;p1lJz>PNzlg
zm@h4SI*D}bosR`4C99{bSbR8&5sdXE9PN$dY^Jw8id7$U&oua@ux0i;0&%WcYRoR^
zpqRSP#+<$j<l#MH?If9)J$U&3-;*$D@)Sz2Nn(F!`{~Phcrg=^>>?yc_;`G5l3iko
z5kv^5Tu>=2C(ABh>8KD43EWiJ$0wRO*2`exd9my<dO?Yb>TP;ScDsR9%YZqeOfI|z
zsU~Edb<bQcrR%9W;&8mxW<E*??XT{UrvqM~*>kpYtKM2Onh$Eq>X-1LW}15^!8%$G
zLGEfIyHVvpX#n&4u#Q$`J$_+S;8fNdpRynV_=ZIhxK9F9{F)uX>BjjcJ5yQcIvO2Q
zl6%^6wWk(x!s7OILG*#u<}UR!7>4bek!auH{RNt1slLibBNXi_C`zdkbF8U{?=$m*
zyaiJr6<_Y_f|nhJuBZc0axHewl8r&5+61AJ>(;%YLXc4U@rmE}?gL3!4I^`If|hQm
zDz3u0tA1ynRLu=d)&vJ3Lk6G6-Qm_ra4VnB>E<P{b*-<amb8W<Q&ilah|;;gy~X;j
z(|UKfeZ;XfPI1)U*?Mz1kl1>E#C=w_Jcq&1;&pty6^FIpb+#cJg+DWkho5vv6k2{*
zx@2%b4$$*OygmYbncm4L*ctl@Wz)G}f1npkTZ3@B-KlJ^eHKr(8={G~)#6(N!BIVF
zWb!A`a)>#DaA#~6?#05Cfx|=F45|6Jllsuc(BT-1eR_#vkc$nLER|)2ZxLli>j+_}
zp&vNgMipDjg~IWR8ihjErudkbrOHB2_mMyOhm4U4+3S)i&{qlRhYYkE5eCd0fBA0p
zsiVMRg3r^CL=U(RM(>c1Yuy8;^zyq3T*6Fj_$>g#Rc3piw%6pP45Y&DVN@qq2DAZ^
z>)gR-I0CJ(@GFSdFSI(@V<qi3N9zIDEHQk}(-CsK4l`1*y`A3Vh(sbWF8<<Jo>AX;
zg!zQ<<SLlvYGG>HZ;grTAXd#&;weI|jE3;bo5GWu39uaf$^b?)yfxF+UqV@UxKBPN
z%rs&vI)-;+d(`qfs1EC~3~bbos1WNW;O|Nl)ASB0ou%o58pRe`Oj6k=Lem&s9K~Ce
znb_F*xnfd=qAcWI;t0=6XmPHky>U;EM3i1+16w0l1&mQmL2NR{HnxiIL_n*VB)Vhn
zdurLzc-vI==<0mu+iQ9D+|&Fy;d$-(7i4)0KVH2cXjpixdA|^hJ5AyAB6Z9C_|)lC
zb|i3gQRQZ|xYdZRub~wbH!l4zTtFQlA)+*nuH(5=DUCLSeJf!u2xu+bO*7^`+r}0<
zOu8vKXixmOzb-T$)8_wVG-Q!{qTYRWekNd^+~n!oPkwN`nIYR$zQ#fnzdr!c)hOuV
z^{i<m9E!1ER=$Cv-W+s@_(uv`675_(3Ehev`AZ7snW$lSbs?kq&uL*&XK!3t$g>P`
zgPmwFGKWj9Wlmh9BF1D)>WdpvKEgr7Ub<3gMhGQJeArayJNRBQzT>_5#zid=lJeUa
zww{zRuDlbrybhuuy0%Gx_P1o!5l7V#x$VSkby2%DSVk)h4A-^uMUm#UV1XHdAY;7v
z4Fin)s^q5dQrMwnQTI5OkIA5N`V1_DQI3k=8tmN6oy-cGMRMs#!W}Ck7)KfXu$E|f
zzP!6u>uyL69tY}j`t`W>EZY%CC>A2b6m`Dj<ch<cJl2=h=Fqj&B&-&fXq(s_Q?8Kp
zB;y)b?-lEupNnT$|8UWJ_Z+JtSU8?rGft)-k|`$1BfgVPht4MffprA$y%mkhTV#Iq
zm6TUB1gUn&jy%1t%xxrQLG#%188QUEONynocZFRG_{F5(@Y<URBBGkXxi>gaa#d9s
zXrLlP-va#Yx4$s?rj}vEtX;Y+>-L*YHHD3G#Yd8G1LU0FCmCk6vxSxpj>yswV9S0>
zY0KWB=S?aEKW~<u=OCEqcYrioH~G*-y*=(2w-wB@icoYcG+sfSXMHO}XJ2YTUyq17
z;U>|vH0}ieZ)2p(9AuVOYm(^*tE`lttzw~thL;^@|6%1M%E@<BO7XHceNf%$7442h
z`j4yrxuywYC>LK|cUvFMe1%OpL7PMp%BxY_-gtIhz+9YK1oT1^7X{tDH<cySyu>tJ
zJdR!~@*97JY?PX!Kxb^_f!i|8Z6|7^Mco*AO~kRCD^N|nVG(DuPd!ug`zvd7Ujk78
z04be8tm>t!<rd9vo^}T)gz4AS{aI>O&gOx})qIrCnq_i^cPF_6wM#S(5Zvd>M(}tK
zn@JE5XC|!Wb~s?G?Q1ol&B=_sbJPl;5+ryEYH@F~ZV71&bl0f73|l}h%o}TdoqpZ2
z-&*DZOFjkQxedY}>DR&0&C<x>q0Y-yvH}X~$annHKFC-5{mv`2Ff{|C<8qtbS@^p}
z`U9Z_+To-oL)~}RGS1p~+%+TKp)wm|4qxXT6h>2JV5<ze?Ms;Pv;wi2l(xf)q=Q=d
zG1}lsb_xm$hQo|pztXn!EwoCO%)*`584VJ>m1FPF*hqmOs+nUJ-%GGg@?)rGkL8|D
zK`4*VtE#w*Mkht=po)p7;U%OsynIO*o3c>@BSD9|{UZf#|C|p~v_f;vCZ64q?yWhN
zv}tIAsk#1mvO(<kYvQQYyb<N8eOTMjQVZ>hM8^$78B;e=u+=Fd5g3>izzolcL@o@I
zWlyswr>sRunn<2?0yvXzflb@8+SWngVP=y<NWbDW_+9~sY=C&iJ9SfkJTa*B9lkIL
zcHudNwyB`&vJ?sWbAO)mNbNnLwAEpc1JaBhy>i<SI6Hj|53t2v#pth0Ik3}N{<l>&
zgw1~9xVl$$KEgKe^RMlgxnErcxHU|E@ayBhSlR4XY%_-vMp&@C-F5_Twb;We$jryc
z$1N#1%!4PS0Pm+tMvn5nYMsEE){1yT1Glwb1mD4TEX|oB*V~477!=V%kb^L76C|SL
z6^ax>GJ};XyA?CV(~)`Wgu%!K1zlxbZ>xFh%c!e7Uk{P3^7*o8ien=lf?K{krKX)P
zo!|I}rH?*_jge>ARlCc~8S-=fStRX`rrQo>Lwu^(465=Lu0cNE>0S=1xQO5lwy)79
zX;zXZQ4A<WOIartDKR0f9G*{Yg1UB`9V`^_1_G+fz+S*OP@c;zuY5~f8ZpjV_)0na
z!XdL~?9c!;P{0mfw9Qb$q|MSj{h0`ASJ9TCTOxX2h{tii2{EZWD=%k;XN9dMbK8V`
zQI4PiL&&q)R&;?#iRhC|fjit^Rg*kc%#v+D8VdqzO_D$8{BQvI=TXrgCz3x8j2_P;
z1ESi(+8NPBF2L`ETs`6@1L1`fIf(0&Ho*0-W*}!1qc6~JJu2~xKCi9WeqCcvz2^Oh
zYm68}JKg{mxN3l9T6X9$L)R)$&bd8@4nxNqtv5#Bk^(LxBN5sYt%?Cx9w<+Gi{0Vf
z`K>$>=3_Mf`Zw`VH^#n80Pb*cp}Ezu4pbF`O!Wcns+WpCH=lyq-EO280k3sV>yX&-
z*8p*@XkGsIVqCo<d&h6}1$JD{Gc3FipM9_z95^|L`IQCGI%(0`f5!rkhams3z{5$T
z@Il~>|Fj1wPq17VnT`a}6!|TPi1zIJa|zR0vdinu=>WmV4ukKET0Ux6XMjKhVz?6P
zS44rV!YQ<#>HIaHvQ&8yyu|}BOM8bKzerw$0ut4ixr~)31TFP2`)s;xlkCol6-5gT
zrHsxL43eeJmu6*6ouw-|);@vHr7Pk?!mTk@WywrwCT2E{ady8wpt>o3FhL(#tS;Ak
zF}_}<y@3Fr30|Kkxm@AFgzpT_JGcPnKmOiQux7)={+0<Iwf(=E!1vJv6oCqUYnp<D
zQ|z}OO4<X&4<^XFyxvw#6^wc?!QPff#yA#cWGL;f6F6*NKEFcQp?Bl1_A4L@=HsD(
zELa-1;@G2+kueYLNm5gKWC1~h-pl1()%@@`ebN29x!tHz&UgD@DMDkssmHUR^X(r?
zn*;6lF0!nnG!yS#^7TSqeC_iAvY_YD1a3kx?$Jdv2@fn_)7YT6MS+o;O6fY;|K%Z=
z&^wfz-!j3YV*N`K#Q!%pf#C@&@o$;nQG5NR35Nfhn?OpP+Wogo@Te93)dYXUf_x~&
zur*By!iQL(^hYe<AiBHW{D&iaIbxFCj!YIJg@e-a6+AA7V^#gpka2F4rHbcd9TtFz
zX~<dAn7EEcL6O&KAXXd}INeSe`~43`=m~fuMsTv5+sT`>dJ}~^+@!2zb!-DwMU&6%
zf_Yre^R|KhBkldcLpbyn_DPLdje?1@Cv5W+h_R0b%&dWS`Aurne5t;B42Rop*Fe$g
z@3Fw*CDL`r?}WogVe79Z_%jv+*E}1j4|r8-bp=}3^iE_qVc1&o<YJ<nJ@7?vPm|uv
z*ah!L23kBcwdoy5^jKN1T-~Kx=Woq#_3g$RsZi8i2|GT{TF<DYrH!LTF;x~~>ci!T
z@#8#Z1exa_V~G;JiARtn?cU5oum*o{V_vHTZwOS>@_<{LIS`nCQBQw=Ak%@&ml1cL
z_Gki0(zfFu;xj^(YMKWVSkw%Cy&^$PDrEy~%s<S*{oMxlqTq%Sz<1_=cM9MgI6q-)
zZw4F*w^uZBbTqRzad`TgiC*8#T3<H{p0R?lavl_?tts6%G{!e(2}6bdHEi^{NeIXZ
zNm7z8FG0|oAXGvrBg*%~{L#*$1<wgnVEXTop6zF}k@t23qx1ih?J)mmZHMJQYdfs}
zS=(Xz&)N?Af7W(5{(o-ASV}n~^G`=d{=?A?^ehd8O!dsHANo1XNCDW#X@5T#=*n6%
zks`4%Vn2#u@Bl|ku)bVFzs<b}+&23A>qFQ1Ls)HIrgA`T)<<(>G4jYra)^srCx>Tk
zYGVanq;C#IgR*r#apyGGtL}-RgO`J)o_NN_Ms#J>hp{NQy<mb!ACQV$<;s5Xjd*==
zt)^7tc^JsmP)P3L+*?~%L_$w0(95V<vT264eYn+6MZZ3b$l)DV-~^WO@?an!D8PwA
zLmLA*dmCE^1_K*=qla$;9EJbiK|<iK93Lw$+3|211930Z=aFhSCM`JY+n<34EkoWC
zxM~f@ZYK7A{;t`qaRTvtd8K({wQ)l`mI1Lr!uzwEQYzR;c-FJ<f;{F%u;x#RTi_C&
zi=xMc>~dKou2qyaHuieS1^7)3Gk$Qeo~Yv8EP8NNNNvpF&C~wt`~XC>E~`COq;f|d
z%laS7JfG17ja7vfW1%87bj_iNxry?yqt9oR@L$2gW0*3w8KlVHY#=OFwO(Z@iprF<
zk94QY*!5Uax!c(iG~t9xj~RZc0-&5I-jut$?I=qpnAwnlB-EozaV};%8%6er@M?B>
zsdnMoVD$DFjDI@K-CRCLjuwaid{t*+#Ny?y%^=T3gq9L}9X~X68i~^1yIYCxcGQ9}
z&T(pX=o8||Pn-ejwLc&$cLyZY-;*SYEkSiQkZzU0q=o#?bkno7{a>~LY4+zWEw0Ua
z_Tj<<@N>kV{fM+G$>3MYOu4x3(`6vyjg{nM<#KAMq?2Ojr#V*RPBoVmxP6&FjzV)g
z=RcOK!7)8QfUVJ7r3KITGMg*cjS6N^Ay9iKSFfOUkfm!Pwky-(L*YCE6=F(gWVocz
zX5Lx0#Jh!qlYmLeqd-l78yMeBa-n3zI_Gga`c|l=|AZ|V&5tI#2{wla7mtJP?vvS;
zQ%_<B;>N4b3ZAfSI3>~ARIShPy^b-AGi7m#7Zv4^=U<#kvbw&jUmSb0TU}8`W}rru
z`s%RO&lO|>!75)$N};OGOduypjk(p-lIGp*8yN01ziMinwdk^j8lo)&TvXA^j-HJ-
z;JT=3I6?XZG9LNm^)a<x;1G!s>1`l_?PSYsS%Ig`Gap0l30u-pFW|us-c}y`c&FGB
zQ<bQEGJ4kJ$im>2lsI`Z%9ybA>^g6l_RjTol0%@L1;*fXqHg098}Uw?sEPijg7UmN
zOl~wH9Eu6E6MT<*X2?*3#wFmmdc!80=WG5FQvaHJjLwFmTU^SbO{?@BP{I3y)}UZ?
zAph!Wz<>OV|9JkzwSjVye+BsK)kyzX`0(5Y^p$_QHtFfYr&nnFT)Gb2>yzs>o-Y3D
zK>5$5ARvygkBk4sc==PDr^CZPk%r;_YZHGPC4P$Xbg1+v${pe_Q6ANmrzlUy4S%9g
zqx=%(QJ{E=@^lXECyEO20QDEc(T_d;Q<SGeNk37dF@A~i*d{+kc{+#k6J-VKmne^&
z=2MiXa|b_BxNv@n^4P3BMR|IX{1c@F?-Awcx$;whr&{<=09w*VfS=0v)1^;!$Dd2D
z$)A3MU#gH#0iVivKLKxnE!<y!<bMMEDe651d@73l1O%b_H-JASvZsJgC8nQ%;MD&H
z@TcJP6!58F^Aiw~=HCGRly9B_K9xUy0>aS#8^E7p$Wy?lV!}^Ac)EWB_)}VV3iz}$
z{|Sgp{|NXyefsZ%X8soFW4HbLp!C3@$N#&c@Ypl|K4|lAfj)M=zYi+K_&Ym&>|cK$
zboRGEAG^@s2Q_E<ot-}Rl21XO_JBVrb@>PAUpm64tN+?`{ag#2&tV1u`CAM2bopP4
j=zm_$#`aIk|17EHBq4xx00_tn;9nu|Do%LLhgbgxu1<k?

literal 0
HcmV?d00001


From 9417b25af17c815098619fa38ed0f1928432acae Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Fri, 29 Oct 2021 16:58:16 -0400
Subject: [PATCH 02/24] move tests into test folder

---
 {Public => __tests__}/Read-OleDbData.TestA.sql |   0
 {Public => __tests__}/Read-OleDbData.TestB.sql |   0
 {Public => __tests__}/Read-OleDbData.TestC.sql |   0
 {Public => __tests__}/Read-OleDbData.TestD.sql |   0
 {Public => __tests__}/Read-OleDbData.TestE.sql |   0
 {Public => __tests__}/Read-OleDbData.Tests.ps1 |   1 -
 {Public => __tests__}/Read-OleDbData.xlsx      | Bin
 7 files changed, 1 deletion(-)
 rename {Public => __tests__}/Read-OleDbData.TestA.sql (100%)
 rename {Public => __tests__}/Read-OleDbData.TestB.sql (100%)
 rename {Public => __tests__}/Read-OleDbData.TestC.sql (100%)
 rename {Public => __tests__}/Read-OleDbData.TestD.sql (100%)
 rename {Public => __tests__}/Read-OleDbData.TestE.sql (100%)
 rename {Public => __tests__}/Read-OleDbData.Tests.ps1 (99%)
 rename {Public => __tests__}/Read-OleDbData.xlsx (100%)

diff --git a/Public/Read-OleDbData.TestA.sql b/__tests__/Read-OleDbData.TestA.sql
similarity index 100%
rename from Public/Read-OleDbData.TestA.sql
rename to __tests__/Read-OleDbData.TestA.sql
diff --git a/Public/Read-OleDbData.TestB.sql b/__tests__/Read-OleDbData.TestB.sql
similarity index 100%
rename from Public/Read-OleDbData.TestB.sql
rename to __tests__/Read-OleDbData.TestB.sql
diff --git a/Public/Read-OleDbData.TestC.sql b/__tests__/Read-OleDbData.TestC.sql
similarity index 100%
rename from Public/Read-OleDbData.TestC.sql
rename to __tests__/Read-OleDbData.TestC.sql
diff --git a/Public/Read-OleDbData.TestD.sql b/__tests__/Read-OleDbData.TestD.sql
similarity index 100%
rename from Public/Read-OleDbData.TestD.sql
rename to __tests__/Read-OleDbData.TestD.sql
diff --git a/Public/Read-OleDbData.TestE.sql b/__tests__/Read-OleDbData.TestE.sql
similarity index 100%
rename from Public/Read-OleDbData.TestE.sql
rename to __tests__/Read-OleDbData.TestE.sql
diff --git a/Public/Read-OleDbData.Tests.ps1 b/__tests__/Read-OleDbData.Tests.ps1
similarity index 99%
rename from Public/Read-OleDbData.Tests.ps1
rename to __tests__/Read-OleDbData.Tests.ps1
index c2cf65b..c0b10bc 100644
--- a/Public/Read-OleDbData.Tests.ps1
+++ b/__tests__/Read-OleDbData.Tests.ps1
@@ -1,4 +1,3 @@
-. .\Read-OleDbData.ps1
 Describe "Read-OleDbData" {
     BeforeAll{
         $tfp = (Get-ChildItem Read-OleDbData.xlsx).fullname # test file path
diff --git a/Public/Read-OleDbData.xlsx b/__tests__/Read-OleDbData.xlsx
similarity index 100%
rename from Public/Read-OleDbData.xlsx
rename to __tests__/Read-OleDbData.xlsx

From e8c3d96f43c89a1c8f8a618eb010cc432cb0e7b4 Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Fri, 29 Oct 2021 21:58:05 -0400
Subject: [PATCH 03/24] Tests relocated and updated. All passing.

---
 .../Read-OleDbData.TestA.sql                  |   0
 .../Read-OleDbData.TestB.sql                  |   0
 .../Read-OleDbData.TestC.sql                  |   0
 .../Read-OleDbData.TestD.sql                  |   0
 .../Read-OleDbData.TestE.sql                  |   0
 .../Read-OleDbData.Tests.ps1                  |  42 ++++++++----------
 .../Read-OleDbData.xlsx                       | Bin
 7 files changed, 19 insertions(+), 23 deletions(-)
 rename __tests__/{ => Read-OleDbDataTests}/Read-OleDbData.TestA.sql (100%)
 rename __tests__/{ => Read-OleDbDataTests}/Read-OleDbData.TestB.sql (100%)
 rename __tests__/{ => Read-OleDbDataTests}/Read-OleDbData.TestC.sql (100%)
 rename __tests__/{ => Read-OleDbDataTests}/Read-OleDbData.TestD.sql (100%)
 rename __tests__/{ => Read-OleDbDataTests}/Read-OleDbData.TestE.sql (100%)
 rename __tests__/{ => Read-OleDbDataTests}/Read-OleDbData.Tests.ps1 (81%)
 rename __tests__/{ => Read-OleDbDataTests}/Read-OleDbData.xlsx (100%)

diff --git a/__tests__/Read-OleDbData.TestA.sql b/__tests__/Read-OleDbDataTests/Read-OleDbData.TestA.sql
similarity index 100%
rename from __tests__/Read-OleDbData.TestA.sql
rename to __tests__/Read-OleDbDataTests/Read-OleDbData.TestA.sql
diff --git a/__tests__/Read-OleDbData.TestB.sql b/__tests__/Read-OleDbDataTests/Read-OleDbData.TestB.sql
similarity index 100%
rename from __tests__/Read-OleDbData.TestB.sql
rename to __tests__/Read-OleDbDataTests/Read-OleDbData.TestB.sql
diff --git a/__tests__/Read-OleDbData.TestC.sql b/__tests__/Read-OleDbDataTests/Read-OleDbData.TestC.sql
similarity index 100%
rename from __tests__/Read-OleDbData.TestC.sql
rename to __tests__/Read-OleDbDataTests/Read-OleDbData.TestC.sql
diff --git a/__tests__/Read-OleDbData.TestD.sql b/__tests__/Read-OleDbDataTests/Read-OleDbData.TestD.sql
similarity index 100%
rename from __tests__/Read-OleDbData.TestD.sql
rename to __tests__/Read-OleDbDataTests/Read-OleDbData.TestD.sql
diff --git a/__tests__/Read-OleDbData.TestE.sql b/__tests__/Read-OleDbDataTests/Read-OleDbData.TestE.sql
similarity index 100%
rename from __tests__/Read-OleDbData.TestE.sql
rename to __tests__/Read-OleDbDataTests/Read-OleDbData.TestE.sql
diff --git a/__tests__/Read-OleDbData.Tests.ps1 b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
similarity index 81%
rename from __tests__/Read-OleDbData.Tests.ps1
rename to __tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
index c0b10bc..7c2b141 100644
--- a/__tests__/Read-OleDbData.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
@@ -1,10 +1,13 @@
-Describe "Read-OleDbData" {
-    BeforeAll{
-        $tfp = (Get-ChildItem Read-OleDbData.xlsx).fullname # test file path
+#Requires -Modules Pester
+Import-Module $PSScriptRoot\..\..\ImportExcel.psd1 -Force
+Describe "All tests for Read-OleDbData" -Tag "Read-OleDbData" {
+    BeforeAll {
+        $scriptPath = $PSScriptRoot
+        $tfp = "$scriptPath\Read-OleDbData.xlsx"
         $cs = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$tfp;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
     }
     Context "When Read-OleDbData.xlsx and we want sheet1 a1" {
-        BeforeAll{
+        BeforeAll {
             $sql = "select ROUND(F1) as [A1] from [sheet1`$A1:A1]"
             $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
         }
@@ -19,7 +22,7 @@ Describe "Read-OleDbData" {
         }
     }
     Context "When Read-OleDbData.xlsx and we want sheet2 a1" {
-        BeforeAll{
+        BeforeAll {
             $sql = "select ROUND(F1) as [A1] from [sheet2`$A1:A1]"
             $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
         }
@@ -34,9 +37,8 @@ Describe "Read-OleDbData" {
         }
     }
     Context "When Read-OleDbData.xlsx and we want a1 on sheet3 and sql is in a file" {
-        BeforeAll{
-            $sql = Get-Content .\Read-OleDbData.TestA.sql -raw
-            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
+        BeforeAll {
+            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement (Get-Content "$scriptPath\Read-OleDbData.TestA.sql" -raw)
         }
         It "should be PSCustomObject" {
             $Results.GetType().Name | Should -Be 'PSCustomObject'
@@ -49,9 +51,8 @@ Describe "Read-OleDbData" {
         }
     }
     Context "When Read-OleDbData.xlsx, we want a1 on sheets1-7, want to validate the values match properly, and sql is in a file" {
-        BeforeAll{
-            $sql = Get-Content .\Read-OleDbData.TestB.sql -raw
-            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
+        BeforeAll {
+            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement (Get-Content "$scriptPath\Read-OleDbData.TestB.sql" -raw)
         }
         It "should be PSCustomObject" {
             $Results[0].GetType().Name | Should -Be 'PSCustomObject'
@@ -65,23 +66,20 @@ Describe "Read-OleDbData" {
         }
     }
     Context "When Read-OleDbData.xlsx, select range sheet1 A1:E10, and sql is in a file" {
-        #note, this spreadsheet doesn't have the fields populated other than A1, so it will, correctly, return only one value
-        BeforeAll{
-            $sql = Get-Content .\Read-OleDbData.TestC.sql -raw
-            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
+        BeforeAll {
+            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement (Get-Content "$scriptPath\Read-OleDbData.TestC.sql" -raw)
         }
         It "should be PSCustomObject" {
             $Results.GetType().Name | Should -Be 'PSCustomObject'
         }
+        #note, this spreadsheet doesn't have the fields populated other than A1, so it will, correctly, return only one value
         It "should have length of 1" {
             @($Results).length | Should -Be 1
         }
     }
     Context "When Read-OleDbData.xlsx, select a1 from all sheets as a single record, and sql is in a file" {
-        #note, this spreadsheet doesn't have the fields populated other than A1, so it will, correctly, return only one value
-        BeforeAll{
-            $sql = Get-Content .\Read-OleDbData.TestD.sql -raw
-            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
+        BeforeAll {
+            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement (Get-Content "$scriptPath\Read-OleDbData.TestD.sql" -raw)
         }
         It "should be PSCustomObject" {
             $Results.GetType().Name | Should -Be 'PSCustomObject'
@@ -94,10 +92,8 @@ Describe "Read-OleDbData" {
         }
     }
     Context "When Read-OleDbData.xlsx, select a1 from all sheets as a single record multiple times to create a range, and sql is in a file" {
-        #note, this spreadsheet doesn't have the fields populated other than A1, so it will, correctly, return only one value
-        BeforeAll{
-            $sql = Get-Content .\Read-OleDbData.TestE.sql -raw
-            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
+        BeforeAll {
+            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement (Get-Content "$scriptPath\Read-OleDbData.TestE.sql" -raw)
         }
         It "should be Object[]" {
             $Results.GetType().Name | Should -Be 'Object[]'
diff --git a/__tests__/Read-OleDbData.xlsx b/__tests__/Read-OleDbDataTests/Read-OleDbData.xlsx
similarity index 100%
rename from __tests__/Read-OleDbData.xlsx
rename to __tests__/Read-OleDbDataTests/Read-OleDbData.xlsx

From 91a124040890cdfc3083e1cf0d88f0c30ead9802 Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Sat, 30 Oct 2021 11:20:27 -0400
Subject: [PATCH 04/24] add skip test check for ace

---
 .../Read-OleDbData.Tests.ps1                  | 120 ++++++------------
 1 file changed, 42 insertions(+), 78 deletions(-)

diff --git a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
index 7c2b141..6af536e 100644
--- a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
@@ -1,108 +1,72 @@
 #Requires -Modules Pester
 Import-Module $PSScriptRoot\..\..\ImportExcel.psd1 -Force
-Describe "All tests for Read-OleDbData" -Tag "Read-OleDbData" {
-    BeforeAll {
-        $scriptPath = $PSScriptRoot
-        $tfp = "$scriptPath\Read-OleDbData.xlsx"
-        $cs = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$tfp;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
+$scriptPath = $PSScriptRoot
+$tfp = "$scriptPath\Read-OleDbData.xlsx"
+$cs = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$tfp;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
+$ACEnotWorking = $false
+try {
+    $Results = Read-OleDbData -ConnectionString $cs -SqlStatement "select 1"             
+}
+catch {
+    $ACEnotWorking = $true
+}
+Describe "Read-OleDbData" -Tag "Read-OleDbData" {
+    $PSDefaultParameterValues = @{ 'It:Skip' = $ACEnotWorking }
+    Context "Basic Tests" {
+        It "should be able to open spreadsheet" {
+            $null = Read-OleDbData -ConnectionString $cs -SqlStatement "select 1"
+            $true | Should -Be $true
+        }
+        It "should return PSCustomObject for single result" {
+            #multiple records will come back as Object[], but not going to test for that
+            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement "select 1"
+            $Results.GetType().Name | Should -Be 'PSCustomObject'
+        }
     }
-    Context "When Read-OleDbData.xlsx and we want sheet1 a1" {
-        BeforeAll {
+    Context "Sheet1`$A1" {
+        It "Should return 1 result with a value of 1" {
             $sql = "select ROUND(F1) as [A1] from [sheet1`$A1:A1]"
             $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
-        }
-        It "should be PSCustomObject" {
-            $Results.GetType().Name | Should -Be 'PSCustomObject'
-        }
-        It "should have length of 1" {
-            @($Results).length | Should -Be 1
-        }
-        It "should be value of 1" {
-            $Results.A1 | Should -Be 1
+            @($Results).length + $Results.A1 | Should -Be 2
         }
     }
-    Context "When Read-OleDbData.xlsx and we want sheet2 a1" {
-        BeforeAll {
+    Context "Sheet2`$A1" {
+        It "Should return 1 result with value of 2" {
             $sql = "select ROUND(F1) as [A1] from [sheet2`$A1:A1]"
             $Results = Read-OleDbData -ConnectionString $cs -SqlStatement $sql
-        }
-        It "should be PSCustomObject" {
-            $Results.GetType().Name | Should -Be 'PSCustomObject'
-        }
-        It "should have length of 1" {
-            @($Results).length | Should -Be 1
-        }
-        It "should be value of 2" {
-            $Results.A1 | Should -Be 2
+            @($Results).length + $Results.A1 | Should -Be 3
         }
     }
-    Context "When Read-OleDbData.xlsx and we want a1 on sheet3 and sql is in a file" {
-        BeforeAll {
+    Context "Sheet3`$A1, Sql from file" {
+        It "Should return 1 result with value of 3" {
             $Results = Read-OleDbData -ConnectionString $cs -SqlStatement (Get-Content "$scriptPath\Read-OleDbData.TestA.sql" -raw)
-        }
-        It "should be PSCustomObject" {
-            $Results.GetType().Name | Should -Be 'PSCustomObject'
-        }
-        It "should have length of 1" {
-            @($Results).length | Should -Be 1
-        }
-        It "should be value of 2" {
-            $Results.A1 | Should -Be 3
+            @($Results).length + $Results.A1 | Should -Be 4
         }
     }
-    Context "When Read-OleDbData.xlsx, we want a1 on sheets1-7, want to validate the values match properly, and sql is in a file" {
-        BeforeAll {
+    Context "Sheets[1-7]`$A1, Sql from file" {
+        It "Should return 7 result with where sum values 1-6 = value 7" {
             $Results = Read-OleDbData -ConnectionString $cs -SqlStatement (Get-Content "$scriptPath\Read-OleDbData.TestB.sql" -raw)
-        }
-        It "should be PSCustomObject" {
-            $Results[0].GetType().Name | Should -Be 'PSCustomObject'
-        }
-        It "should have length of 7" {
-            @($Results).length | Should -Be 7
-        }
-        It "should have data where sum of all initial records match the value of the last record" {
             $a = $Results.A1
-            ($a[0..5] | Measure-Object -sum).sum | Should -Be $a[6]
+            $a.length + ($a[0..5] | Measure-Object -sum).sum | Should -Be (7+$a[6])
         }
     }
-    Context "When Read-OleDbData.xlsx, select range sheet1 A1:E10, and sql is in a file" {
-        BeforeAll {
-            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement (Get-Content "$scriptPath\Read-OleDbData.TestC.sql" -raw)
-        }
-        It "should be PSCustomObject" {
-            $Results.GetType().Name | Should -Be 'PSCustomObject'
-        }
+    Context "Sheet1`$:A1:E10, Sql from file" {
         #note, this spreadsheet doesn't have the fields populated other than A1, so it will, correctly, return only one value
-        It "should have length of 1" {
-            @($Results).length | Should -Be 1
+        It "Should return 1 result with value of 1" {
+            $Results = Read-OleDbData -ConnectionString $cs -SqlStatement (Get-Content "$scriptPath\Read-OleDbData.TestC.sql" -raw)
+            @($Results).length + $Results.F1 | Should -Be 2
         }
     }
     Context "When Read-OleDbData.xlsx, select a1 from all sheets as a single record, and sql is in a file" {
-        BeforeAll {
+        It "should return one row with 8 columns" {
             $Results = Read-OleDbData -ConnectionString $cs -SqlStatement (Get-Content "$scriptPath\Read-OleDbData.TestD.sql" -raw)
-        }
-        It "should be PSCustomObject" {
-            $Results.GetType().Name | Should -Be 'PSCustomObject'
-        }
-        It "should have length of 1" {
-            @($Results).length | Should -Be 1
-        }
-        It "should have 8 properties" {
-            @($Results.psobject.Properties).length | Should -Be 8
+            @($Results).length + @($Results.psobject.Properties).length | Should -Be 9
         }
     }
     Context "When Read-OleDbData.xlsx, select a1 from all sheets as a single record multiple times to create a range, and sql is in a file" {
-        BeforeAll {
+        It "should return 4 records with 5 columns" {
             $Results = Read-OleDbData -ConnectionString $cs -SqlStatement (Get-Content "$scriptPath\Read-OleDbData.TestE.sql" -raw)
-        }
-        It "should be Object[]" {
-            $Results.GetType().Name | Should -Be 'Object[]'
-        }
-        It "should have length of 4" {
-            @($Results).length | Should -Be 4
-        }
-        It "should have 5 properties on first record" {
-            @($Results[0].psobject.Properties).length | Should -Be 5
+            @($Results).length + @($Results[0].psobject.Properties).length | Should -Be 9
         }
     }
 }

From 5e7b404dafe6c47c636002e9cda9ffc7322a63e6 Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Wed, 3 Nov 2021 10:35:22 -0400
Subject: [PATCH 05/24] Add helper method

---
 Public/Read-OleDbData.ps1                     |  3 +-
 Public/Read-XlsxUsingOleDb.ps1                | 47 +++++++++++++++++++
 .../Read-OleDbData.Tests.ps1                  |  4 +-
 .../Read-XlsxUsingOleDb.Tests..ps1            | 20 ++++++++
 4 files changed, 70 insertions(+), 4 deletions(-)
 create mode 100644 Public/Read-XlsxUsingOleDb.ps1
 create mode 100644 __tests__/Read-OleDbDataTests/Read-XlsxUsingOleDb.Tests..ps1

diff --git a/Public/Read-OleDbData.ps1 b/Public/Read-OleDbData.ps1
index 579d904..ccd0fdc 100644
--- a/Public/Read-OleDbData.ps1
+++ b/Public/Read-OleDbData.ps1
@@ -26,7 +26,6 @@ function Read-OleDbData {
             ConnectionString = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=file.xlsx;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
         }
         $Results = Read-OleDbData @ReadDataArgs
-        
     #>
     param(
         [Parameter(Mandatory)]
@@ -61,7 +60,7 @@ function Read-OleDbData {
     #      and get rid of the extra fields that come back with the datatable or rows.
     
     $DataTable = new-object System.Data.DataTable
-    $DataAdapter = new-object System.Data.OleDb.OleDbDataAdapter $SqlStatement,$ConnectionString
+    $DataAdapter = new-object System.Data.OleDb.OleDbDataAdapter $SqlStatement, $ConnectionString
     $null = $DataAdapter.Fill($DataTable)
     $null = $DataAdapter.Dispose()
     $DataTable.Rows | Select-Object $DataTable.Columns.ColumnName
diff --git a/Public/Read-XlsxUsingOleDb.ps1 b/Public/Read-XlsxUsingOleDb.ps1
new file mode 100644
index 0000000..0d75f41
--- /dev/null
+++ b/Public/Read-XlsxUsingOleDb.ps1
@@ -0,0 +1,47 @@
+#Requires -Version 5
+function Read-XlsxUsingOleDb {
+    <#
+        .SYNOPSIS
+        Helper method for executing Read-OleDbData with some basic defaults.
+
+        For additional help, see documentation for Read-OleDbData cmdlet.
+
+        .DESCRIPTION
+        Uses Read-OleDbData to execute a sql statement against a xlsx file. For finer grained control over the interaction, you may use that cmdlet. This cmdlet assumes a file path will be passed in and the connection string will be built with no headers and treating all results as text.
+
+        Running this command is equivalent to running the following:
+
+        $FullName = (Get-ChildItem $Path).FullName
+        Read-OleDbData `
+            -ConnectionString "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$FullName;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'" `
+            -SqlStatement $Query
+
+        .EXAMPLE
+        Read-XlsxUsingOleDb .\test.xlsx 'select ROUND(F1) as [A1] from [sheet3$A1:A1]'
+
+        .EXAMPLE
+        $Path = (Get-ChildItem 'test.xlsx').FullName
+        $Query = "select ROUND(F1) as [A] from [sheet1$A1:A1]"
+        Read-XlsxUsingOleDb -Path $Path -Query $Query
+
+        .EXAMPLE
+        $ReadDataArgs = @{
+            Path = .\test.xlsx
+            Query = Get-Content query.sql -Raw
+        }
+        $Results = Read-XlsxUsingOleDb @ReadDataArgs
+    #>
+    param(
+        #The path to the file to open.
+        [Parameter(Mandatory)]
+        [ValidateNotNullOrEmpty()]
+        [String] $Path, # var name consistent with Import-Excel
+        [Parameter(Mandatory)]
+        [ValidateNotNullOrEmpty()]
+        [String] $Query # var name consistent with Invoke-Sqlcmd
+    )
+    $FullName = (Get-ChildItem $Path).FullName
+    Read-OleDbData `
+        -ConnectionString "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$FullName;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'" `
+        -SqlStatement $Query
+}
\ No newline at end of file
diff --git a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
index 6af536e..1abb368 100644
--- a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
@@ -47,7 +47,7 @@ Describe "Read-OleDbData" -Tag "Read-OleDbData" {
         It "Should return 7 result with where sum values 1-6 = value 7" {
             $Results = Read-OleDbData -ConnectionString $cs -SqlStatement (Get-Content "$scriptPath\Read-OleDbData.TestB.sql" -raw)
             $a = $Results.A1
-            $a.length + ($a[0..5] | Measure-Object -sum).sum | Should -Be (7+$a[6])
+            $a.length + ($a[0..5] | Measure-Object -sum).sum | Should -Be (7 + $a[6])
         }
     }
     Context "Sheet1`$:A1:E10, Sql from file" {
@@ -69,4 +69,4 @@ Describe "Read-OleDbData" -Tag "Read-OleDbData" {
             @($Results).length + @($Results[0].psobject.Properties).length | Should -Be 9
         }
     }
-}
+}
\ No newline at end of file
diff --git a/__tests__/Read-OleDbDataTests/Read-XlsxUsingOleDb.Tests..ps1 b/__tests__/Read-OleDbDataTests/Read-XlsxUsingOleDb.Tests..ps1
new file mode 100644
index 0000000..d88ec4b
--- /dev/null
+++ b/__tests__/Read-OleDbDataTests/Read-XlsxUsingOleDb.Tests..ps1
@@ -0,0 +1,20 @@
+#Requires -Modules Pester
+$scriptPath = $PSScriptRoot
+Import-Module $scriptPath\..\..\ImportExcel.psd1 -Force
+$tfp = "$scriptPath\Read-OleDbData.xlsx"
+$ACEnotWorking = $false
+try {
+    $Results = Read-XlsxUsingOleDb $tfp "select 1"      
+}
+catch {
+    $ACEnotWorking = $true
+}
+Describe "Read-XlsxUsingOleDb" -Tag "Read-XlsxUsingOleDb" {
+    $PSDefaultParameterValues = @{ 'It:Skip' = $ACEnotWorking }
+    Context "Sheet1`$A1" {
+        It "Should return 1 result with a value of 1" {
+            $Results = Read-XlsxUsingOleDb $tfp "select ROUND(F1) as [A1] from [sheet1`$A1:A1]"
+            @($Results).length + $Results.A1 | Should -Be 2
+        }
+    }
+}

From 85f2433ffc99b240c6e75b119b731c0a0be7d279 Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Thu, 4 Nov 2021 10:44:14 -0400
Subject: [PATCH 06/24] Rename helper function to make it more helpful. =)

---
 Public/{Read-XlsxUsingOleDb.ps1 => Invoke-ExcelQuery.ps1} | 8 ++++----
 ...xUsingOleDb.Tests..ps1 => Invoke-ExcelQuery.Tests.ps1} | 6 +++---
 2 files changed, 7 insertions(+), 7 deletions(-)
 rename Public/{Read-XlsxUsingOleDb.ps1 => Invoke-ExcelQuery.ps1} (89%)
 rename __tests__/Read-OleDbDataTests/{Read-XlsxUsingOleDb.Tests..ps1 => Invoke-ExcelQuery.Tests.ps1} (67%)

diff --git a/Public/Read-XlsxUsingOleDb.ps1 b/Public/Invoke-ExcelQuery.ps1
similarity index 89%
rename from Public/Read-XlsxUsingOleDb.ps1
rename to Public/Invoke-ExcelQuery.ps1
index 0d75f41..00fb4a0 100644
--- a/Public/Read-XlsxUsingOleDb.ps1
+++ b/Public/Invoke-ExcelQuery.ps1
@@ -1,5 +1,5 @@
 #Requires -Version 5
-function Read-XlsxUsingOleDb {
+function Invoke-ExcelQuery {
     <#
         .SYNOPSIS
         Helper method for executing Read-OleDbData with some basic defaults.
@@ -17,7 +17,7 @@ function Read-XlsxUsingOleDb {
             -SqlStatement $Query
 
         .EXAMPLE
-        Read-XlsxUsingOleDb .\test.xlsx 'select ROUND(F1) as [A1] from [sheet3$A1:A1]'
+        Invoke-ExcelQuery .\test.xlsx 'select ROUND(F1) as [A1] from [sheet3$A1:A1]'
 
         .EXAMPLE
         $Path = (Get-ChildItem 'test.xlsx').FullName
@@ -29,7 +29,7 @@ function Read-XlsxUsingOleDb {
             Path = .\test.xlsx
             Query = Get-Content query.sql -Raw
         }
-        $Results = Read-XlsxUsingOleDb @ReadDataArgs
+        $Results = Invoke-ExcelQuery @ReadDataArgs
     #>
     param(
         #The path to the file to open.
@@ -41,7 +41,7 @@ function Read-XlsxUsingOleDb {
         [String] $Query # var name consistent with Invoke-Sqlcmd
     )
     $FullName = (Get-ChildItem $Path).FullName
-    Read-OleDbData `
+    Invoke-ExcelQuery `
         -ConnectionString "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$FullName;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'" `
         -SqlStatement $Query
 }
\ No newline at end of file
diff --git a/__tests__/Read-OleDbDataTests/Read-XlsxUsingOleDb.Tests..ps1 b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
similarity index 67%
rename from __tests__/Read-OleDbDataTests/Read-XlsxUsingOleDb.Tests..ps1
rename to __tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
index d88ec4b..3d0842f 100644
--- a/__tests__/Read-OleDbDataTests/Read-XlsxUsingOleDb.Tests..ps1
+++ b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
@@ -4,16 +4,16 @@ Import-Module $scriptPath\..\..\ImportExcel.psd1 -Force
 $tfp = "$scriptPath\Read-OleDbData.xlsx"
 $ACEnotWorking = $false
 try {
-    $Results = Read-XlsxUsingOleDb $tfp "select 1"      
+    $Results = Invoke-ExcelQuery $tfp "select 1"      
 }
 catch {
     $ACEnotWorking = $true
 }
-Describe "Read-XlsxUsingOleDb" -Tag "Read-XlsxUsingOleDb" {
+Describe "Invoke-ExcelQuery" -Tag "Invoke-ExcelQuery" {
     $PSDefaultParameterValues = @{ 'It:Skip' = $ACEnotWorking }
     Context "Sheet1`$A1" {
         It "Should return 1 result with a value of 1" {
-            $Results = Read-XlsxUsingOleDb $tfp "select ROUND(F1) as [A1] from [sheet1`$A1:A1]"
+            $Results = Invoke-ExcelQuery $tfp "select ROUND(F1) as [A1] from [sheet1`$A1:A1]"
             @($Results).length + $Results.A1 | Should -Be 2
         }
     }

From 7a8bbb97710256a4f2eb01b0d234e0671ae3ac3b Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Thu, 4 Nov 2021 20:54:51 -0400
Subject: [PATCH 07/24] add to exported functions, formatting, fix typo.

---
 ImportExcel.psd1                                       | 2 ++
 Public/Invoke-ExcelQuery.ps1                           | 2 +-
 __tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 | 4 ++--
 3 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/ImportExcel.psd1 b/ImportExcel.psd1
index 8a267c3..8143435 100644
--- a/ImportExcel.psd1
+++ b/ImportExcel.psd1
@@ -64,6 +64,7 @@ Check out the How To Videos https://www.youtube.com/watch?v=U3Ne_yX4tYo&list=PL5
         'Import-USPS',
         'Invoke-AllTests',
         'Invoke-Sum',
+        'Invoke-ExcelQuery',
         'Join-Worksheet',
         'LineChart',
         'Merge-MultipleSheets',
@@ -80,6 +81,7 @@ Check out the How To Videos https://www.youtube.com/watch?v=U3Ne_yX4tYo&list=PL5
         'Pivot',
         'Read-Clipboard',
         'ReadClipboardImpl',
+        'Read-OleDbData',
         'Remove-Worksheet',
         'Select-Worksheet',
         'Send-SQLDataToExcel',
diff --git a/Public/Invoke-ExcelQuery.ps1 b/Public/Invoke-ExcelQuery.ps1
index 00fb4a0..f799a11 100644
--- a/Public/Invoke-ExcelQuery.ps1
+++ b/Public/Invoke-ExcelQuery.ps1
@@ -41,7 +41,7 @@ function Invoke-ExcelQuery {
         [String] $Query # var name consistent with Invoke-Sqlcmd
     )
     $FullName = (Get-ChildItem $Path).FullName
-    Invoke-ExcelQuery `
+    Read-OleDbData `
         -ConnectionString "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$FullName;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'" `
         -SqlStatement $Query
 }
\ No newline at end of file
diff --git a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
index 1abb368..500772f 100644
--- a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
@@ -1,9 +1,9 @@
 #Requires -Modules Pester
-Import-Module $PSScriptRoot\..\..\ImportExcel.psd1 -Force
 $scriptPath = $PSScriptRoot
+Import-Module $scriptPath\..\..\ImportExcel.psd1 -Force
 $tfp = "$scriptPath\Read-OleDbData.xlsx"
-$cs = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$tfp;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
 $ACEnotWorking = $false
+$cs = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$tfp;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
 try {
     $Results = Read-OleDbData -ConnectionString $cs -SqlStatement "select 1"             
 }

From eabaab79c5e4431d522a8d0542f8549947a5747a Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 14:43:57 -0500
Subject: [PATCH 08/24] add ace check

---
 Public/Invoke-ExcelQuery.ps1 | 11 +++++++++++
 1 file changed, 11 insertions(+)

diff --git a/Public/Invoke-ExcelQuery.ps1 b/Public/Invoke-ExcelQuery.ps1
index f799a11..3b8817e 100644
--- a/Public/Invoke-ExcelQuery.ps1
+++ b/Public/Invoke-ExcelQuery.ps1
@@ -16,6 +16,10 @@ function Invoke-ExcelQuery {
             -ConnectionString "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$FullName;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'" `
             -SqlStatement $Query
 
+        Note that this command uses the MICROSOFT.ACE.OLEDB provider and will not work without it.
+        
+        If needed, please download the appropriate package from https://www.microsoft.com/en-us/download/details.aspx?id=54920.
+        
         .EXAMPLE
         Invoke-ExcelQuery .\test.xlsx 'select ROUND(F1) as [A1] from [sheet3$A1:A1]'
 
@@ -40,6 +44,13 @@ function Invoke-ExcelQuery {
         [ValidateNotNullOrEmpty()]
         [String] $Query # var name consistent with Invoke-Sqlcmd
     )
+
+    $IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -like "Microsoft.ACE.OLEDB*")
+    if($IsMissingACE){
+        Write-Error "MICROSOFT.ACE.OLEDB is missing! Please see https://www.microsoft.com/en-us/download/details.aspx?id=54920"
+        return
+    }
+
     $FullName = (Get-ChildItem $Path).FullName
     Read-OleDbData `
         -ConnectionString "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$FullName;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'" `

From aa1b0018abc216b56ab7c9ea514cfc90a108d18c Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 15:14:46 -0500
Subject: [PATCH 09/24] Add examples

---
 Examples/InvokeExcelQuery/Examples.ps1   |  13 +++++++++++++
 Examples/InvokeExcelQuery/testOleDb.xlsx | Bin 0 -> 2955 bytes
 2 files changed, 13 insertions(+)
 create mode 100644 Examples/InvokeExcelQuery/Examples.ps1
 create mode 100644 Examples/InvokeExcelQuery/testOleDb.xlsx

diff --git a/Examples/InvokeExcelQuery/Examples.ps1 b/Examples/InvokeExcelQuery/Examples.ps1
new file mode 100644
index 0000000..c565292
--- /dev/null
+++ b/Examples/InvokeExcelQuery/Examples.ps1
@@ -0,0 +1,13 @@
+try {Import-Module $PSScriptRoot\..\..\ImportExcel.psd1} catch {throw ; return}
+
+$queries = 
+  'select * from [sheet1$A:A]',
+  'select * from [sheet1$]',
+  'select * from [sheet1$A2:E11]',
+  'select F2,F5 from [sheet1$A2:E11]',
+  'select * from [sheet1$A2:E11] where F2 = "Grocery"',
+  'select F2 as [Category], F5 as [Discount], F5*2 as [DiscountPlus] from [sheet1$A2:E11]'
+
+foreach ($query in $queries) {
+  Invoke-ExcelQuery .\testOleDb.xlsx $query| Format-Table  
+}
diff --git a/Examples/InvokeExcelQuery/testOleDb.xlsx b/Examples/InvokeExcelQuery/testOleDb.xlsx
new file mode 100644
index 0000000000000000000000000000000000000000..944751c9ec1543e296e4688272d7a6b35e089c93
GIT binary patch
literal 2955
zcmZ{m2{@E%8^=eOtTpzXP$$g6Oty|BTWF-lU^=o)WNc$8GX|4{VvucYW6%0kiex`Z
znGA8#XD3TYc12kdlbm-t-}RY%-?`uGn&+Bp=6SC7{@wrk`QJzjb`D_>7w~A9xLHq`
zmpu|^1A#QyK_DRz2xM>U=S#r)5->KwSFm_TwICnwn&eIdI~>xA!M59GRm++bkI0tF
z+E(Z{WpogYR}(v`YyJGRp$a7tvZSoT;~g8@TpAeJr?@W}?X6w0nK`lGn+dH^me8?*
z#1#8(h%s!!r3o!fq5H!zazZF;H};xB_j7>=%2GFbdj?Fn<n|Cd6*qQ|gTyOkGuS&H
z-vft%^FB?;i>_Y#aA~&QE`IrD0QAricdaWlRLfb-gJgw-)Mm$RXJ5Gv?v5#56E{K)
znUp=ZpqYP9^RON@v3+3Txq{1}UHmT#x$26^C{tLUVr5_@dY%>qU*kM>DqOvMVWQ8h
zV`M|TuF^}4O<ZANv+kQpaw(Q*2$?K0)-{8xP~(>5i-M;e(!c4^I-~p`w$#wi+(?W4
zT&(zzKY`5{Uc6ln06hx8<poxZKh_(suEzX~@1z}wfboGRF4&Q%;rlQ1-MkHd=k<wg
z1`TE#z-IIKEho6>1$l36FnTMUW@1iulfxf|DZC0Vur1z?;}{)&wZ!$SiDyktTVa?8
zd!KD^%b8Y!;1-QDollBm4c;I$Yq);=RT9NGTU%z+u2qxNoNH60*|rfJI`O?XtgKy9
zwMnM|G*EtE=kfhTaHKmbQNrh&)#o(x$~ACN7<N*9<$bP4hak^WmCYI20zWfN)?|K8
z^W?2pOM^gqC?L&)z#8PO9_Z(fbMf=TF|`wa{<T*#j4xEAh_OH*A2UjpZLlb)pyda4
zQlzX$cq*%w-je1N*G1fPZhfpz(Vu5f2+zl|KGp?J^8R*_NA=+a5iY(glGHiXv+a6~
zO|>{R`t13_s<s6P&ez%8^I=IG$>7mstSMin2HmM-bmdGk9Q__T-+tMA<&RSB<nH^a
zIxVyWftl(Z*A|M^`c;FoW3R28hCq(i<+>N2(#I7opss^$r$(C_Yy?tYit6(?-3;pZ
za{aq3$DlU**oFi(y5(+K@A9u5Q=@9lbyp5)orv`s8r=plnXqR5Gm~(a<iY`fh2}mG
zNb+wce^;NC%nv1$Q97A;$e5P7XUFF+#aY!+;d3=NWDVXRq_f_%_6$#d#A%~ZN|(ub
zw_Mk@^&d`e;iUUwj%~ojJxV2oIi(!sblJ(<FMFX%qI?I);%!YPe2e}57VHQ+=kcTH
zNP#zDVrTp=R;_C)T!33Csrz&&B0TYTg@zT0#bN>5g{wJib!Tg?f>&C#^1oOY?o?bJ
z`I0_Xv@NaNmdkZLVY4mIoJSaM<}#t{vhA2viZmRj+UcR9u#$gboGe4Po=@J9lU#<U
zjGR#;=h=6>e12?eX7~XwlMX9`pXuO@tb(inGVi!RAn{$r;61Qd0$%;c0{<bM^iKb5
z7;x=GzH^e7|4xEszDe@*%atYF{aySQa4<&#!}sJGWs4{6Ow>cPYmQXK_|o^a@ak`j
zRh?^J-`>d)D`-7;z@93wpzCsPx<D$ye&d)HMsP#EvmA9Yu?&(@G~HZpWnY*3uhrJo
z)&`O5T7to%Hxsl_>;1)k{oN^+z5Rtr>m_2%Q=6%#d4hMnddgaBPb4nkYC8KQI}tWy
ziGov&=?f2hudIIPnqSZ%nZ>v*>Q`E}XDkK<Yd^R_oYalL&DN%3Q2OJ(&tz)Va%=LA
zgl0XtapB*Fn*Gz7+ew3u3QyFg5S2q7VQj3^+!0PPAGv$=);~^K3B}=kROQxtsX1<A
zx0kG+D-&xYGGiOdK3Sf2hSuL#@;P0hq-WWf&){v?Nw18kUM=&NkBAz&3W3BTgrh*+
zFT{_0)>X!s|CXePBaSM>LzFva3=!fh<*+}C0@6nfrWfzgA|#vKxU?#e6!mKvGAl_i
zZx!Lg+(!I-HaBFn9FWF`4nn*<3JTqTAGg?IP`7wE1%`#O9{H64eIADKmo#okzC9f|
z#CD!ryFa?<68B9)u7=a`%k^$_iSg?U*oipv2q%Oy-Krvv-z+7d$5q}^e=PUORCy6B
zn;{U><-)6E{(0%VeifsaQ!8>@&&X2MCgdE=TU6-jb6)Yp$M^3WN|vP@+tTM(a?p|G
z=!<AEOQUr5q!z#V-I!)|yC)j8LgTu0DFpteOVaU+&Esh2@C*&Xml@(JAt4ZPD@MqA
z(+8jR#Bk(!)H$QqMIoDZ?K|vDS+a`nXIa{~H*Yusm)ImAP5!@Si6;bmWARKaMxrc$
zK7};Kc0Ic)?V;R46y-G5DM(UlNb_kIog-@=AU+y6o!AGpk<+<~kbfX?vN2J!XJhul
z<2%I$N4;bQOX-_#w*_p=mg*NQ^!7cuMrV{>f$@^0l{lw1|4a)4!Oq*w42L5owTF?@
zff=>QVVGfkd@HKwk3-1{BR+KpGY>nwD@7JPJaW8m!OAe^x+_}b>;vA^_Z(lR4iXez
zC<ePKD|qh9i<yX@7I-bFBRZSr5M9-O;Ap-IA6q`UKQui3yN`A?9RY3lYBI@B@@vmo
zUK))3J;cArXV(3XXQmOQMo6%sbMPrTxSG498+0+M{C)9YZ1fxhYA?C`7A2qXPG)S#
zwuz3EoU;&fmx!vUrg^AZ{Gzi)7l+H44k7Jd&hk~);12ic&Ue!XK0W9+us6Rm?;nOI
z>b2W`lQ{j($0CF)(`9-j&H;2`$FV(|D8yK$RK9XA3;l7KveN&#E@Qzeja@)Jh(K+G
z|5FzaXMe1#HNoH0*ZoI%ut~IY9P$v88<AH6B~E(xfuyR+TT|c*`yn3g=J|#VFY9lp
zd+F6nF9rnIqDUK=DPO}(ALmnMv(NAtV2tQPZG+S&RTj$8GGHoC1jX!-G7&BGc<zz8
zsanVSMdcM<SF>ys8)Lex`a}IIZD=XKLq2C19wpXWZ98;2^#xUAs12%&j$4L)*deG#
zqpPlavke<}#g?JR=fsv}njMBer=EJkLEyQACQApy#!a{8<zDb&C+5ic476TciB3%e
z*t0@TFvyp4=g58iob7!~rLyLYv|!t(3S|FJLmPmFCH~I{xAy>6KW7dnyI}`lW*JrX
zZe%q*rrqy`0nj&l%zy7@R!?C1;%?Xg(<iX`A6#<pIjl|0bdcTf=lJhS{EwgPJ%v?r
w%!c0$?)^Vb*=_&5TUl$v)Z}gy0Hy{cg5Rr9qy;CK`LF%J^8!Fz#m)To7f7(cRsaA1

literal 0
HcmV?d00001


From 676ecc17e05f8a950044070796900d65e7deb99e Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 15:25:26 -0500
Subject: [PATCH 10/24] update tests to match

---
 .../Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1   | 11 ++++-------
 .../Read-OleDbDataTests/Read-OleDbData.Tests.ps1      | 11 ++++-------
 2 files changed, 8 insertions(+), 14 deletions(-)

diff --git a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1 b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
index 3d0842f..4f428e2 100644
--- a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
@@ -2,15 +2,12 @@
 $scriptPath = $PSScriptRoot
 Import-Module $scriptPath\..\..\ImportExcel.psd1 -Force
 $tfp = "$scriptPath\Read-OleDbData.xlsx"
-$ACEnotWorking = $false
-try {
-    $Results = Invoke-ExcelQuery $tfp "select 1"      
-}
-catch {
-    $ACEnotWorking = $true
+$IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -like "Microsoft.ACE.OLEDB*")
+if($IsMissingACE){
+    Write-Host "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped. Please see https://www.microsoft.com/en-us/download/details.aspx?id=54920"
 }
 Describe "Invoke-ExcelQuery" -Tag "Invoke-ExcelQuery" {
-    $PSDefaultParameterValues = @{ 'It:Skip' = $ACEnotWorking }
+    $PSDefaultParameterValues = @{ 'It:Skip' = $IsMissingACE }
     Context "Sheet1`$A1" {
         It "Should return 1 result with a value of 1" {
             $Results = Invoke-ExcelQuery $tfp "select ROUND(F1) as [A1] from [sheet1`$A1:A1]"
diff --git a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
index 500772f..db8ff73 100644
--- a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
@@ -2,16 +2,13 @@
 $scriptPath = $PSScriptRoot
 Import-Module $scriptPath\..\..\ImportExcel.psd1 -Force
 $tfp = "$scriptPath\Read-OleDbData.xlsx"
-$ACEnotWorking = $false
 $cs = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$tfp;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
-try {
-    $Results = Read-OleDbData -ConnectionString $cs -SqlStatement "select 1"             
-}
-catch {
-    $ACEnotWorking = $true
+$IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -like "Microsoft.ACE.OLEDB*")
+if($IsMissingACE){
+    Write-Host "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped. Please see https://www.microsoft.com/en-us/download/details.aspx?id=54920"
 }
 Describe "Read-OleDbData" -Tag "Read-OleDbData" {
-    $PSDefaultParameterValues = @{ 'It:Skip' = $ACEnotWorking }
+    $PSDefaultParameterValues = @{ 'It:Skip' = $IsMissingACE }
     Context "Basic Tests" {
         It "should be able to open spreadsheet" {
             $null = Read-OleDbData -ConnectionString $cs -SqlStatement "select 1"

From 5ec299ff2d5b546c77407a81cb5f4c3d613dcb3d Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 15:43:36 -0500
Subject: [PATCH 11/24] some debugging messages for troubleshooting CI

---
 __tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
index db8ff73..70e9a8b 100644
--- a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
@@ -7,6 +7,10 @@ $IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetEle
 if($IsMissingACE){
     Write-Host "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped. Please see https://www.microsoft.com/en-us/download/details.aspx?id=54920"
 }
+write-host "`$tfp = '$tfp'"
+write-host "`Test-Path $tfp = '$(Test-Path $tfp)'"
+write-host "`$cs = '$cs'"
+write-host "`$IsMissingACE = '$IsMissingACE'"
 Describe "Read-OleDbData" -Tag "Read-OleDbData" {
     $PSDefaultParameterValues = @{ 'It:Skip' = $IsMissingACE }
     Context "Basic Tests" {

From 1e522c562b89a9a16be974718d2c2042a14bfb5c Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 16:00:06 -0500
Subject: [PATCH 12/24] test skips for linux/macos, switch to warnings

---
 .../Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1 |  8 ++++++--
 .../Read-OleDbDataTests/Read-OleDbData.Tests.ps1    | 13 +++++++------
 2 files changed, 13 insertions(+), 8 deletions(-)

diff --git a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1 b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
index 4f428e2..314f154 100644
--- a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
@@ -4,10 +4,14 @@ Import-Module $scriptPath\..\..\ImportExcel.psd1 -Force
 $tfp = "$scriptPath\Read-OleDbData.xlsx"
 $IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -like "Microsoft.ACE.OLEDB*")
 if($IsMissingACE){
-    Write-Host "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped. Please see https://www.microsoft.com/en-us/download/details.aspx?id=54920"
+    Write-Warning "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped. Please see https://www.microsoft.com/en-us/download/details.aspx?id=54920"
 }
+Write-Warning "`$tfp = '$tfp'"
+Write-Warning "`Test-Path $tfp = '$(Test-Path $tfp)'"
+Write-Warning "`$IsMissingACE = '$IsMissingACE'"
+$skip = $IsLinux -or $IsMacOS -or $IsMissingACE
 Describe "Invoke-ExcelQuery" -Tag "Invoke-ExcelQuery" {
-    $PSDefaultParameterValues = @{ 'It:Skip' = $IsMissingACE }
+    $PSDefaultParameterValues = @{ 'It:Skip' = $skip }
     Context "Sheet1`$A1" {
         It "Should return 1 result with a value of 1" {
             $Results = Invoke-ExcelQuery $tfp "select ROUND(F1) as [A1] from [sheet1`$A1:A1]"
diff --git a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
index 70e9a8b..3697b35 100644
--- a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
@@ -5,14 +5,15 @@ $tfp = "$scriptPath\Read-OleDbData.xlsx"
 $cs = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$tfp;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
 $IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -like "Microsoft.ACE.OLEDB*")
 if($IsMissingACE){
-    Write-Host "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped. Please see https://www.microsoft.com/en-us/download/details.aspx?id=54920"
+    Write-Warning "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped. Please see https://www.microsoft.com/en-us/download/details.aspx?id=54920"
 }
-write-host "`$tfp = '$tfp'"
-write-host "`Test-Path $tfp = '$(Test-Path $tfp)'"
-write-host "`$cs = '$cs'"
-write-host "`$IsMissingACE = '$IsMissingACE'"
+Write-Warning "`$tfp = '$tfp'"
+Write-Warning "`Test-Path $tfp = '$(Test-Path $tfp)'"
+Write-Warning "`$cs = '$cs'"
+Write-Warning "`$IsMissingACE = '$IsMissingACE'"
+$skip = $IsLinux -or $IsMacOS -or $IsMissingACE
 Describe "Read-OleDbData" -Tag "Read-OleDbData" {
-    $PSDefaultParameterValues = @{ 'It:Skip' = $IsMissingACE }
+    $PSDefaultParameterValues = @{ 'It:Skip' = $skip }
     Context "Basic Tests" {
         It "should be able to open spreadsheet" {
             $null = Read-OleDbData -ConnectionString $cs -SqlStatement "select 1"

From a3dced10e4ecc15bc3ff5b19e820927668380b13 Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 16:37:00 -0500
Subject: [PATCH 13/24] update tests

---
 .../Invoke-ExcelQuery.Tests.ps1               | 29 ++++++++++++++++---
 1 file changed, 25 insertions(+), 4 deletions(-)

diff --git a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1 b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
index 314f154..8c62e95 100644
--- a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
@@ -2,16 +2,37 @@
 $scriptPath = $PSScriptRoot
 Import-Module $scriptPath\..\..\ImportExcel.psd1 -Force
 $tfp = "$scriptPath\Read-OleDbData.xlsx"
-$IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -like "Microsoft.ACE.OLEDB*")
-if($IsMissingACE){
-    Write-Warning "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped. Please see https://www.microsoft.com/en-us/download/details.aspx?id=54920"
+$skip = $IsLinux -or $IsMacOS #init default, not supported on mac or linux
+try {
+    $IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -like "Microsoft.ACE.OLEDB*")    
+    if ($IsMissingACE) {
+        Write-Warning "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped."
+    }
+    $skip = $skip -or $IsMissingACE
 }
+catch {
+    Write-Warning "Unable to get sources from System.Data.OleDb. Tests will be skipped."
+    $skip = $true #this will fail if the call to get the sources fails, usually means System.Data.OleDb isn't installed/supported
+}
+
+
 Write-Warning "`$tfp = '$tfp'"
 Write-Warning "`Test-Path $tfp = '$(Test-Path $tfp)'"
 Write-Warning "`$IsMissingACE = '$IsMissingACE'"
-$skip = $IsLinux -or $IsMacOS -or $IsMissingACE
+
 Describe "Invoke-ExcelQuery" -Tag "Invoke-ExcelQuery" {
     $PSDefaultParameterValues = @{ 'It:Skip' = $skip }
+    Context "Basic Checks" {
+        It "Should have a valid Test file" {
+            Test-Path $tfp | Should -Be $true
+        }
+        It "Should have the Read-OleDbData command loaded" {
+            (Get-Command Read-OleDbData) -ne $null | Should -Be $true
+        }
+        It "Should have the Invoke-ExcelQuery command loaded" {
+            (Get-Command Invoke-ExcelQuery) -ne $null | Should -Be $true
+        }
+    }
     Context "Sheet1`$A1" {
         It "Should return 1 result with a value of 1" {
             $Results = Invoke-ExcelQuery $tfp "select ROUND(F1) as [A1] from [sheet1`$A1:A1]"

From ebe49f1650d8b18cc16a41adb2d28815849c8e27 Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 16:56:23 -0500
Subject: [PATCH 14/24] fix or/and

---
 __tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1 | 2 +-
 __tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1    | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1 b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
index 8c62e95..6bf6951 100644
--- a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
@@ -8,7 +8,7 @@ try {
     if ($IsMissingACE) {
         Write-Warning "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped."
     }
-    $skip = $skip -or $IsMissingACE
+    $skip = $skip -and $IsMissingACE
 }
 catch {
     Write-Warning "Unable to get sources from System.Data.OleDb. Tests will be skipped."
diff --git a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
index 3697b35..6611a4d 100644
--- a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
@@ -11,7 +11,7 @@ Write-Warning "`$tfp = '$tfp'"
 Write-Warning "`Test-Path $tfp = '$(Test-Path $tfp)'"
 Write-Warning "`$cs = '$cs'"
 Write-Warning "`$IsMissingACE = '$IsMissingACE'"
-$skip = $IsLinux -or $IsMacOS -or $IsMissingACE
+$skip = ($IsLinux -or $IsMacOS) -and $IsMissingACE
 Describe "Read-OleDbData" -Tag "Read-OleDbData" {
     $PSDefaultParameterValues = @{ 'It:Skip' = $skip }
     Context "Basic Tests" {

From d41fbf8a05da0a03c049ead66a9e801fa9afb3bb Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 17:13:40 -0500
Subject: [PATCH 15/24] updating tests

---
 .../Invoke-ExcelQuery.Tests.ps1               | 78 +++++++++----------
 .../Read-OleDbData.Tests.ps1                  | 42 ++++++----
 2 files changed, 67 insertions(+), 53 deletions(-)

diff --git a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1 b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
index 6bf6951..2b02de9 100644
--- a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
@@ -1,42 +1,42 @@
-#Requires -Modules Pester
-$scriptPath = $PSScriptRoot
-Import-Module $scriptPath\..\..\ImportExcel.psd1 -Force
-$tfp = "$scriptPath\Read-OleDbData.xlsx"
-$skip = $IsLinux -or $IsMacOS #init default, not supported on mac or linux
-try {
-    $IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -like "Microsoft.ACE.OLEDB*")    
-    if ($IsMissingACE) {
-        Write-Warning "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped."
-    }
-    $skip = $skip -and $IsMissingACE
-}
-catch {
-    Write-Warning "Unable to get sources from System.Data.OleDb. Tests will be skipped."
-    $skip = $true #this will fail if the call to get the sources fails, usually means System.Data.OleDb isn't installed/supported
-}
+# #Requires -Modules Pester
+# $scriptPath = $PSScriptRoot
+# Import-Module $scriptPath\..\..\ImportExcel.psd1 -Force
+# $tfp = "$scriptPath\Read-OleDbData.xlsx"
+# $skip = $IsLinux -or $IsMacOS #init default, not supported on mac or linux
+# try {
+#     $IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -like "Microsoft.ACE.OLEDB*")    
+#     if ($IsMissingACE) {
+#         Write-Warning "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped."
+#     }
+#     $skip = $skip -and $IsMissingACE
+# }
+# catch {
+#     Write-Warning "Unable to get sources from System.Data.OleDb. Tests will be skipped."
+#     $skip = $true #this will fail if the call to get the sources fails, usually means System.Data.OleDb isn't installed/supported
+# }
 
 
-Write-Warning "`$tfp = '$tfp'"
-Write-Warning "`Test-Path $tfp = '$(Test-Path $tfp)'"
-Write-Warning "`$IsMissingACE = '$IsMissingACE'"
+# Write-Warning "`$tfp = '$tfp'"
+# Write-Warning "`Test-Path $tfp = '$(Test-Path $tfp)'"
+# Write-Warning "`$IsMissingACE = '$IsMissingACE'"
 
-Describe "Invoke-ExcelQuery" -Tag "Invoke-ExcelQuery" {
-    $PSDefaultParameterValues = @{ 'It:Skip' = $skip }
-    Context "Basic Checks" {
-        It "Should have a valid Test file" {
-            Test-Path $tfp | Should -Be $true
-        }
-        It "Should have the Read-OleDbData command loaded" {
-            (Get-Command Read-OleDbData) -ne $null | Should -Be $true
-        }
-        It "Should have the Invoke-ExcelQuery command loaded" {
-            (Get-Command Invoke-ExcelQuery) -ne $null | Should -Be $true
-        }
-    }
-    Context "Sheet1`$A1" {
-        It "Should return 1 result with a value of 1" {
-            $Results = Invoke-ExcelQuery $tfp "select ROUND(F1) as [A1] from [sheet1`$A1:A1]"
-            @($Results).length + $Results.A1 | Should -Be 2
-        }
-    }
-}
+# Describe "Invoke-ExcelQuery" -Tag "Invoke-ExcelQuery" {
+#     $PSDefaultParameterValues = @{ 'It:Skip' = $skip }
+#     Context "Basic Checks" {
+#         It "Should have a valid Test file" {
+#             Test-Path $tfp | Should -Be $true
+#         }
+#         It "Should have the Read-OleDbData command loaded" {
+#             (Get-Command Read-OleDbData) -ne $null | Should -Be $true
+#         }
+#         It "Should have the Invoke-ExcelQuery command loaded" {
+#             (Get-Command Invoke-ExcelQuery) -ne $null | Should -Be $true
+#         }
+#     }
+#     Context "Sheet1`$A1" {
+#         It "Should return 1 result with a value of 1" {
+#             $Results = Invoke-ExcelQuery $tfp "select ROUND(F1) as [A1] from [sheet1`$A1:A1]"
+#             @($Results).length + $Results.A1 | Should -Be 2
+#         }
+#     }
+# }
diff --git a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
index 6611a4d..9302801 100644
--- a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
@@ -1,25 +1,39 @@
 #Requires -Modules Pester
-$scriptPath = $PSScriptRoot
-Import-Module $scriptPath\..\..\ImportExcel.psd1 -Force
-$tfp = "$scriptPath\Read-OleDbData.xlsx"
-$cs = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$tfp;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
-$IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -like "Microsoft.ACE.OLEDB*")
-if($IsMissingACE){
-    Write-Warning "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped. Please see https://www.microsoft.com/en-us/download/details.aspx?id=54920"
+Import-Module $PSScriptRoot\..\..\ImportExcel.psd1 -Force
+$skip = $true
+try {
+    $IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -like "Microsoft.ACE.OLEDB*")    
+    if ($IsMissingACE) {
+        Write-Warning "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped."
+    }
+    $skip = $IsMissingACE
 }
-Write-Warning "`$tfp = '$tfp'"
-Write-Warning "`Test-Path $tfp = '$(Test-Path $tfp)'"
-Write-Warning "`$cs = '$cs'"
-Write-Warning "`$IsMissingACE = '$IsMissingACE'"
-$skip = ($IsLinux -or $IsMacOS) -and $IsMissingACE
+catch {
+    Write-Warning "Unable to get sources from System.Data.OleDb. Tests will be skipped."
+}
+
 Describe "Read-OleDbData" -Tag "Read-OleDbData" {
     $PSDefaultParameterValues = @{ 'It:Skip' = $skip }
+    BeforeAll{
+        $scriptPath = $PSScriptRoot
+        $tfp = "$scriptPath\Read-OleDbData.xlsx"
+        $cs = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$tfp;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
+        Write-Warning "`$tfp = '$tfp'"
+        Write-Warning "`Test-Path $tfp = '$(Test-Path $tfp)'"
+        Write-Warning "`$cs = '$cs'"
+    }
     Context "Basic Tests" {
-        It "should be able to open spreadsheet" {
+        It "Should have a valid Test file" {
+            Test-Path $tfp | Should -Be $true
+        }
+        It "Should have the Read-OleDbData command loaded" {
+            (Get-Command Read-OleDbData) -ne $null | Should -Be $true
+        }
+        It "Should be able to open spreadsheet" {
             $null = Read-OleDbData -ConnectionString $cs -SqlStatement "select 1"
             $true | Should -Be $true
         }
-        It "should return PSCustomObject for single result" {
+        It "Should return PSCustomObject for single result" {
             #multiple records will come back as Object[], but not going to test for that
             $Results = Read-OleDbData -ConnectionString $cs -SqlStatement "select 1"
             $Results.GetType().Name | Should -Be 'PSCustomObject'

From 5f00f5cbbc40993f20835f08be6227748a9caf00 Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 17:40:33 -0500
Subject: [PATCH 16/24] update tests

---
 .../Read-OleDbData.Tests.ps1                  | 32 ++++++++++++-------
 1 file changed, 21 insertions(+), 11 deletions(-)

diff --git a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
index 9302801..8031ba8 100644
--- a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
@@ -1,33 +1,43 @@
 #Requires -Modules Pester
-Import-Module $PSScriptRoot\..\..\ImportExcel.psd1 -Force
-$skip = $true
+if (-not (Get-command Import-Excel -ErrorAction SilentlyContinue)) {
+    Import-Module $PSScriptRoot\..\ImportExcel.psd1
+}
+
+$skip = $false
+if ($IsLinux -or $IsMacOS) {
+    $skip = $true
+    Write-Warning "Read-OleDbData: Linux and MacOs are not supported. Skipping tests."
+}
 try {
-    $IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -like "Microsoft.ACE.OLEDB*")    
-    if ($IsMissingACE) {
-        Write-Warning "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped."
+    if ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -notcontains "Microsoft.ACE.OLEDB.12.0") {
+        $skip = $true
+        Write-Warning "Read-OleDbData: Microsoft.ACE.OLEDB.12.0 provider not found. Skipping tests."
     }
     $skip = $IsMissingACE
 }
 catch {
-    Write-Warning "Unable to get sources from System.Data.OleDb. Tests will be skipped."
+    $skip = $true
+    Write-Warning "Read-OleDbData: Calls to System.Data.OleDb failed. Skipping tests."
 }
 
 Describe "Read-OleDbData" -Tag "Read-OleDbData" {
     $PSDefaultParameterValues = @{ 'It:Skip' = $skip }
-    BeforeAll{
+    BeforeAll {
         $scriptPath = $PSScriptRoot
         $tfp = "$scriptPath\Read-OleDbData.xlsx"
         $cs = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$tfp;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
-        Write-Warning "`$tfp = '$tfp'"
-        Write-Warning "`Test-Path $tfp = '$(Test-Path $tfp)'"
-        Write-Warning "`$cs = '$cs'"
+        if (!$skip) {
+            Write-Warning "`$tfp = '$tfp'"
+            Write-Warning "`Test-Path $tfp = '$(Test-Path $tfp)'"
+            Write-Warning "`$cs = '$cs'"
+        }
     }
     Context "Basic Tests" {
         It "Should have a valid Test file" {
             Test-Path $tfp | Should -Be $true
         }
         It "Should have the Read-OleDbData command loaded" {
-            (Get-Command Read-OleDbData) -ne $null | Should -Be $true
+            (Get-Command Read-OleDbData -ErrorAction SilentlyContinue) -ne $null | Should -Be $true
         }
         It "Should be able to open spreadsheet" {
             $null = Read-OleDbData -ConnectionString $cs -SqlStatement "select 1"

From f0b4cb28f612855335cacb63c877fc4e8076c525 Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 17:44:03 -0500
Subject: [PATCH 17/24] update tests to skip other pre-checks if linux/mac

---
 .../Read-OleDbData.Tests.ps1                  | 21 ++++++++++---------
 1 file changed, 11 insertions(+), 10 deletions(-)

diff --git a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
index 8031ba8..8b7f07d 100644
--- a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
@@ -7,17 +7,18 @@ $skip = $false
 if ($IsLinux -or $IsMacOS) {
     $skip = $true
     Write-Warning "Read-OleDbData: Linux and MacOs are not supported. Skipping tests."
-}
-try {
-    if ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -notcontains "Microsoft.ACE.OLEDB.12.0") {
-        $skip = $true
-        Write-Warning "Read-OleDbData: Microsoft.ACE.OLEDB.12.0 provider not found. Skipping tests."
+}else{
+    try {
+        if ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -notcontains "Microsoft.ACE.OLEDB.12.0") {
+            $skip = $true
+            Write-Warning "Read-OleDbData: Microsoft.ACE.OLEDB.12.0 provider not found. Skipping tests."
+        }
+        $skip = $IsMissingACE
+    }
+    catch {
+        $skip = $true
+        Write-Warning "Read-OleDbData: Calls to System.Data.OleDb failed. Skipping tests."
     }
-    $skip = $IsMissingACE
-}
-catch {
-    $skip = $true
-    Write-Warning "Read-OleDbData: Calls to System.Data.OleDb failed. Skipping tests."
 }
 
 Describe "Read-OleDbData" -Tag "Read-OleDbData" {

From 9daefc2925b12e64a4b7b65eafca5d4dbf712555 Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 17:45:33 -0500
Subject: [PATCH 18/24] fix typo in skip logic

---
 __tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 | 1 -
 1 file changed, 1 deletion(-)

diff --git a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
index 8b7f07d..8ad8ff5 100644
--- a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
@@ -13,7 +13,6 @@ if ($IsLinux -or $IsMacOS) {
             $skip = $true
             Write-Warning "Read-OleDbData: Microsoft.ACE.OLEDB.12.0 provider not found. Skipping tests."
         }
-        $skip = $IsMissingACE
     }
     catch {
         $skip = $true

From 9ba20b8ec8245ce440c2de7ce884f223b2b11736 Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 17:57:44 -0500
Subject: [PATCH 19/24] add back invoke tests, sync skip methods

---
 .../Invoke-ExcelQuery.Tests.ps1               | 84 ++++++++++---------
 .../Read-OleDbData.Tests.ps1                  |  6 --
 2 files changed, 44 insertions(+), 46 deletions(-)

diff --git a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1 b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
index 2b02de9..89405d8 100644
--- a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
@@ -1,42 +1,46 @@
-# #Requires -Modules Pester
-# $scriptPath = $PSScriptRoot
-# Import-Module $scriptPath\..\..\ImportExcel.psd1 -Force
-# $tfp = "$scriptPath\Read-OleDbData.xlsx"
-# $skip = $IsLinux -or $IsMacOS #init default, not supported on mac or linux
-# try {
-#     $IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -like "Microsoft.ACE.OLEDB*")    
-#     if ($IsMissingACE) {
-#         Write-Warning "MICROSOFT.ACE.OLEDB is missing! Tests will be skipped."
-#     }
-#     $skip = $skip -and $IsMissingACE
-# }
-# catch {
-#     Write-Warning "Unable to get sources from System.Data.OleDb. Tests will be skipped."
-#     $skip = $true #this will fail if the call to get the sources fails, usually means System.Data.OleDb isn't installed/supported
-# }
+#Requires -Modules Pester
+if (-not (Get-command Import-Excel -ErrorAction SilentlyContinue)) {
+    Import-Module $PSScriptRoot\..\ImportExcel.psd1
+}
+
+$skip = $false
+if ($IsLinux -or $IsMacOS) {
+    $skip = $true
+    Write-Warning "Read-OleDbData: Linux and MacOs are not supported. Skipping tests."
+}else{
+    try {
+        if ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -notcontains "Microsoft.ACE.OLEDB.12.0") {
+            $skip = $true
+            Write-Warning "Read-OleDbData: Microsoft.ACE.OLEDB.12.0 provider not found. Skipping tests."
+        }
+    }
+    catch {
+        $skip = $true
+        Write-Warning "Read-OleDbData: Calls to System.Data.OleDb failed. Skipping tests."
+    }
+}
 
 
-# Write-Warning "`$tfp = '$tfp'"
-# Write-Warning "`Test-Path $tfp = '$(Test-Path $tfp)'"
-# Write-Warning "`$IsMissingACE = '$IsMissingACE'"
-
-# Describe "Invoke-ExcelQuery" -Tag "Invoke-ExcelQuery" {
-#     $PSDefaultParameterValues = @{ 'It:Skip' = $skip }
-#     Context "Basic Checks" {
-#         It "Should have a valid Test file" {
-#             Test-Path $tfp | Should -Be $true
-#         }
-#         It "Should have the Read-OleDbData command loaded" {
-#             (Get-Command Read-OleDbData) -ne $null | Should -Be $true
-#         }
-#         It "Should have the Invoke-ExcelQuery command loaded" {
-#             (Get-Command Invoke-ExcelQuery) -ne $null | Should -Be $true
-#         }
-#     }
-#     Context "Sheet1`$A1" {
-#         It "Should return 1 result with a value of 1" {
-#             $Results = Invoke-ExcelQuery $tfp "select ROUND(F1) as [A1] from [sheet1`$A1:A1]"
-#             @($Results).length + $Results.A1 | Should -Be 2
-#         }
-#     }
-# }
+Describe "Invoke-ExcelQuery" -Tag "Invoke-ExcelQuery" {
+    $PSDefaultParameterValues = @{ 'It:Skip' = $skip }
+    BeforeAll {
+        $tfp = "$PSScriptRoot\Read-OleDbData.xlsx"
+    }
+    Context "Basic Checks" {
+        It "Should have a valid Test file" {
+            Test-Path $tfp | Should -Be $true
+        }
+        It "Should have the Read-OleDbData command loaded" {
+            (Get-Command Read-OleDbData) -ne $null | Should -Be $true
+        }
+        It "Should have the Invoke-ExcelQuery command loaded" {
+            (Get-Command Invoke-ExcelQuery) -ne $null | Should -Be $true
+        }
+    }
+    Context "Sheet1`$A1" {
+        It "Should return 1 result with a value of 1" {
+            $Results = Invoke-ExcelQuery $tfp "select ROUND(F1) as [A1] from [sheet1`$A1:A1]"
+            @($Results).length + $Results.A1 | Should -Be 2
+        }
+    }
+}
diff --git a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1 b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
index 8ad8ff5..7a9ef5c 100644
--- a/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Read-OleDbData.Tests.ps1
@@ -19,18 +19,12 @@ if ($IsLinux -or $IsMacOS) {
         Write-Warning "Read-OleDbData: Calls to System.Data.OleDb failed. Skipping tests."
     }
 }
-
 Describe "Read-OleDbData" -Tag "Read-OleDbData" {
     $PSDefaultParameterValues = @{ 'It:Skip' = $skip }
     BeforeAll {
         $scriptPath = $PSScriptRoot
         $tfp = "$scriptPath\Read-OleDbData.xlsx"
         $cs = "Provider=Microsoft.ACE.OLEDB.12.0;Data Source=$tfp;Extended Properties='Excel 12.0 Xml;HDR=NO;IMEX=1;'"
-        if (!$skip) {
-            Write-Warning "`$tfp = '$tfp'"
-            Write-Warning "`Test-Path $tfp = '$(Test-Path $tfp)'"
-            Write-Warning "`$cs = '$cs'"
-        }
     }
     Context "Basic Tests" {
         It "Should have a valid Test file" {

From ae31e9d39bd5ae6545f19c0e6c7b65d9fc64723a Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 18:08:45 -0500
Subject: [PATCH 20/24] update skip/help msgs in invoke cmd

---
 Public/Invoke-ExcelQuery.ps1 | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/Public/Invoke-ExcelQuery.ps1 b/Public/Invoke-ExcelQuery.ps1
index 3b8817e..4cb01a9 100644
--- a/Public/Invoke-ExcelQuery.ps1
+++ b/Public/Invoke-ExcelQuery.ps1
@@ -45,9 +45,14 @@ function Invoke-ExcelQuery {
         [String] $Query # var name consistent with Invoke-Sqlcmd
     )
 
-    $IsMissingACE = $null -eq ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -like "Microsoft.ACE.OLEDB*")
-    if($IsMissingACE){
-        Write-Error "MICROSOFT.ACE.OLEDB is missing! Please see https://www.microsoft.com/en-us/download/details.aspx?id=54920"
+    try {
+        if ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -notcontains "Microsoft.ACE.OLEDB.12.0") {
+            Write-Error "Microsoft.ACE.OLEDB.12.0 provider is missing! Please install from https://www.microsoft.com/en-us/download/details.aspx?id=54920"
+            return
+        }
+    }
+    catch {
+        Write-Error "System.Data.OleDb is not working or you are on an unsupported platform."
         return
     }
 

From 487cf51abc372a4223f3a0656eb5d86c82c515dc Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 18:19:23 -0500
Subject: [PATCH 21/24] fix test skip label name

---
 __tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1 | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1 b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
index 89405d8..46e964d 100644
--- a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
@@ -6,17 +6,17 @@ if (-not (Get-command Import-Excel -ErrorAction SilentlyContinue)) {
 $skip = $false
 if ($IsLinux -or $IsMacOS) {
     $skip = $true
-    Write-Warning "Read-OleDbData: Linux and MacOs are not supported. Skipping tests."
+    Write-Warning "Invoke-ExcelQuery: Linux and MacOs are not supported. Skipping tests."
 }else{
     try {
         if ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -notcontains "Microsoft.ACE.OLEDB.12.0") {
             $skip = $true
-            Write-Warning "Read-OleDbData: Microsoft.ACE.OLEDB.12.0 provider not found. Skipping tests."
+            Write-Warning "Invoke-ExcelQuery: Microsoft.ACE.OLEDB.12.0 provider not found. Skipping tests."
         }
     }
     catch {
         $skip = $true
-        Write-Warning "Read-OleDbData: Calls to System.Data.OleDb failed. Skipping tests."
+        Write-Warning "Invoke-ExcelQuery: Calls to System.Data.OleDb failed. Skipping tests."
     }
 }
 

From 7177623104c63aaf6bdf2bda0416b97e746d5b2c Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Mon, 8 Nov 2021 18:21:11 -0500
Subject: [PATCH 22/24] add silent continue to cmd check tests

---
 __tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1 | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1 b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
index 46e964d..e79f96d 100644
--- a/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
+++ b/__tests__/Read-OleDbDataTests/Invoke-ExcelQuery.Tests.ps1
@@ -31,10 +31,10 @@ Describe "Invoke-ExcelQuery" -Tag "Invoke-ExcelQuery" {
             Test-Path $tfp | Should -Be $true
         }
         It "Should have the Read-OleDbData command loaded" {
-            (Get-Command Read-OleDbData) -ne $null | Should -Be $true
+            (Get-Command Read-OleDbData -ErrorAction SilentlyContinue) -ne $null | Should -Be $true
         }
         It "Should have the Invoke-ExcelQuery command loaded" {
-            (Get-Command Invoke-ExcelQuery) -ne $null | Should -Be $true
+            (Get-Command Invoke-ExcelQuery -ErrorAction SilentlyContinue) -ne $null | Should -Be $true
         }
     }
     Context "Sheet1`$A1" {

From 4fa6fbda012feb2d830ae19a9ed4c674babe8c03 Mon Sep 17 00:00:00 2001
From: Roy Ashbrook <rashbrook@patriottrans.com>
Date: Tue, 9 Nov 2021 09:05:15 -0500
Subject: [PATCH 23/24] update warning msg for Read-OleDbData

---
 Public/Read-OleDbData.ps1 | 27 +++++++--------------------
 1 file changed, 7 insertions(+), 20 deletions(-)

diff --git a/Public/Read-OleDbData.ps1 b/Public/Read-OleDbData.ps1
index ccd0fdc..491b54c 100644
--- a/Public/Read-OleDbData.ps1
+++ b/Public/Read-OleDbData.ps1
@@ -36,28 +36,15 @@ function Read-OleDbData {
         [String] $SqlStatement   
     )
     
-    if ($IsLinux -or $IsMacOS) {
-        #todo: possibly add support for linux/mac somehow. i haven't researched this at all as i 
-        #      don't have a need for that and i'm not sure anyone else would in this context, but it does
-        #      appear that once upon a time mono had support for oledb, so maybe it is (or was) supported.
-        #      mono link here: https://www.mono-project.com/archived/ole_db/ 
-        Write-Error "Read-OleDbData only runs on Windows"
+    try {
+        if ((New-Object system.data.oledb.oledbenumerator).GetElements().SOURCES_NAME -notcontains "Microsoft.ACE.OLEDB.12.0") {
+            Write-Warning "Microsoft.ACE.OLEDB.12.0 provider is missing! You will not be able to query Excel files without it. Please install from https://www.microsoft.com/en-us/download/details.aspx?id=54920"
+        }
+    }
+    catch {
+        Write-Error "System.Data.OleDb is not working or you are on an unsupported platform."
         return
     }
-
-    #todo: add checks for dotnet libs
-    #todo: possibly add checks for ace drivers, but maybe only needed if we want to restrict usage.
-    #      i currently just pass through the query and connection string so user is only limited by
-    #      their own machine setup, but they have to check for those dependencies themselves.
-    #todo: possibly try/catch. i personally do not do this, as i let them throw naturally and catch
-    #      them/handle them outside of this function.
-    #todo: possibly allow for DataSets instead of just DataTable. I used to use a similar method before
-    #      switching to the sqlcmd module and use datasets as I had multiple tables coming back from
-    #      sql sometimes. i think in this case, it's best to just keep it simple, but maybe someone
-    #      out there would prefer to be able to get multiple tables back. i have not tested that
-    #      with the OleDbDataAdapter.
-    #todo: possibly just return the datatable, i do it as below because i prefer to simplify the output
-    #      and get rid of the extra fields that come back with the datatable or rows.
     
     $DataTable = new-object System.Data.DataTable
     $DataAdapter = new-object System.Data.OleDb.OleDbDataAdapter $SqlStatement, $ConnectionString

From 1ee9493bc005702f8f983e120d838b4e9df4cd9e Mon Sep 17 00:00:00 2001
From: dfinke <finked@hotmail.com>
Date: Fri, 12 Nov 2021 17:25:45 -0500
Subject: [PATCH 24/24] Minor tweak. Display query executed

---
 Examples/InvokeExcelQuery/Examples.ps1 | 17 +++++++++--------
 1 file changed, 9 insertions(+), 8 deletions(-)

diff --git a/Examples/InvokeExcelQuery/Examples.ps1 b/Examples/InvokeExcelQuery/Examples.ps1
index c565292..a42da76 100644
--- a/Examples/InvokeExcelQuery/Examples.ps1
+++ b/Examples/InvokeExcelQuery/Examples.ps1
@@ -1,13 +1,14 @@
-try {Import-Module $PSScriptRoot\..\..\ImportExcel.psd1} catch {throw ; return}
+try { Import-Module $PSScriptRoot\..\..\ImportExcel.psd1 } catch { throw ; return }
 
 $queries = 
-  'select * from [sheet1$A:A]',
-  'select * from [sheet1$]',
-  'select * from [sheet1$A2:E11]',
-  'select F2,F5 from [sheet1$A2:E11]',
-  'select * from [sheet1$A2:E11] where F2 = "Grocery"',
-  'select F2 as [Category], F5 as [Discount], F5*2 as [DiscountPlus] from [sheet1$A2:E11]'
+'select * from [sheet1$A:A]',
+'select * from [sheet1$]',
+'select * from [sheet1$A2:E11]',
+'select F2,F5 from [sheet1$A2:E11]',
+'select * from [sheet1$A2:E11] where F2 = "Grocery"',
+'select F2 as [Category], F5 as [Discount], F5*2 as [DiscountPlus] from [sheet1$A2:E11]'
 
 foreach ($query in $queries) {
-  Invoke-ExcelQuery .\testOleDb.xlsx $query| Format-Table  
+  "query: $($query)"
+  Invoke-ExcelQuery .\testOleDb.xlsx $query | Format-Table  
 }