Clarifying comments

Also a minor change to a test to simplify and clarify it's purpose

Author: Jaykul

Source/Classes/AliasVisitor.ps1

@@ -2,6 +2,9 @@ using namespace System.Management.Automation.Language
 using namespace System.Collections.Generic
 
 # This is used only to parse the parameters to New|Set|Remove-Alias
+# NOTE: this is _part of_ the implementation of AliasVisitor, but ...
+#       PowerShell can't handle nested classes so I left it outside,
+#       but I kept it here in this file.
 class AliasParameterVisitor : AstVisitor {
     [string]$Parameter = $null
     [string]$Command = $null

Tests/Private/GetCommandAlias.Tests.ps1

@@ -168,13 +168,9 @@ Describe "GetCommandAlias" {
             $Result | Should -Be 'Alias1', 'Alias2', 'Alias3'
         }
 
-        It "Ignores aliases that already have global scope" {
+        It "Does not fail when removing aliases that were ignored because of global scope" {
             $Result = &$CommandInfo -Ast {
                 Set-Alias -Name Alias1 -Scope Global -Value Write-Verbose
-                Set-Alias -Scope Global -Value 'Write-Verbose' -Name 'Alias2'
-                Set-Alias -Scope Global 'Alias3' 'Write-Verbose'
-                Set-Alias -Value 'Write-Verbose' 'Alias4' -Scope Global
-                Set-Alias 'Alias5' -Value 'Write-Verbose' -Scope Global
                 Remove-Alias -Name Alias1
             }.Ast