xPackage fixes and updates

[iainbrighton]

xPackage:

• No longer checks for package installation when a reboot is required
  • Fixes xPackage should allow some option to not validate installation until after reboot #52
• Ensures a space is added to MSI installation arguments
  • Fixes xPackage arguments missing a space #195
• Adds RunAsCredential parameter to permit installing packages with specific user account
  • Fixes xPackage: RunAsCredential Parameter Removed #221
• Fixes null verbose log output error
  • Fixes xPackage: Mispelled variable errLogPath and inverse conditional (minor) #224

---

This change is 

[PlagueHO]

@iainbrighton - change looks great. I didn't review using the HQRM guidelines though. Sorry about the Out-Null nit picks :)

---

Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

---

Tests/Unit/MSFT_xPackageResource.Tests.ps1, line 38 at r1 (raw file):

                $script:testExecutablePath = Join-Path -Path $script:testDirectoryPath -ChildPath 'TestExecutable.exe'
                
                New-TestExecutable -DestinationPath $script:testExecutablePath | Out-Null

Not lines that you wrote, but could you assign these two lines to $null rather than pipe to Out-Null?

---

Comments from Reviewable

[iainbrighton]

Review status: 3 of 4 files reviewed at latest revision, 1 unresolved discussion.

---

Tests/Unit/MSFT_xPackageResource.Tests.ps1, line 38 at r1 (raw file):

Previously, PlagueHO (Daniel Scott-Raynsford) wrote…

> Not lines that you wrote, but could you assign these two lines to $null rather than pipe to Out-Null?

No problem. Done!

---

Comments from Reviewable

[TravisEz13]

piping to $null looks cleaner to me, but LGTM

---

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

---

Comments from Reviewable

[TravisEz13]

Please resolve all merge conflicts. For more information see GitHowTo - 30. Resolving Conflicts

[iainbrighton]

I don't see any merge conflicts?

---

Review status: all files reviewed at latest revision, 1 unresolved discussion, some commit checks failed.

---

Comments from Reviewable

[TravisEz13]

GitHub says readme.md has a conflict

[TravisEz13]

Reviewed 2 of 3 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

---

Comments from Reviewable