Andreas Jordan
Andreas Jordan
Ok, @niphlod - now you can have a look. I basically moved your code to Connect-DbaInstance to generally support this case and just changed Invoke-DbaQuery a bit so that AppendConnectionString...
Ups, just forgot the case when AppendConnectionString is used with a simple open connection. Will add this soon...
Yes. We can just merge this PR here to solve the issue. It's mostly your code - just a bit rearranged.
Just added Purpose and Approach to the initial post. @potatoqualitee if you need anything else before merging, please let me know.
@potatoqualitee / @jpomfret What can I do to get this merged?
I think we should skip this test on AppVeyor and only test this command from time to time on a local instance with a big database. What do you think?
New idea for those tests: We could use a freely availably database backup for the tests. I refactored the test to use the Stackoverflow database: https://github.com/BrentOzarULTD/Stack-Overflow-Database/releases/download/20230114/StackOverflowMini.bak I also changed the...
@potatoqualitee : If I change `pester.groups.p1` here in this branch, this might result in a merge conflict, because the file was changes by a recently merged pull request. Can you...
This fixes the issue: ``` PS C:\Users\initialAdmin> 'y' | winrm quickconfig WinRM service is already running on this machine. WinRM is not set up to allow remote access to this...
This is the script I have used to setu up the maschine: https://github.com/andreasjordan/testing-dbatools/blob/main/setup_azure.ps1