Make previously inconclusive tests pass or ignored

Author: MDoerner

RubberduckTests/CodeExplorer/CodeExplorerProjectViewModelTests.cs

@@ -127,8 +127,8 @@ public void IsNotFiltered(string filter)
         [Category("Code Explorer")]
         [TestCase(DeclarationType.Parameter)]
         [TestCase(DeclarationType.LineLabel)]
-        [TestCase(DeclarationType.UnresolvedMember)]        // TODO: Inconclusive pending test setup that will actually create one :-/
-        //[TestCase(DeclarationType.BracketedExpression)]   // TODO: This causes a parser error in testing due to no host application.
+        [TestCase(DeclarationType.UnresolvedMember, Ignore = "Pending test setup that will actually create one")]
+        [TestCase(DeclarationType.BracketedExpression, Ignore = "This causes a parser error in testing due to no host application.")]
         public void TrackedDeclarations_ExcludesNonNodeTypes(DeclarationType excluded)
         {
             var declarations = CodeExplorerTestSetup.TestProjectOneDeclarations;
@@ -148,7 +148,7 @@ public void TrackedDeclarations_ExcludesNonNodeTypes(DeclarationType excluded)
         [Category("Code Explorer")]
         [TestCase(DeclarationType.Variable)]
         [TestCase(DeclarationType.Control)]
-        [TestCase(DeclarationType.Constant)]    // TODO: Inconclusive pending test setup that will actually create one :-/
+        [TestCase(DeclarationType.Constant, Ignore = "Pending test setup that will actually create one.")]
         public void TrackedDeclarations_ExcludesMemberEnclosedTypes(DeclarationType excluded)
         {
             var declarations = CodeExplorerTestSetup.TestProjectOneDeclarations;

RubberduckTests/Inspections/ThunderCode/ThunderCodeInspectionTests.cs

@@ -15,19 +15,36 @@ namespace RubberduckTests.Inspections.ThunderCode
     public class ThunderCodeInspectionTests
     {
         [Test]
+        [Category("Inspections")]
         [TestCase(1, "Public Sub foo\u00A0bar()" + @"
 End Sub")]
         [TestCase(0, @"Public Sub foo()
 End Sub")]
-        [TestCase(0, @"Public Sub foo bar()
-End Sub")] // Correctly provokes a parser error
         public void NonBreakingSpaceIdentifier_ReturnsResult(int expectedCount, string inputCode)
         {
             var func = new Func<RubberduckParserState, IInspection>(state =>
                 new NonBreakingSpaceIdentifierInspection(state));
             ThunderCatsGo(func, inputCode, expectedCount);
         }
 
+        [Test]
+        [Category("Inspections")]
+        [TestCase(0, @"Public Sub foo bar()
+End Sub")]
+        public void NonBreakingSpaceIdentifier_IllegalInputCausesParserError(int expectedCount, string inputCode)
+        {
+            var vbe = MockVbeBuilder.BuildFromSingleStandardModule(inputCode, out var component);
+            var parser = MockParser.Create(vbe.Object);
+            using (var state = parser.State)
+            {
+                parser.Parse(new CancellationTokenSource());
+                var actualStatus = state.Status;
+                Assert.AreEqual(ParserState.Error, actualStatus);
+            }
+        }
+
+
+
         [Test]
         [TestCase(2, @"Do")]
         [TestCase(2, @"Loop")]