fixed broken test

retailcoder · retailcoder · commit 0f42c9c1c45c · 2017-03-02T10:58:09.000-05:00
diff --git a/Rubberduck.Parsing/Symbols/BoundExpressionVisitor.cs b/Rubberduck.Parsing/Symbols/BoundExpressionVisitor.cs
@@ -48,9 +48,10 @@ private void Visit(
             bool isAssignmentTarget,
             bool hasExplicitLetStatement)
         {
-            if (isAssignmentTarget && expression.Context.Parent is VBAParser.IndexExprContext)
+            if (isAssignmentTarget && expression.Context.Parent is VBAParser.IndexExprContext && !expression.ReferencedDeclaration.IsArray)
             {
                 // 'SomeDictionary' is not the assignment target in 'SomeDictionary("key") = 42'
+                // ..but we want to treat array index assignment as assignment to the array itself.
                 isAssignmentTarget = false;
             }
 

Original file line number	Diff line number	Diff line change
`@@ -48,9 +48,10 @@ private void Visit(`
`48`	`48`	`bool isAssignmentTarget,`
`49`	`49`	`bool hasExplicitLetStatement)`
`50`	`50`	`{`
`51`		`- if (isAssignmentTarget && expression.Context.Parent is VBAParser.IndexExprContext)`
	`51`	`+ if (isAssignmentTarget && expression.Context.Parent is VBAParser.IndexExprContext && !expression.ReferencedDeclaration.IsArray)`
`52`	`52`	`{`
`53`	`53`	`// 'SomeDictionary' is not the assignment target in 'SomeDictionary("key") = 42'`
	`54`	`+ // ..but we want to treat array index assignment as assignment to the array itself.`
`54`	`55`	`isAssignmentTarget = false;`
`55`	`56`	`}`
`56`	`57`