more test debuging

Author: alerickson

src/code/InstallHelper.cs

@@ -173,6 +173,7 @@ public IEnumerable<PSResourceInfo> BeginInstallPackages(
                 skipDependencyCheck: skipDependencyCheck,
                 scope: scope ?? ScopeType.CurrentUser);
 
+            _cmdletPassedIn.WriteVerbose("Returning from BeginInstallpackages");
             return installedPkgs;
         }
 
@@ -341,11 +342,13 @@ private List<PSResourceInfo> ProcessRepositories(
 
                 repositoryNamesToSearch.Add(repoName);
 
+                _cmdletPassedIn.WriteVerbose("InstallHelper line 345");
                 List<PSResourceInfo> installedPkgs = InstallPackages(_pkgNamesToInstall.ToArray(), currentRepository, currentServer, currentResponseUtil, scope, skipDependencyCheck, findHelper);
                 foreach (PSResourceInfo pkg in installedPkgs)
                 {
                     _pkgNamesToInstall.RemoveAll(x => x.Equals(pkg.Name, StringComparison.InvariantCultureIgnoreCase));
                 }
+                _cmdletPassedIn.WriteVerbose("InstallHelper line 351");
 
                 allPkgsInstalled.AddRange(installedPkgs);
             }
@@ -359,6 +362,7 @@ private List<PSResourceInfo> ProcessRepositories(
                     ErrorCategory.InvalidData,
                     _cmdletPassedIn));
             }
+            _cmdletPassedIn.WriteVerbose("InstallHelper line 365");
 
             return allPkgsInstalled;
         }

src/code/SavePSResource.cs

@@ -313,10 +313,16 @@ private void ProcessSaveHelper(string[] pkgNames, string pkgVersion, bool pkgPre
                 tmpPath: _tmpPath,
                 pkgsInstalled: new HashSet<string>(StringComparer.InvariantCultureIgnoreCase));
 
+            WriteVerbose("Returned from BeginPackages in Save");
+
             if (PassThru)
             {
                 foreach (PSResourceInfo pkg in installedPkgs)
                 {
+                    if (pkg == null)
+                    {
+                        WriteVerbose("pkg is null");
+                    }
                     WriteObject(pkg);
                 }
             }

src/code/V3ServerAPICalls.cs

@@ -754,6 +754,11 @@ private Stream InstallVersion(string packageName, string version, out ErrorRecor
             _cmdletPassedIn.WriteDebug("In V3ServerAPICalls::InstallVersion()");
             if (!NuGetVersion.TryParse(version, out NuGetVersion requiredVersion))
             {
+                _cmdletPassedIn.WriteDebug("InstallVersion - could not parse version");
+                if (string.IsNullOrEmpty(version))
+                {
+                    _cmdletPassedIn.WriteDebug("version is null or empty");
+                }
                 errRecord = new ErrorRecord(
                     new ArgumentException($"Version {version} to be installed is not a valid NuGet version."),
                     "InstallVersionFailure",
@@ -762,6 +767,7 @@ private Stream InstallVersion(string packageName, string version, out ErrorRecor
 
                 return null;
             }
+            _cmdletPassedIn.WriteDebug("InstallVersion 765");
 
             var inst = InstallHelper(packageName, requiredVersion, out errRecord);
             _cmdletPassedIn.WriteDebug("Returning from V3ServerAPICalls::InstallVersion()");

test/InstallPSResourceTests/InstallPSResourceV3Server.Tests.ps1

@@ -393,7 +393,7 @@ Describe 'Test Install-PSResource for V3Server scenarios' -tags 'CI' {
     }
 
     It "Install module and its dependencies" {
-        $res = Install-PSResource 'TestModuleWithDependencyE' -Repository $NuGetGalleryName -TrustRepository -PassThru
+        $res = Install-PSResource 'TestModuleWithDependencyE' -Repository $NuGetGalleryName -TrustRepository -PassThru -Verbose -Debug 
         $res.Length | Should -Be 4
     }
 }