Correct null checks in VBComponent(s).

Author: comintern

Rubberduck.VBEEditor/SafeComWrappers/VBA/VBComponent.cs

@@ -131,10 +131,7 @@ public string ExportAsSourceFile(string folder)
 
         public IVBProject ParentProject
         {
-            get
-            {
-                return Collection != null ? Collection.Parent : null;
-            }
+            get { return Collection.Parent; }
         }
 
         private void ExportUserFormModule(string path)

Rubberduck.VBEEditor/SafeComWrappers/VBA/VBComponents.cs

@@ -42,7 +42,10 @@ public IVBComponent this[object index]
 
         public void Remove(IVBComponent item)
         {
-            if (!IsWrappingNullReference) Target.Remove((VB.VBComponent)item.Target);
+            if (item != null && item.Target != null && !IsWrappingNullReference)
+            {
+                Target.Remove((VB.VBComponent)item.Target);
+            }
         }
 
         public IVBComponent Add(ComponentType type)

RubberduckTests/Inspections/ProcedureNotUsedInspectionTests.cs

@@ -196,7 +196,7 @@ Private Sub abc_Foo(ByVal arg1 As Integer, ByVal arg2 As String)
             var inspection = new ProcedureNotUsedInspection(parser.State);
             var inspectionResults = inspection.GetInspectionResults();
 
-            Assert.AreEqual(1, inspectionResults.Count(result => result.Target.DeclarationType == DeclarationType.Procedure));
+            Assert.AreEqual(0, inspectionResults.Count(result => result.Target.DeclarationType == DeclarationType.Procedure));
         }
 
         [TestMethod]