Add more information on streams for plugins (+ adapt GitReleases) & a…

…dd more pester tests

AU/Plugins/GitReleases.ps1

@@ -21,9 +21,6 @@ param(
     # The text that should be used in the description of the release.
     [string]$releaseDescription = $null,
 
-    # The text that should be used in the description when a release is created for a stream (by default it uses the latest commit message).
-    [string]$streamReleaseDescription = '',
-
     # The formatting to use when replacing <date> in release header/description and on date based releases.
     [string]$dateFormat = '{0:yyyy-MM-dd}',
 
@@ -70,14 +67,12 @@ $packagesToRelease = New-Object 'System.Collections.Generic.List[hashtable]'
 
 $packages | % {
     if ($_.Streams) {
-        $pkg = $_
-        $data = ConvertFrom-Json ($pkg.Streams -replace '@', '' -replace '\s*=\s*', '":"' -replace '{\s*', '{"' -replace '\s*}', '"}' -replace '\s*;\s*', '","')
-        ($data | Get-Member -MemberType NoteProperty).Name | % {
-            $value = $data.$_
+        $_.StreamsDetails.Values | ? { $_.Updated } | % {
             $packagesToRelease.Add(@{
-                    Name          = $pkg.Name
-                    RemoteVersion = $value
-                    NuFile        = Resolve-Path ("$($pkg.Path)/*.$($value).nupkg")
+                    Name          = $_.Name
+                    NuspecVersion = $_.NuspecVersion
+                    RemoteVersion = $_.RemoteVersion
+                    NuFile        = Resolve-Path ("$($_.Path)/$($_.Name).$($_.RemoteVersion).nupkg")
                 })
         }
     }
@@ -143,13 +138,7 @@ $packagesToRelease | % {
 
     if ($releaseType -eq 'package') {
         $releaseName = $releaseHeader -replace '<PackageName>', $_.Name -replace '<RemoteVersion>', $_.RemoteVersion -replace '<NuspecVersion>', $_.NuspecVersion -replace '<date>', $date
-        if ($_.NuspecVersion) {
-            $packageDesc = $releaseDescription
-        }
-        else {
-            $packageDesc = $streamReleaseDescription
-        }
-        $packageDesc = $packageDesc -replace '<PackageName>', $_.Name -replace '<RemoteVersion>', $_.RemoteVersion -replace '<NuspecVersion>', $_.NuspecVersion -replace '<date>', $date
+        $packageDesc = $releaseDescription -replace '<PackageName>', $_.Name -replace '<RemoteVersion>', $_.RemoteVersion -replace '<NuspecVersion>', $_.NuspecVersion -replace '<date>', $date
 
         $release = GetOrCreateRelease `
             -tagName "$($_.Name)-$($_.RemoteVersion)" `

AU/Private/AUPackage.ps1

@@ -14,6 +14,7 @@ class AUPackage {
 
     [string]         $StreamsPath
     [pscustomobject] $Streams
+    [hashtable]      $StreamsDetails
 
     AUPackage([string] $Path ){
         if ([String]::IsNullOrWhiteSpace( $Path )) { throw 'Package path can not be empty' }
@@ -31,6 +32,16 @@ class AUPackage {
         $this.Streams     = [AUPackage]::LoadStreams( $this.StreamsPath )
     }
 
+    [PSCustomObject] GetStreamDetails() {
+        return [PSCustomObject] @{
+            Path          = $this.Path
+            Name          = $this.Name
+            Updated       = $this.Updated
+            RemoteVersion = $this.RemoteVersion
+            NuspecVersion = $this.NuspecVersion
+        }
+    }
+
     static [xml] LoadNuspecFile( $NuspecPath ) {
         $nu = New-Object xml
         $nu.PSBase.PreserveWhitespace = $true
@@ -77,4 +88,19 @@ class AUPackage {
         return "$d\_output"
     }
 
+    AUPackage( [hashtable] $obj ) {
+        if (!$obj) { throw 'Obj can not be empty' }
+        $obj.Keys | % {
+            $this.$_ = $obj[$_]
+        }
+    }
+
+    [hashtable] Serialize() {
+        $res = @{}
+        $this | Get-Member -Type Properties | % {
+            $property = $_.Name
+            $res.Add($property, $this.$property)
+        }
+        return $res
+    }
 }

AU/Public/Update-AUPackages.ps1

@@ -129,6 +129,8 @@ function Update-AUPackages {
                     } else {
                         $pkg.Error = 'Job returned no object, Vector smash ?'
                     }
+                } else {
+                    $pkg = [AUPackage]::new($pkg)
                 }
 
 
@@ -228,7 +230,7 @@ function Update-AUPackages {
                 . $s $using:package_name $Options
             }
 
-            $pkg
+            $pkg.Serialize()
         } | Out-Null
     }
     $result = $result | sort Name

AU/Public/Update-Package.ps1

@@ -200,6 +200,8 @@ function Update-Package {
     }
 
     function process_stream() {
+        $package.Updated = $false
+
         if (!(is_version $package.NuspecVersion)) {
             Write-Warning "Invalid nuspec file Version '$($package.NuspecVersion)' - using 0.0"
             $global:Latest.NuspecVersion = $package.NuspecVersion = '0.0'
@@ -433,6 +435,7 @@ function Update-Package {
         $res.Keys | ? { $_ -ne 'Streams' } | % { $global:au_Latest.Remove($_) }
         $global:au_Latest += $res
 
+        $package.StreamsDetails = @{}
         $streams | % {
             $stream = $res.Streams[$_]
 
@@ -450,7 +453,10 @@ function Update-Package {
 
             set_latest $stream $package.Streams.$_ $_
             process_stream
+
+            $package.StreamsDetails.Add($_, $package.GetStreamDetails())
         }
+        $package.StreamsDetails.Values | ? { $_.updated } | % { $package.Updated = $true }
     } else {
         '' | result
         set_latest $res $package.NuspecVersion

test.ps1

@@ -2,7 +2,8 @@ param(
     [switch]$Chocolatey,
 
     [switch]$Pester,
-    [string]$Tag
+    [string]$Tag,
+    [switch]$CodeCoverage
 )
 
 if (!$Chocolatey -and !$Pester) { $Chocolatey = $Pester = $true }
@@ -20,5 +21,10 @@ if ($Pester) {
     Write-Host "`n==| Running Pester tests"
 
     $testResultsFile = "$build_dir/TestResults.xml"
-    Invoke-Pester -Tag $Tag -OutputFormat NUnitXml -OutputFile $testResultsFile -PassThru
+    if ($CodeCoverage) {
+        $files = @(ls $PSScriptRoot/AU/* -Filter *.ps1 -Recurse | % FullName)
+        Invoke-Pester -Tag $Tag -OutputFormat NUnitXml -OutputFile $testResultsFile -PassThru -CodeCoverage $files
+    } else {
+        Invoke-Pester -Tag $Tag -OutputFormat NUnitXml -OutputFile $testResultsFile -PassThru
+    }
 }

tests/Update-AUPackages.Streams.Tests.ps1

@@ -33,65 +33,33 @@ Describe 'Update-AUPackages using streams' -Tag updateallstreams {
 
     Context 'Plugins' {
 
-        It 'should ignore the package that returns "ignore"' {
+        It 'should execute GitReleases plugin when there are updates' {
             gc $global:au_Root\test_package_with_streams_1\update.ps1 | set content
-            $content -replace 'update', "Write-Host 'test ignore'; 'ignore'" | set content
+            $content -replace '@\{.+1\.3.+\}', "@{ Version = '1.3.2' }" | set content
+            $content -replace '@\{.+1\.2.+\}', "@{ Version = '1.2.4' }" | set content
             $content | sc $global:au_Root\test_package_with_streams_1\update.ps1
-
-            $res = updateall -Options $Options -NoPlugins:$false 6>$null
-
-            $res[0].Ignored | Should Be $true
-            $res[0].IgnoreMessage | Should Be 'test ignore'
-        }
-
-        It 'should repeat and ignore on specific error' {
-            gc $global:au_Root\test_package_with_streams_1\update.ps1 | set content
-            $content -replace 'update', "1|Out-File -Append $TestDrive\tmp_test; throw 'test ignore'; update" | set content
-            $content | sc $global:au_Root\test_package_with_streams_1\update.ps1
-
-            $Options.RepeatOn = @('test ignore')
-            $Options.RepeatCount = 2
-            $Options.IgnoreOn = @('test ignore')
-
-            $res = updateall -Options $Options -NoPlugins:$false 6>$null
-
-            $res[0].Ignored | Should Be $true
-            $res[0].IgnoreMessage | Should BeLike 'AU ignored on*test ignore'
-
-            (gc $TestDrive\tmp_test).Count | Should be 3
-        }
-
-        It 'should execute Report plugin' {
-            $Options.Report = @{
-                Type = 'markdown'
-                Path = "$global:au_Root\report.md"
-                Params = @{ Github_UserRepo = 'majkinetor/chocolatey' }
-            }
-
-            updateall -NoPlugins:$false -Options $Options  6> $null
-
-            Test-Path $Options.Report.Path | Should Be $true
-
-            $report = gc $Options.Report.Path 
-            ($report -match "test_package_with_streams_[1-$pkg_no]").Count | Should Be (3 * $pkg_no)
-        }
-
-        It 'should execute RunInfo plugin' {
-            $Options.RunInfo = @{
-                Path    = "$global:au_Root\update_info.xml"
-                Exclude = 'password'
+
+            $Options.GitReleases = @{
+                ApiToken    = 'apiToken'
+                ReleaseType = 'package'
+                Force       = $true
             }
-            $Options.Test = @{
-                MyPassword = 'password'
-                Parameter2 = 'p2'`
-            }
-
-            $res = updateall -NoPlugins:$false -Options $Options  6> $null
 
-            Test-Path $Options.RunInfo.Path | Should Be $true
-            $info = Import-Clixml $Options.RunInfo.Path
-            $info.plugin_results.RunInfo -match 'Test.MyPassword' | Should Be $true
-            $info.Options.Test.MyPassword | Should Be '*****' 
+            Mock Invoke-RestMethod {
+                return @{
+                    tag_name = 'test_package_with_streams_1-1.2.4'
+                    assets = @(
+                        @{
+                            url = 'https://api.github.com/test_package_with_streams_1.1.2.4.nupkg'
+                            name = 'test_package_with_streams_1.1.2.4.nupkg'
+                        }
+                    )
+                }
+            } -ModuleName AU
+
+            updateall -NoPlugins:$false -Options $Options 6> $null
+
+            Assert-MockCalled Invoke-RestMethod -Exactly 6 -ModuleName AU
         }
     }
 
@@ -100,7 +68,7 @@ Describe 'Update-AUPackages using streams' -Tag updateallstreams {
         $choco_path = gcm choco.exe | % Source
         $choco_hash = Get-FileHash $choco_path -Algorithm SHA256 | % Hash
         gc $global:au_Root\test_package_with_streams_1\update.ps1 | set content
-        $content -replace '@\{.+\}', "@{ Version = '1.3'; ChecksumType32 = 'sha256'; Checksum32 = '$choco_hash'}" | set content
+        $content -replace '@\{.+1\.3.+\}', "@{ Version = '1.3.2'; ChecksumType32 = 'sha256'; Checksum32 = '$choco_hash'}" | set content
         $content -replace 'update', "update -ChecksumFor 32" | set content
         $content | sc $global:au_Root\test_package_with_streams_1\update.ps1
 

tests/Update-AUPackages.Tests.ps1

@@ -136,6 +136,68 @@ Describe 'Update-AUPackages' -Tag updateall {
             $info.plugin_results.RunInfo -match 'Test.MyPassword' | Should Be $true
             $info.Options.Test.MyPassword | Should Be '*****' 
         }
+
+        It 'should execute GitReleases plugin when there are no updates' {
+            $Options.GitReleases = @{
+                ApiToken    = 'apiToken'
+                ReleaseType = 'package'
+                Force       = $true
+            }
+
+            Mock Invoke-RestMethod {} -ModuleName AU
+
+            updateall -NoPlugins:$false -Options $Options 6> $null
+
+            Assert-MockCalled Invoke-RestMethod -Exactly 0 -ModuleName AU
+        }
+
+        It 'should execute GitReleases plugin per package when there are updates' {
+            gc $global:au_Root\test_package_1\update.ps1 | set content
+            $content -replace '@\{.+\}', "@{ Version = '1.3' }" | set content
+            $content | sc $global:au_Root\test_package_1\update.ps1
+
+            $Options.GitReleases = @{
+                ApiToken    = 'apiToken'
+                ReleaseType = 'package'
+                Force       = $true
+            }
+
+            Mock Invoke-RestMethod {
+                return @{
+                    tag_name = 'test_package_1-1.3'
+                    assets = @(
+                        @{
+                            url = 'https://api.github.com/test_package_1.1.3.nupkg'
+                            name = 'test_package_1.1.3.nupkg'
+                        }
+                    )
+                }
+            } -ModuleName AU
+
+            updateall -NoPlugins:$false -Options $Options 6> $null
+
+            Assert-MockCalled Invoke-RestMethod -Exactly 3 -ModuleName AU
+        }
+
+        It 'should execute GitReleases plugin per date when there are updates' {
+            gc $global:au_Root\test_package_1\update.ps1 | set content
+            $content -replace '@\{.+\}', "@{ Version = '1.3' }" | set content
+            $content | sc $global:au_Root\test_package_1\update.ps1
+
+            $Options.GitReleases = @{
+                ApiToken    = 'apiToken'
+                ReleaseType = 'date'
+                Force       = $true
+            }
+
+            Mock Get-Date { return '2017-11-05' } -ParameterFilter { $UFormat -eq '{0:yyyy-MM-dd}' } -ModuleName AU
+            Mock Invoke-RestMethod { return @{ tag_name = '2017-11-05' } } -ModuleName AU
+
+            updateall -NoPlugins:$false -Options $Options 6> $null
+
+            Assert-MockCalled Get-Date -Exactly 1 -ModuleName AU
+            Assert-MockCalled Invoke-RestMethod -Exactly 2 -ModuleName AU
+        }
     }
 
     It 'should update package with checksum verification mode' {

tests/test_package_with_streams/update.ps1

@@ -3,10 +3,10 @@ function global:au_SearchReplace() {
 }
 
 function global:au_GetLatest() {
-    @{ Streams = @{
+    @{ Streams = [ordered] @{
         '1.4' = @{ Version = '1.4-beta1' }
-        '1.2' = @{ Version = '1.2.3' }
         '1.3' = @{ Version = '1.3.1' }
+        '1.2' = @{ Version = '1.2.3' }
     } }
 }