Fix mock for components count

Extend mocks for properties

Author: Hosch250

Rubberduck.CodeAnalysis/Inspections/Concrete/SheetAccessedUsingStringInspection.cs

@@ -101,20 +101,27 @@ private IVBComponent GetVBComponentMatchingSheetName(IdentifierReferenceInspecti
             var sheetName = FormatSheetName(sheetArgumentContext.GetText());
             var project = State.Projects.First(p => p.ProjectId == reference.QualifiedName.ProjectId);
 
+
+            //return project.VBComponents.FirstOrDefault(c =>
+            //    c.Type == ComponentType.Document &&
+            //    (string)c.Properties.First(property => property.Name == "Name").Value == sheetName);
             using (var components = project.VBComponents)
             {
                 for (var i = 0; i < components.Count; i++)
                 {
                     using (var component = components[i])
                     using (var properties = component.Properties)
                     {
-                        for (var j = 0; j < properties.Count; j++)
+                        if (component.Type == ComponentType.Document)
                         {
-                            using (var property = properties[j])
+                            for (var j = 0; j < properties.Count; j++)
                             {
-                                if (component.Type == ComponentType.Document && property.Name == "Name" && (string)property.Value == sheetName)
+                                using (var property = properties[j])
                                 {
-                                    return component;
+                                    if (property.Name == "Name" && (string)property.Value == sheetName)
+                                    {
+                                        return component;
+                                    }
                                 }
                             }
                         }

RubberduckTests/Mocks/MockProjectBuilder.cs

@@ -189,7 +189,7 @@ private Mock<IVBComponents> CreateComponentsMock()
 
             result.Setup(m => m[It.IsAny<int>()]).Returns<int>(index => Components.ElementAt(index));
             result.Setup(m => m[It.IsAny<string>()]).Returns<string>(name => Components.Single(item => item.Name == name));
-            result.SetupGet(m => m.Count).Returns(Components.Count);
+            result.SetupGet(m => m.Count).Returns(() => Components.Count);
 
             result.Setup(m => m.Add(It.IsAny<ComponentType>()))
                 .Callback((ComponentType c) =>
@@ -284,6 +284,8 @@ private Mock<IVBComponent> CreateComponentMock(string name, ComponentType type,
 
             var propertiesMock = new Mock<IProperties>();
             propertiesMock.Setup(m => m.GetEnumerator()).Returns(() => properties?.GetEnumerator());
+            propertiesMock.SetupGet(m => m.Count).Returns(properties?.Count() ?? 0);
+            propertiesMock.Setup(m => m[It.IsAny<int>()]).Returns<int>(index => properties.ElementAt(index));
             result.SetupGet(m => m.Properties).Returns(propertiesMock.Object);
 
             var module = CreateCodeModuleMock(name, content, selection, result);