Merge pull request #2359 from retailcoder/next

Additional tests (ref. #2206)

Author: retailcoder

RetailCoder.VBE/API/ParserState.cs

@@ -52,7 +52,6 @@ public sealed class ParserState : IParserState, IDisposable
         private RubberduckParserState _state;
         private AttributeParser _attributeParser;
         private RubberduckParser _parser;
-        private Sinks _sinks;
         private VBE _vbe;
 
         public ParserState()
@@ -156,10 +155,6 @@ public void Dispose()
                 _state.StateChanged -= _state_StateChanged;
             }
 
-            if (_sinks != null)
-            {
-                _sinks.Dispose();
-            }
 
             _vbe.Release();            
             _disposed = true;

RubberduckTests/Grammar/VBAParserTests.cs

@@ -1503,13 +1503,41 @@ End Type
             AssertTree(parseResult.Item1, parseResult.Item2, "//udtMember", matches => matches.Count == 13);
         }
 
+
+        [TestMethod]
+        public void TestNestedParensForLiteralExpression()
+        {
+            //Assert.Inconclusive("See issue #2206");
+            const string code = @"
+Sub Test()
+    Dim foo As Integer
+    foo = ((42) + ((12)))
+End Sub
+";
+            var parseResult = Parse(code);
+            AssertTree(parseResult.Item1, parseResult.Item2, "//literalExpression", matches => matches.Count == 2);
+        }
+        
+        [TestMethod]
+        public void TestParensForByValSingleArg()
+        {
+            //Assert.Inconclusive("See issue #2206");
+            const string code = @"
+Sub Test()
+    DoSomething (foo)
+End Sub
+";
+            var parseResult = Parse(code);
+            AssertTree(parseResult.Item1, parseResult.Item2, "//argumentExpression", matches => matches.Count == 1);
+        }
+
         [TestMethod]
-        public void TestNestedParensForByValArgument()
+        public void TestParensForByValFirstArg()
         {
-            Assert.Inconclusive("See issue #");
+            Assert.Inconclusive("See issue #2206");
             const string code = @"
 Sub Test()
-    DoSomething (foo), (bar)
+    DoSomething (foo), bar
 End Sub
 ";
             var parseResult = Parse(code);