Bug fix for setting 'unknown' repository APIVersion (#1377)

Author: alerickson

src/code/RepositorySettings.cs

@@ -204,14 +204,6 @@ public static PSRepositoryInfo UpdateRepositoryStore(string repoName, Uri repoUr
                 }
             }
 
-            // determine if either 1 of 5 values are attempting to be set: Uri, Priority, Trusted, APIVerison, CredentialInfo.
-            // if none are (i.e only Name parameter was provided, write error)
-            if (repoUri == null && repoPriority == defaultPriority && _trustedNullable == null && repoCredentialInfo == null && (apiVersion == PSRepositoryInfo.APIVersion.unknown || apiVersion == null))
-            {
-                errorMsg = "Must set Uri, Priority, Trusted, ApiVersion, or CredentialInfo parameter";
-                return null;
-            }
-
             if (!cmdletPassedIn.ShouldProcess(repoName, "Set repository's value(s) in repository store"))
             {
                 return null;

src/code/SetPSResourceRepository.cs

@@ -111,6 +111,22 @@ protected override void BeginProcessing()
 
         protected override void ProcessRecord()
         {
+            // determine if either 1 of 5 values are attempting to be set: Uri, Priority, Trusted, APIVerison, CredentialInfo.
+            // if none are (i.e only Name parameter was provided, write error)
+            if (ParameterSetName.Equals(NameParameterSet) &&
+                !MyInvocation.BoundParameters.ContainsKey(nameof(Uri)) &&
+                !MyInvocation.BoundParameters.ContainsKey(nameof(Priority)) &&
+                !MyInvocation.BoundParameters.ContainsKey(nameof(Trusted)) &&
+                !MyInvocation.BoundParameters.ContainsKey(nameof(ApiVersion)) &&
+                !MyInvocation.BoundParameters.ContainsKey(nameof(CredentialInfo)))
+            {
+                ThrowTerminatingError(new ErrorRecord(
+                    new ArgumentException("Must set Uri, Priority, Trusted, ApiVersion, or CredentialInfo parameter"),
+                    "SetRepositoryParameterBindingFailure",
+                    ErrorCategory.InvalidArgument,
+                    this));
+            }
+
             if (MyInvocation.BoundParameters.ContainsKey(nameof(Uri)))
             {
                 if (!Utils.TryCreateValidUri(Uri, this, out _uri, out ErrorRecord errorRecord))

test/ResourceRepositoryTests/SetPSResourceRepository.Tests.ps1

@@ -100,7 +100,7 @@ Describe "Test Set-PSResourceRepository" -tags 'CI' {
 
     It "not set repository and write error given just Name parameter" {
         Register-PSResourceRepository -Name $TestRepoName1 -Uri $tmpDir1Path
-        {Set-PSResourceRepository -Name $TestRepoName1 -ErrorAction Stop} | Should -Throw -ErrorId "ErrorInNameParameterSet,Microsoft.PowerShell.PSResourceGet.Cmdlets.SetPSResourceRepository"
+        {Set-PSResourceRepository -Name $TestRepoName1 -ErrorAction Stop} | Should -Throw -ErrorId "SetRepositoryParameterBindingFailure,Microsoft.PowerShell.PSResourceGet.Cmdlets.SetPSResourceRepository"
     }
 
     $testCases = @{Type = "contains *";     Name = "test*Repository"; ErrorId = "ErrorInNameParameterSet"},
@@ -344,4 +344,16 @@ Describe "Test Set-PSResourceRepository" -tags 'CI' {
         $repo.ApiVersion | Should -Be "local"
         $repo.Priority | Should -Be 25
     }
+
+    It "should not change ApiVersion of repository if -ApiVersion parameter was not used" {
+        Register-PSResourceRepository -Name $TestRepoName1 -Uri $tmpDir1Path
+        $repo = Get-PSResourceRepository $TestRepoName1
+        $repoApiVersion = $repo.ApiVersion
+        $repoApiVersion | Should -Be "local"
+
+        Set-PSResourceRepository -Name $TestRepoName1 -ApiVersion "unknown" -ErrorVariable err -ErrorAction SilentlyContinue
+        $repo = Get-PSResourceRepository $TestRepoName1
+        $repo.ApiVersion | Should -Be "unknown"
+        $err.Count | Should -Be 0
+    }
 }