Remove IDisposable usage of TestParserManager.

Author: N. Taylor Mullen

test/System.Web.Razor.Test/Parser/PartialParsing/CSharpPartialParsingTest.cs

@@ -85,20 +85,18 @@ public void ImplicitExpressionRejectsWholeIdentifierReplacementToKeyword()
             RazorEngineHost host = CreateHost();
             RazorEditorParser parser = new RazorEditorParser(host, @"C:\This\Is\A\Test\Path");
 
-            using (TestParserManager manager = new TestParserManager(parser))
-            {
-                StringTextBuffer old = new StringTextBuffer("foo @date baz");
-                StringTextBuffer changed = new StringTextBuffer("foo @if baz");
-                TextChange textChange = new TextChange(5, 4, old, 2, changed);
-                manager.InitializeWithDocument(old);
+            TestParserManager manager = new TestParserManager(parser);
+            StringTextBuffer old = new StringTextBuffer("foo @date baz");
+            StringTextBuffer changed = new StringTextBuffer("foo @if baz");
+            TextChange textChange = new TextChange(5, 4, old, 2, changed);
+            manager.InitializeWithDocument(old);
 
-                // Act
-                PartialParseResult result = manager.CheckForStructureChangesAndWait(textChange);
+            // Act
+            PartialParseResult result = manager.CheckForStructureChangesAndWait(textChange);
 
-                // Assert
-                Assert.Equal(PartialParseResult.Rejected, result);
-                Assert.Equal(2, manager.ParseCount);
-            }
+            // Assert
+            Assert.Equal(PartialParseResult.Rejected, result);
+            Assert.Equal(2, manager.ParseCount);
         }
 
         [Fact]
@@ -108,20 +106,18 @@ public void ImplicitExpressionRejectsWholeIdentifierReplacementToDirective()
             RazorEngineHost host = CreateHost();
             RazorEditorParser parser = new RazorEditorParser(host, @"C:\This\Is\A\Test\Path");
 
-            using (var manager = new TestParserManager(parser))
-            {
-                StringTextBuffer old = new StringTextBuffer("foo @date baz");
-                StringTextBuffer changed = new StringTextBuffer("foo @inherits baz");
-                TextChange textChange = new TextChange(5, 4, old, 8, changed);
-                manager.InitializeWithDocument(old);
+            TestParserManager manager = new TestParserManager(parser);
+            StringTextBuffer old = new StringTextBuffer("foo @date baz");
+            StringTextBuffer changed = new StringTextBuffer("foo @inherits baz");
+            TextChange textChange = new TextChange(5, 4, old, 8, changed);
+            manager.InitializeWithDocument(old);
 
-                // Act
-                PartialParseResult result = manager.CheckForStructureChangesAndWait(textChange);
+            // Act
+            PartialParseResult result = manager.CheckForStructureChangesAndWait(textChange);
 
-                // Assert
-                Assert.Equal(PartialParseResult.Rejected | PartialParseResult.SpanContextChanged, result);
-                Assert.Equal(2, manager.ParseCount);
-            }
+            // Assert
+            Assert.Equal(PartialParseResult.Rejected | PartialParseResult.SpanContextChanged, result);
+            Assert.Equal(2, manager.ParseCount);
         }
 
         [Fact]
@@ -415,52 +411,50 @@ public void ImplicitExpressionProvisionallyAcceptsCaseInsensitiveDotlessCommitIn
             StringTextBuffer old = new StringTextBuffer("foo @date baz");
             StringTextBuffer changed = new StringTextBuffer("foo @date. baz");
             TextChange textChange = new TextChange(9, 0, old, 1, changed);
