Test-DbaConnection - added -SkipPSRemoting / fixed getting Port

[andreasjordan]

Type of Change

• Bug fix (non-breaking change, fixes [Bug] Test-DBAConnection ignoring -SQLCredential #6157 )
• New feature (non-breaking change, adds functionality, fixes Test-DbaConnection - PSRemoting #6148 )
• Breaking change (effects multiple commands or functionality, fixes # )
• Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
• Adding code coverage to existing functionality
• Pester test is included
• If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
• Nunit test is included
• Documentation
• Build system

[potatoqualitee]

Thank you! DatabaseIsAccessible should be $false and Status Unknown for that last one.

[andreasjordan]

I thought it this way: DatabaseIsAccessible and Status are $null, because there is no database "unknown" in the instance. A not-NULL value would indicate, that the database can be found in the SMO.
And Status is of type Microsoft.SqlServer.Management.Smo.DatabaseStatus which has no "Unknown".

A new commit with a refresh on .Databases is on its way to ensure that SMO is up to date when using it.

[potatoqualitee]

If we're comparing null, the following may be equally useful? if (-not (Get-DbaDatabase -SqlInstance..)

[potatoqualitee]

Looks good, thank you!