Fix issue PowerShell#1013 as well by making sure the rule looks only the LHS

Author: bergmeister

Rules/AvoidAssignmentToAutomaticVariable.cs

@@ -44,9 +44,9 @@ public IEnumerable<DiagnosticRecord> AnalyzeScript(Ast ast, string fileName)
             if (ast == null) throw new ArgumentNullException(Strings.NullAstErrorMessage);
 
             IEnumerable<Ast> assignmentStatementAsts = ast.FindAll(testAst => testAst is AssignmentStatementAst, searchNestedScriptBlocks: true);
-            foreach (var assignmentStatementAst in assignmentStatementAsts)
+            foreach (AssignmentStatementAst assignmentStatementAst in assignmentStatementAsts)
             {
-                var variableExpressionAst = assignmentStatementAst.Find(testAst => testAst is VariableExpressionAst, searchNestedScriptBlocks: false) as VariableExpressionAst;
+                var variableExpressionAst = assignmentStatementAst.Left.Find(testAst => testAst is VariableExpressionAst, searchNestedScriptBlocks: false) as VariableExpressionAst;
                 if (variableExpressionAst == null) { continue; }
                 var variableName = variableExpressionAst.VariablePath.UserPath;
                 if (_readOnlyAutomaticVariables.Contains(variableName, StringComparer.OrdinalIgnoreCase))

Tests/Rules/AvoidAssignmentToAutomaticVariable.tests.ps1

@@ -77,6 +77,10 @@ Describe "AvoidAssignmentToAutomaticVariables" {
             $exceptionThrown | Should -Be $false
         }
 
+        It "Does not flag RHS of variable assignment (Bug in 1.17.0, issue 1013)" {
+            [System.Array] $warnings = Invoke-ScriptAnalyzer -ScriptDefinition '[foo]::bar = $true'
+            $warnings.Count | Should -Be 0
+        }
 
         It "Setting Variable <VariableName> throws exception in applicable PowerShell version to verify the variables is read-only" -TestCases $testCases_ReadOnlyVariables {
             param ($VariableName, $ExpectedSeverity, $OnlyPresentInCoreClr)