-            using (TestParserManager manager = CreateParserManager())
+            TestParserManager manager = CreateParserManager();
+            Action<TextChange, PartialParseResult, string> applyAndVerifyPartialChange = (changeToApply, expectedResult, expectedCode) =>
             {
-                Action<TextChange, PartialParseResult, string> applyAndVerifyPartialChange = (changeToApply, expectedResult, expectedCode) =>
-                {
-                    PartialParseResult result = manager.CheckForStructureChangesAndWait(textChange);
+                PartialParseResult result = manager.CheckForStructureChangesAndWait(textChange);
 
-                    // Assert
-                    Assert.Equal(expectedResult, result);
-                    Assert.Equal(1, manager.ParseCount);
+                // Assert
+                Assert.Equal(expectedResult, result);
+                Assert.Equal(1, manager.ParseCount);
 
-                    ParserTestBase.EvaluateParseTree(manager.Parser.CurrentParseTree, new MarkupBlock(
-                        factory.Markup("foo "),
-                        new ExpressionBlock(
-                            factory.CodeTransition(),
-                            factory.Code(expectedCode).AsImplicitExpression(CSharpCodeParser.DefaultKeywords).Accepts(AcceptedCharacters.NonWhiteSpace)),
-                        factory.Markup(" baz")));
-                };
+                ParserTestBase.EvaluateParseTree(manager.Parser.CurrentParseTree, new MarkupBlock(
+                    factory.Markup("foo "),
+                    new ExpressionBlock(
+                        factory.CodeTransition(),
+                        factory.Code(expectedCode).AsImplicitExpression(CSharpCodeParser.DefaultKeywords).Accepts(AcceptedCharacters.NonWhiteSpace)),
+                    factory.Markup(" baz")));
+            };
 
-                manager.InitializeWithDocument(textChange.OldBuffer);
+            manager.InitializeWithDocument(textChange.OldBuffer);
 
-                // This is the process of a dotless commit when doing "." insertions to commit intellisense changes.
+            // This is the process of a dotless commit when doing "." insertions to commit intellisense changes.
 
-                // @date => @date.
-                applyAndVerifyPartialChange(textChange, PartialParseResult.Accepted | PartialParseResult.Provisional, "date.");
+            // @date => @date.
+            applyAndVerifyPartialChange(textChange, PartialParseResult.Accepted | PartialParseResult.Provisional, "date.");
 
-                old = changed;
-                changed = new StringTextBuffer("foo @date baz");
-                textChange = new TextChange(9, 1, old, 0, changed);
+            old = changed;
+            changed = new StringTextBuffer("foo @date baz");
+            textChange = new TextChange(9, 1, old, 0, changed);
 
-                // @date. => @date
-                applyAndVerifyPartialChange(textChange, PartialParseResult.Accepted, "date");
+            // @date. => @date
+            applyAndVerifyPartialChange(textChange, PartialParseResult.Accepted, "date");
 
-                old = changed;
-                changed = new StringTextBuffer("foo @DateTime baz");
-                textChange = new TextChange(5, 4, old, 8, changed);
+            old = changed;
+            changed = new StringTextBuffer("foo @DateTime baz");
+            textChange = new TextChange(5, 4, old, 8, changed);
 
-                // @date => @DateTime
-                applyAndVerifyPartialChange(textChange, PartialParseResult.Accepted, "DateTime");
+            // @date => @DateTime
+            applyAndVerifyPartialChange(textChange, PartialParseResult.Accepted, "DateTime");
 
-                old = changed;
-                changed = new StringTextBuffer("foo @DateTime. baz");
-                textChange = new TextChange(13, 0, old, 1, changed);
+            old = changed;
+            changed = new StringTextBuffer("foo @DateTime. baz");
+            textChange = new TextChange(13, 0, old, 1, changed);
 
-                // @DateTime => @DateTime.
-                applyAndVerifyPartialChange(textChange, PartialParseResult.Accepted | PartialParseResult.Provisional, "DateTime.");
-            }
+            // @DateTime => @DateTime.
+            applyAndVerifyPartialChange(textChange, PartialParseResult.Accepted | PartialParseResult.Provisional, "DateTime.");
         }
 
         [Fact]