🧪Fix regressions from breaking change

Author: JustinGrote

ModuleFast.psm1

@@ -785,7 +785,7 @@ class ModuleFastInfo: IComparable {
   }
 
   [string] ToString() {
-    return "$($this.Name)@$($this.ModuleVersion)"
+    return "$($this.Name)($($this.ModuleVersion))"
   }
   [string] ToUniqueString() {
     return "$($this.Name)-$($this.ModuleVersion)-$($this.Location)"
@@ -1452,7 +1452,7 @@ filter ConvertFrom-RequiredSpec {
         continue
       }
       if ($kv.Value -as [NuGetVersion]) {
-        [ModuleFastSpec]"$($kv.Name)@$($kv.Value)"
+        [ModuleFastSpec]"$($kv.Name)=$($kv.Value)"
         continue
       }
 
@@ -1490,6 +1490,7 @@ function Read-RequiredSpecFile ($RequiredSpecPath) {
   if ($uri.scheme -in 'http', 'https') {
     [string]$content = (Invoke-WebRequest -Uri $uri).Content
     if ($content.StartsWith('@{')) {
+      #HACK: Cannot read PowerShell Data Files from a string, the interface is private, so we write to a temp file as a workaround.
       $tempFile = [io.path]::GetTempFileName()
       $content > $tempFile
       return Import-PowerShellDataFile -Path $tempFile

ModuleFast.tests.ps1

@@ -9,7 +9,7 @@ Import-Module $PSScriptRoot/ModuleFast.psm1 -Force
 
 BeforeAll {
   if ($env:MFURI) {
-    $PSDefaultParameterValues['Get-ModuleFastPlan:Source'] = $env:MFURI
+    $PSDefaultParameterValues['*-ModuleFast*:Source'] = $env:MFURI
   }
 }
 
@@ -361,7 +361,7 @@ Describe 'Get-ModuleFastPlan' -Tag 'E2E' {
     $actual.ModuleVersion | Should -Be '0.0.2-prerelease'
   }
   It 'Detects Prerelease even if Prerelease not specified' {
-    $actual = Get-ModuleFastPlan 'PrereleaseTest@0.0.2-prerelease'
+    $actual = Get-ModuleFastPlan 'PrereleaseTest=0.0.2-prerelease'
     $actual.ModuleVersion | Should -Be '0.0.2-prerelease'
   }
 
@@ -409,7 +409,7 @@ Describe 'Install-ModuleFast' -Tag 'E2E' {
     Get-Item $installTempPath\Az.Accounts\*\Az.Accounts.psd1 | Should -Not -BeNullOrEmpty
   }
   It '4 section version numbers (VMware.PowerCLI)' {
-    Install-ModuleFast @imfParams 'VMware.VimAutomation.Common@13.2.0.22643733'
+    Install-ModuleFast @imfParams 'VMware.VimAutomation.Common=13.2.0.22643733'
     Get-Item $installTempPath\VMware*\*\*.psd1 | ForEach-Object {
       $moduleFolderVersion = $_ | Split-Path | Split-Path -Leaf
       Import-PowerShellDataFile -Path $_.FullName | Select-Object -ExpandProperty ModuleVersion | Should -Be $moduleFolderVersion
@@ -487,23 +487,23 @@ Describe 'Install-ModuleFast' -Tag 'E2E' {
 		| Should -BeGreaterThan ([version]'5.0.0')
   }
   It 'Errors trying to install prerelease over regular module' {
-    Install-ModuleFast @imfParams 'PrereleaseTest@0.0.1'
-    { Install-ModuleFast @imfParams 'PrereleaseTest@0.0.1-prerelease' }
+    Install-ModuleFast @imfParams 'PrereleaseTest=0.0.1'
+    { Install-ModuleFast @imfParams 'PrereleaseTest=0.0.1-prerelease' }
     | Should -Throw '*is newer than the requested prerelease version*'
   }
   It 'Errors trying to install older prerelease over regular module' {
-    Install-ModuleFast @imfParams 'PrereleaseTest@0.0.1'
-    { Install-ModuleFast @imfParams 'PrereleaseTest@0.0.1-prerelease' }
+    Install-ModuleFast @imfParams 'PrereleaseTest=0.0.1'
+    { Install-ModuleFast @imfParams 'PrereleaseTest=0.0.1-prerelease' }
     | Should -Throw '*is newer than the requested prerelease version*'
   }
   It 'Installs regular module over prerelease module with warning' {
-    Install-ModuleFast @imfParams 'PrereleaseTest@0.0.1-prerelease'
-    Install-ModuleFast @imfParams 'PrereleaseTest@0.0.1' -WarningVariable actual *>&1 | Out-Null
+    Install-ModuleFast @imfParams 'PrereleaseTest=0.0.1-prerelease'
+    Install-ModuleFast @imfParams 'PrereleaseTest=0.0.1' -WarningVariable actual *>&1 | Out-Null
     $actual | Should -BeLike '*is newer than existing prerelease version*'
   }
   It 'Installs newer prerelease with warning' {
-    Install-ModuleFast @imfParams 'PrereleaseTest@0.0.1-aprerelease'
-    Install-ModuleFast @imfParams 'PrereleaseTest@0.0.1-bprerelease' -WarningVariable actual *>&1 | Out-Null
+    Install-ModuleFast @imfParams 'PrereleaseTest=0.0.1-aprerelease'
+    Install-ModuleFast @imfParams 'PrereleaseTest=0.0.1-bprerelease' -WarningVariable actual *>&1 | Out-Null
     $actual | Should -BeLike '*is newer than existing prerelease version*'
   }
 
@@ -587,7 +587,7 @@ Describe 'Install-ModuleFast' -Tag 'E2E' {
 
   It 'Installs from CI File and Installs CI Pinned Version' {
     Set-Location $testDrive
-    Install-ModuleFast @imfParams -CI -Specification 'PreReleaseTest@0.0.1-prerelease'
+    Install-ModuleFast @imfParams -CI -Specification 'PreReleaseTest=0.0.1-prerelease'
     Get-Item 'requires.lock.json' | Should -Not -BeNullOrEmpty
 
     Remove-Item $imfParams.Destination -Recurse -Force

Test/Mocks/ModuleFast.requires.json

@@ -1,6 +1,6 @@
 {
   "PnP.PowerShell": "2.2.156-nightly",
-  "Pester": "@5.4.0",
+  "Pester": "=5.4.0",
   "Az.Accounts": ":[2.0.0, 2.13.2)",
   "ImportExcel": "latest",
   "PSScriptAnalyzer": "<=1.21.0"

Test/Mocks/ModuleFast.requires.psd1

@@ -2,6 +2,6 @@
   'ImportExcel'      = 'latest'
   'PnP.PowerShell'   = '2.2.156-nightly'
   'PSScriptAnalyzer' = '<=1.21.0'
-  'Pester'           = '@5.4.0'
+  'Pester'           = '=5.4.0'
   'Az.Accounts'      = ':[2.0.0, 2.13.2)'
 }

Test/Mocks/ModuleFastArray.requires.json

@@ -1,7 +1,7 @@
 [
   "ImportExcel",
-  "PnP.PowerShell@2.2.156-nightly",
+  "PnP.PowerShell=2.2.156-nightly",
   "PSScriptAnalyzer<=1.21.0",
-  "Pester@5.4.0",
+  "Pester=5.4.0",
   "Az.Accounts:[2.0.0, 2.13.2)"
 ]