* Get connection string call was changing the connection info (ref object) received from cache. Changing it to just get and make changes to only the returned string..
* Change to ensure PR validation works
* Initial Investigation
* Working code with include, connect, on error and tests
* Adding some loc strings
* Some cleanup and more tests
* Some dummy change to trigger build
* Adding PR comments
* Addressing PR comments
* Excplitly erroring out on some failing commands that was not happening before
* Ensuring the error message goes through without getting converted to different error
* Part1 : Changes to make cmdcmd script to work with parameters in script
* Stop SQL intellisense for SQLCMD
* Adding test for Intellisense handling of SQLCMD page
* Removing unintentional spacing changes caused by formatting
* Updating with smaller CR comments. Will discuss regarding script vs other options in batch info
* Removing unintentional change
* Adding latest PR comments
* updating dacfx with private nuget for testing
* Updating to real dacfx-preview version
* Removing additional refs added by nuget package manager (these were not there previously)
* Fix Schema compare Publish and Script generation task to return back error messages correctly.
* Elaborating test a bit more to cover tasks
* Validate one task at a time for better clarity
* send back only errors not whole messages since task view doesnt have a good way (other than hover text) to show long messages
* Fixing the negative test cases
* First cut of scmp Save related changes and some test refactoring
* Adding Exclude/Include objects in saving
* Add diff entry validation as part of test
* Adding PR comments - major change is change to nameparts in place of name hence preserving any "."/"[" in name and avoiding any string operations
* One more UT scenario addition for create excluded object
* Adding some default values to match DacFx deploy Extension and other tools. This help the default behavior but doesn't fix the issue.
* Fixing a test for default values and adding comment
* fixing the order of setting backup properties so that they are correctly propogated to actual action
* Adding a backup type specific test and enabling 'few' old ones which seem to work consistentenly
* Adding finally block, close connection and name change as per PR comments
* Proposed changes to stop query execution if an exception has occured before/during batch execution
* Sending the error message was causing a test to fail - so removing for now. We can update the test to expect this is needed.
* Initial code for Including/Excluding individual changes (no tests added yet)
* Adding Exclude include tests. Default options call and additional options tests.
* Taking PR comments
* Retry in test for reliability
* Fix for : 4045: Cannot cancel a Query. Query runs too long.
HandleExecuteRequest was returning a task to awaiter - which was getting waited on. changed it to async function to start task in new thread and return nothing to awaiter . Also the cancellation token set by cancel request was getting checked only after making the connection to execute batches. Added an additional check for cancellation token befor the connection has been made.
Fix for 4319: Error showing dbs when using AAD in... 1.5.0-alpha.74
David has already created a new version of SMO nuget with the fix. - incorporating the same (150.18096.0-preview).
* Adding awaitable internal task for tests to run properly
* Adding more cancel tests
* First cut of CMS get Server Api - returns only names
* Adding Proper naming and some more functions
* Adding add remove Server and tests for it
* Changing relative path to derive from KeyChain.Urn and Nuget update
* Adding async thread for requests
* Addressing CR comments and adding one more test
* Adressed PR comments around naming
Udeesha Gautam