Add support for piping into Install-Module

chaliy · Mar 15, 2012 · 69cea26 · 69cea26
1 parent 442095e
commit 69cea26
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 7 deletions.
diff --git a/PsGet/PsGet.psm1 b/PsGet/PsGet.psm1
@@ -13,15 +13,15 @@ function Install-Module {
 Param(
     [Parameter(ValueFromPipeline=$true, ValueFromPipelineByPropertyName=$true, Mandatory=$true, Position=0, ParameterSetName="Repo")]    
     [String]$Module,
-    [Parameter(ValueFromPipeline=$true, ValueFromPipelineByPropertyName=$true, Mandatory=$true, ParameterSetName="Web")]
+    [Parameter(ValueFromPipelineByPropertyName=$true, Mandatory=$true, ParameterSetName="Web")]
     [String]$ModuleUrl,    
-    [Parameter(ValueFromPipeline=$true, ValueFromPipelineByPropertyName=$true, Mandatory=$true, ParameterSetName="Local")]
+    [Parameter(ValueFromPipelineByPropertyName=$true, Mandatory=$true, ParameterSetName="Local")]
     $ModulePath,        
-    [Parameter(ValueFromPipeline=$true, ValueFromPipelineByPropertyName=$true, Mandatory=$false, ParameterSetName="Web")]
-    [Parameter(ValueFromPipeline=$true, ValueFromPipelineByPropertyName=$true, Mandatory=$false, ParameterSetName="Local")]
+    [Parameter(ValueFromPipelineByPropertyName=$true, Mandatory=$false, ParameterSetName="Web")]
+    [Parameter(ValueFromPipelineByPropertyName=$true, Mandatory=$false, ParameterSetName="Local")]
     [String]$ModuleName,
-    [Parameter(ValueFromPipeline=$true, ValueFromPipelineByPropertyName=$true, Mandatory=$false, ParameterSetName="Web")]
-    [Parameter(ValueFromPipeline=$true, ValueFromPipelineByPropertyName=$true, Mandatory=$false, ParameterSetName="Local")]
+    [Parameter(ValueFromPipelineByPropertyName=$true, Mandatory=$false, ParameterSetName="Web")]
+    [Parameter(ValueFromPipelineByPropertyName=$true, Mandatory=$false, ParameterSetName="Local")]
     [String]$Type,
 
     [Parameter(ValueFromPipelineByPropertyName=$true)]
@@ -37,10 +37,15 @@ Param(
     [String]$DirectoryUrl = "https://github.com/chaliy/psget/raw/master/Directory.xml"
 )
 
+begin {
+
     if($PSVersionTable.PSVersion.Major -lt 2) {
         Write-Error "PsGet requires PowerShell 2.0 or better; you have version $($Host.Version)."    
         return
     }
+}
+
+process {
 
     switch($PSCmdlet.ParameterSetName) {
         "Repo"   {            
@@ -125,7 +130,7 @@ Param(
 
         }
         default {
-            throw "Unknown ParameterSetName '$_'"
+            throw "Unknown ParameterSetName '$($PSCmdlet.ParameterSetName)'"
         }
     }
 
@@ -158,6 +163,7 @@ Param(
     }
 
     InstallModuleFromLocalFolder -SourceFolderPath:$TempModuleFolderPath -ModuleName:$ModuleName -Destination $Destination -DoNotImport:$DoNotImport -Startup:$Startup -Force:$Force 
+}
 
 <#
 .Synopsis

diff --git a/Tests.ps1 b/Tests.ps1
@@ -126,6 +126,11 @@ $retrieved = Get-PsGetModuleInfo -ModuleName HelloWorld -DirectoryUrl:"file://$h
 Assert-Equals $retrieved.ModuleName HelloWorld
 Assert-Equals $retrieved.ModuleUrl https://github.com/chaliy/psget/raw/master/TestModules/HelloWorld.psm1
 
+write-host Should support piping from Get-PsGetModuleInfo to Install-Module
+Get-PsGetModuleInfo -ModuleName HelloWorld -DirectoryUrl:"file://$here\TestModules\Directory.xml" -Verbose | Install-Module -Verbose
+assert-moduleinstalled "HelloWorld"
+drop-module "HelloWorld"
+
 write-host Should support alternate install destination
 install-module -ModuleUrl https://github.com/chaliy/psget/raw/master/TestModules/HelloWorld.psm1 -Destination $Env:TEMP\Modules -Verbose
 if (-not (Test-Path -Path $Env:TEMP\Modules\HelloWorld\HelloWorld.psm1)) {