Implementation complete with tests passing

Author: adityapatwardhan

src/dsc/psresourceget.ps1

@@ -467,35 +467,6 @@ function ExportOperation {
     }
 }
 
-filter Where-PSResource {
-    param(
-        [string]$name,
-        [string]$version,
-        [Scope]$scope,
-        [string]$repositoryName
-    )
-
-    process {
-        $nameMatch = if ($name) { $_.Name -eq $name } else { $true }
-        $versionMatch = if ($_.Version) {
-            try {
-                SatisfiesVersion -version $_.Version -versionRange $version
-            }
-            catch {
-                $_.Version.ToString() -eq $version
-            }
-        }
-        else { $true }
-        $scopeMatch = if ($_.Scope) { $_.Scope -eq $scope } else { $true }
-        $repositoryMatch = if ($_.Repository) { $_.Repository -eq $repositoryName } else { $true }
-
-        if ($nameMatch -and $versionMatch -and $scopeMatch -and $repositoryMatch) {
-            Write-Trace -message "Resource matches filter criteria: Name=$($_.Name), Version=$($_.Version), Scope=$($_.Scope), Repository=$($_.Repository)" -level trace
-            $_
-        }
-    }
-}
-
 function SetPSResourceList {
     param(
         $inputObj
@@ -516,7 +487,7 @@ function SetPSResourceList {
         $scope = if ($resourceDesiredState.scope) { $resourceDesiredState.scope } else { "CurrentUser" }
 
         # Resource should not exist - uninstall if it does
-        $currentState.resources | Where-PSResource -name $name -version $version -scope $scope -repositoryName $repositoryName | ForEach-Object {
+        $currentState.resources | ForEach-Object {
 
             $isInDesiredState = $_.IsInDesiredState($resourceDesiredState)
 
@@ -564,7 +535,8 @@ function SetPSResourceList {
 
         Write-Trace -message "Installing resources: $($resourcesToInstall.Values | ForEach-Object { " $($_.Name) -- $($_.Version) " })"
         $resourcesToInstall.Values | ForEach-Object {
-            Install-PSResource -Name $_.Name -Version $_.Version -Scope $scope -Repository $repositoryName -ErrorAction Stop -TrustRepository:$inputObj.trustedRepository
+            $usePrerelease = if ($_.preRelease) { $true } else { $false }
+            Install-PSResource -Name $_.Name -Version $_.Version -Scope $scope -Repository $repositoryName -ErrorAction Stop -TrustRepository:$inputObj.trustedRepository -Prerelease:$usePrerelease -Reinstall
         }
 
         $resourcesChanged = $true

test/DscResource/PSResourceGetDSCResource.Tests.ps1

@@ -387,4 +387,25 @@ Describe 'E2E tests for PSResourceList resource' -Tags 'CI' {
             $version | Should -BeIn @("101.0.99.beta1", "0.0.93")
         }
     }
+
+    It 'Can install modules with one existing other not' {
+        $mod = Get-PSResource -Name 'testmodule99' -ErrorAction SilentlyContinue
+        if ($mod) {
+           $mod | Uninstall-PSResource -ErrorAction SilentlyContinue
+        }
+
+        Install-PSResource -Name 'testmodule99' -ErrorAction SilentlyContinue -Repository PSGallery -Reinstall -TrustRepository -Version '0.0.93'
+
+        $configPath = Join-Path -Path $PSScriptRoot -ChildPath 'configs/psresourcegetlist.oneexisting.install.dsc.yaml'
+        & $script:dscExe config set -f $configPath
+
+        $psresource = Get-PSResource -Name 'testmodule99' -ErrorAction SilentlyContinue
+        $psresource | Should -HaveCount 2
+
+        $psresource | ForEach-Object {
+            $version = if ($_.prerelease) { "$($_.Version)" + '.' + "$($_.PreRelease)" } else { $_.Version.ToString() }
+
+            $version | Should -BeIn @("101.0.99.beta1", "0.0.93")
+        }
+    }
 }

test/DscResource/configs/psresourcegetlist.prerelease.install.dsc.yaml

@@ -9,8 +9,9 @@ resources:
     - name: testmodule99
       version: '[[0.0.93,)'
       _exist: true
-      preRelease: true
+      preRelease: false
     - name: testmodule99
       version: '[[100.0.99,)'
       _exist: true
       preRelease: true
+