Bonus Assertions

Author: JustinGrote

test/PowerShellEditorServices.Test.Shared/Refactoring/Variables/RefactorVariableTestCases.cs

@@ -8,8 +8,8 @@ public class RefactorVariableTestCases
         new ("SimpleVariableAssignment.ps1",                   Line:  1, Column:  1),
         new ("VariableCommandParameter.ps1",                   Line:  3, Column: 17),
         new ("VariableCommandParameter.ps1",                   Line: 10, Column:  9),
-        new ("VariableCommandParameterSplatted.ps1",           Line: 19, Column: 10),
-        new ("VariableCommandParameterSplatted.ps1",           Line:  8, Column:  6),
+        new ("VariableCommandParameterSplatted.ps1",           Line: 16, Column:  5),
+        new ("VariableCommandParameterSplatted.ps1",           Line: 21, Column: 11),
         new ("VariableDotNotationFromInnerFunction.ps1",       Line:  1, Column:  1),
         new ("VariableDotNotationFromInnerFunction.ps1",       Line: 11, Column: 26),
         new ("VariableInForeachDuplicateAssignment.ps1",       Line:  6, Column: 18),

test/PowerShellEditorServices.Test/Refactoring/RenameHandlerTests.cs

@@ -64,9 +64,10 @@ public async void RenamedFunction(RenameTestTarget s)
 
         ScriptFile scriptFile = workspace.GetFile(testScriptUri);
 
+        Assert.NotEmpty(response.Changes[testScriptUri]);
+
         string actual = GetModifiedScript(scriptFile.Contents, response.Changes[testScriptUri].ToArray());
 
-        Assert.NotEmpty(response.Changes[testScriptUri]);
         Assert.Equal(expected, actual);
     }
 
@@ -85,6 +86,9 @@ public async void RenamedVariable(RenameTestTarget s)
 
         ScriptFile scriptFile = workspace.GetFile(testScriptUri);
 
+        Assert.NotNull(response);
+        Assert.NotEmpty(response.Changes[testScriptUri]);
+
         string actual = GetModifiedScript(scriptFile.Contents, response.Changes[testScriptUri].ToArray());
 
         Assert.Equal(expected, actual);