Some more test fixes to support remote instances#10060
Some more test fixes to support remote instances#10060potatoqualitee merged 5 commits intodevelopmentfrom
Conversation
|
Will work on this tomorrow... |
|
Running more tests now, so more commits will be added... |
|
Ready to be reviewed and merged... |
| } | ||
| } | ||
| $results = Test-DbaSpn -ComputerName $env:COMPUTERNAME -WarningAction SilentlyContinue | ||
| $results = Test-DbaSpn -ComputerName $TestConfig.instance1 |
There was a problem hiding this comment.
🤔 how did this pass on appveyor?
There was a problem hiding this comment.
On AppVeyor this is the output:
There was a problem hiding this comment.
I think this test is only usefull in a bigger test environment with a domain and kerberos like I have it in my lab.
| $null = Invoke-DbaQuery -SqlInstance $instance -Database tempdb -Query $dropTableQuery | ||
| } | ||
|
|
||
| $null = Remove-DbaLogin -SqlInstance $TestConfig.instance1, $TestConfig.instance2 -Login 'claudio', 'port', 'tester' |
There was a problem hiding this comment.
No. Initialize-TestLogin removes all the test logins. So there are no more logins to removeand so this command only outputs a warning about not finding those logins. I capture all the warnings while running the tests so I found this and removed all the code that does no work but only outputs warnings.
2 participants
This will not be the last pr as these tests still fail:
Get-DbaExternalProcess.Tests.ps1
Import-DbaCsv.Tests.ps1
Stop-DbaExternalProcess.Tests.ps1
I will try to work on those the next days.