Adding latest PR comments

microsoft · Aug 9, 2019 · 516c54c · 516c54c
1 parent 26eca53
commit 516c54c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/test/Microsoft.SqlTools.ManagedBatchParser.IntegrationTests/BatchParser/BatchParserTests.cs b/test/Microsoft.SqlTools.ManagedBatchParser.IntegrationTests/BatchParser/BatchParserTests.cs
@@ -346,7 +346,7 @@ public void VerifyRunSqlCmd()
         {
             using (ExecutionEngine executionEngine = new ExecutionEngine())
             {
-                string sqlCmdQuery = @"
+                const string sqlCmdQuery = @"
 :setvar __var1 1
 :setvar __var2 2
 :setvar __IsSqlCmdEnabled " + "\"True\"" + @"

diff --git a/test/Microsoft.SqlTools.ServiceLayer.IntegrationTests/LanguageServer/LanguageServiceTests.cs b/test/Microsoft.SqlTools.ServiceLayer.IntegrationTests/LanguageServer/LanguageServiceTests.cs
@@ -383,7 +383,7 @@ public async Task HandleRequestToChangeToSqlcmdFile()
                 }, eventContextSqlCmd.Object);
                 await langService.DelayedDiagnosticsTask;
 
-                Assert.True(countOfValidationCalls == 2, string.Format("Validation should be called 2 time but is called {0} times", countOfValidationCalls));
+                Assert.True(countOfValidationCalls == 2, $"Validation should be called 2 time but is called {countOfValidationCalls} times");
             }
             finally
             {
@@ -397,7 +397,7 @@ public async Task HandleRequestToChangeToSqlcmdFile()
         private bool ValidateNotification(PublishDiagnosticsNotification notif, int errors, ref int countOfValidationCalls)
         {
             countOfValidationCalls++;
-            Assert.True(notif.Diagnostics.Length == errors, string.Format("Notification errors {0} are not as expected {1}", notif.Diagnostics.Length, errors));
+            Assert.True(notif.Diagnostics.Length == errors, $"Notification errors {notif.Diagnostics.Length} are not as expected {errors}");
             return true;
         }
     }