Fixed all variable PrepareRenameHandler Tests

Author: JustinGrote

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

@@ -5,30 +5,30 @@ public class RefactorVariableTestCases
 {
     public static RenameTestTarget[] TestCases =
     [
-        new ("SimpleVariableAssignment.ps1",                   Line: 1,  Column: 1  ),
-        new ("VariableRedefinition.ps1",                       Line: 1,  Column: 1  ),
-        new ("VariableNestedScopeFunction.ps1",                Line: 1,  Column: 1  ),
-        new ("VariableInLoop.ps1",                             Line: 1,  Column: 1  ),
-        new ("VariableInPipeline.ps1",                         Line: 23, Column: 2  ),
-        new ("VariableInScriptblockScoped.ps1",                Line: 36, Column: 3  ),
-        new ("VariablewWithinHastableExpression.ps1",          Line: 46, Column: 3  ),
-        new ("VariableNestedFunctionScriptblock.ps1",          Line: 20, Column: 4  ),
-        new ("VariableWithinCommandAstScriptBlock.ps1",        Line: 75, Column: 3  ),
-        new ("VariableWithinForeachObject.ps1",                Line: 1,  Column: 2  ),
-        new ("VariableusedInWhileLoop.ps1",                    Line: 5,  Column: 2  ),
-        new ("VariableInParam.ps1",                            Line: 16, Column: 24 ),
-        new ("VariableCommandParameter.ps1",                   Line: 9,  Column: 10 ),
-        new ("VariableCommandParameter.ps1",                   Line: 17, Column: 3  ),
-        new ("VariableScriptWithParamBlock.ps1",               Line: 30, Column: 1  ),
-        new ("VariableNonParam.ps1",                           Line: 1,  Column: 7  ),
-        new ("VariableParameterCommandWithSameName.ps1",       Line: 13, Column: 9  ),
-        new ("VariableCommandParameterSplatted.ps1",           Line: 10, Column: 21 ),
-        new ("VariableCommandParameterSplatted.ps1",           Line: 5,  Column: 16 ),
-        new ("VariableInForeachDuplicateAssignment.ps1",       Line: 18, Column: 6  ),
-        new ("VariableInForloopDuplicateAssignment.ps1",       Line: 14, Column: 9  ),
-        new ("VariableNestedScopeFunctionRefactorInner.ps1",   Line: 5,  Column: 3  ),
-        new ("VariableSimpleFunctionParameter.ps1",            Line: 9,  Column: 6  ),
-        new ("VariableDotNotationFromInnerFunction.ps1",       Line: 26, Column: 11 ),
-        new ("VariableDotNotationFromInnerFunction.ps1",       Line: 1,  Column: 1  )
+        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 ("VariableDotNotationFromInnerFunction.ps1",       Line:  1, Column:  1),
+        new ("VariableDotNotationFromInnerFunction.ps1",       Line: 11, Column: 26),
+        new ("VariableInForeachDuplicateAssignment.ps1",       Line:  6, Column: 18),
+        new ("VariableInForloopDuplicateAssignment.ps1",       Line:  9, Column: 14),
+        new ("VariableInLoop.ps1",                             Line:  1, Column:  1),
+        new ("VariableInParam.ps1",                            Line: 24, Column: 16),
+        new ("VariableInPipeline.ps1",                         Line:  2, Column: 23),
+        new ("VariableInScriptblockScoped.ps1",                Line:  2, Column: 16),
+        new ("VariableNestedFunctionScriptblock.ps1",          Line:  4, Column: 20),
+        new ("VariableNestedScopeFunction.ps1",                Line:  1, Column:  1),
+        new ("VariableNestedScopeFunctionRefactorInner.ps1",   Line:  3, Column:  5),
+        new ("VariableNonParam.ps1",                           Line:  7, Column:  1),
+        new ("VariableParameterCommandWithSameName.ps1",       Line:  9, Column: 13),
+        new ("VariableRedefinition.ps1",                       Line:  1, Column:  1),
+        new ("VariableScriptWithParamBlock.ps1",               Line:  1, Column: 30),
+        new ("VariableSimpleFunctionParameter.ps1",            Line:  6, Column:  9),
+        new ("VariableusedInWhileLoop.ps1",                    Line:  2, Column:  5),
+        new ("VariableWithinCommandAstScriptBlock.ps1",        Line:  3, Column: 75),
+        new ("VariableWithinForeachObject.ps1",                Line:  2, Column:  1),
+        new ("VariablewWithinHastableExpression.ps1",          Line:  3, Column: 46),
     ];
 }

test/PowerShellEditorServices.Test/Refactoring/PrepareRenameHandlerTests.cs

@@ -82,6 +82,8 @@ public async Task FindsVariable(RenameTestTarget s)
         Assert.NotNull(result);
         Assert.True(result?.DefaultBehavior?.DefaultBehavior);
     }
+
+    // TODO: Bad Path Tests (strings, parameters, etc.)
 }
 
 public static partial class RenameTestTargetExtensions