Merge pull request #2246 from ThunderFrame/next

Make test for Class event procedures a case-insensitive test
rubberduck-vba · Sep 20, 2016 · 39273ba · 39273ba
2 parents 81a62a0 + cfebc7a
commit 39273ba
Show file tree

Hide file tree

Showing 2 changed files with 86 additions and 1 deletion.
diff --git a/RetailCoder.VBE/Common/DeclarationExtensions.cs b/RetailCoder.VBE/Common/DeclarationExtensions.cs
@@ -295,7 +295,7 @@ public static IEnumerable<Declaration> FindBuiltInEventHandlers(this IEnumerable
             var classModuleHandlers = declarationList.Where(item =>
                         item.DeclarationType == DeclarationType.Procedure &&
                         item.ParentDeclaration.DeclarationType == DeclarationType.ClassModule &&
-                        (item.IdentifierName == "Class_Initialize" || item.IdentifierName == "Class_Terminate"));
+                        (item.IdentifierName.Equals("Class_Initialize", StringComparison.InvariantCultureIgnoreCase) || item.IdentifierName.Equals("Class_Terminate", StringComparison.InvariantCultureIgnoreCase)));
 
             var handlers = declarationList.Where(declaration => !declaration.IsBuiltIn
                                                      && declaration.DeclarationType == DeclarationType.Procedure

diff --git a/RubberduckTests/Inspections/ProcedureNotUsedInspectionTests.cs b/RubberduckTests/Inspections/ProcedureNotUsedInspectionTests.cs
@@ -224,6 +224,91 @@ public void ProcedureNotUsed_NoResultForClassInitialize()
             Assert.AreEqual(0, inspectionResults.Count());
         }
 
+        [TestMethod]
+        [TestCategory("Inspections")]
+        public void ProcedureNotUsed_NoResultForCasedClassInitialize()
+        {
+            //Input
+            const string inputCode =
+@"Private Sub class_initialize()
+End Sub";
+
+            //Arrange
+            var builder = new MockVbeBuilder();
+            var vbe = builder.ProjectBuilder("TestProject", vbext_ProjectProtection.vbext_pp_none)
+                             .AddComponent("TestClass", vbext_ComponentType.vbext_ct_ClassModule, inputCode)
+                             .MockVbeBuilder().Build();
+
+            var mockHost = new Mock<IHostApplication>();
+            mockHost.SetupAllProperties();
+            var parser = MockParser.Create(vbe.Object, new RubberduckParserState(new Mock<ISinks>().Object));
+
+            parser.Parse(new CancellationTokenSource());
+            if (parser.State.Status >= ParserState.Error) { Assert.Inconclusive("Parser Error"); }
+
+            var inspection = new ProcedureNotUsedInspection(parser.State);
+            var inspectionResults = inspection.GetInspectionResults();
+
+            Assert.AreEqual(0, inspectionResults.Count());
+        }
+
+        [TestMethod]
+        [TestCategory("Inspections")]
+        public void ProcedureNotUsed_NoResultForClassTerminate()
+        {
+            //Input
+            const string inputCode =
+@"Private Sub Class_Terminate()
+End Sub";
+
+            //Arrange
+            var builder = new MockVbeBuilder();
+            var vbe = builder.ProjectBuilder("TestProject", vbext_ProjectProtection.vbext_pp_none)
+                             .AddComponent("TestClass", vbext_ComponentType.vbext_ct_ClassModule, inputCode)
+                             .MockVbeBuilder().Build();
+
+            var mockHost = new Mock<IHostApplication>();
+            mockHost.SetupAllProperties();
+            var parser = MockParser.Create(vbe.Object, new RubberduckParserState(new Mock<ISinks>().Object));
+
+            parser.Parse(new CancellationTokenSource());
+            if (parser.State.Status >= ParserState.Error) { Assert.Inconclusive("Parser Error"); }
+
+            var inspection = new ProcedureNotUsedInspection(parser.State);
+            var inspectionResults = inspection.GetInspectionResults();
+
+            Assert.AreEqual(0, inspectionResults.Count());
+        }
+
+        [TestMethod]
+        [TestCategory("Inspections")]
+        public void ProcedureNotUsed_NoResultForCasedClassTerminate()
+        {
+            //Input
+            const string inputCode =
+@"Private Sub class_terminate()
+End Sub";
+
+            //Arrange
+            var builder = new MockVbeBuilder();
+            var vbe = builder.ProjectBuilder("TestProject", vbext_ProjectProtection.vbext_pp_none)
+                             .AddComponent("TestClass", vbext_ComponentType.vbext_ct_ClassModule, inputCode)
+                             .MockVbeBuilder().Build();
+
+            var mockHost = new Mock<IHostApplication>();
+            mockHost.SetupAllProperties();
+            var parser = MockParser.Create(vbe.Object, new RubberduckParserState(new Mock<ISinks>().Object));
+
+            parser.Parse(new CancellationTokenSource());
+            if (parser.State.Status >= ParserState.Error) { Assert.Inconclusive("Parser Error"); }
+
+            var inspection = new ProcedureNotUsedInspection(parser.State);
+            var inspectionResults = inspection.GetInspectionResults();
+
+            Assert.AreEqual(0, inspectionResults.Count());
+        }
+
+
         [TestMethod]
         [TestCategory("Inspections")]
         public void ProcedureNotUsed_Ignored_DoesNotReturnResult()