CollectionResolver supports all default collection types now

That includes IReadOnlyCollection<> and IReadOnlyList<> in .NET 4.5 (or whatever Array types decide to implement in the future)

src/Castle.Windsor.Tests/Castle.Windsor.Tests.csproj

@@ -386,6 +386,7 @@
     <Compile Include="Components\ADisposable.cs" />
     <Compile Include="Components\AppHost.cs" />
     <Compile Include="Components\AppScreenCBA.cs" />
+    <Compile Include="Components\ArrayDepAsConstructor.cs" />
     <Compile Include="Components\BDisposable.cs" />
     <Compile Include="Components\BoundComponent.cs" />
     <Compile Include="Components\CacheWithClassConstraint.cs" />
@@ -491,6 +492,7 @@
     <Compile Include="Components\NeedsGenericType.cs" />
     <Compile Include="Components\NullCache.cs" />
     <Compile Include="Components\PlasmaGunArm.cs" />
+    <Compile Include="Components\ReadOnlyListDepAsConstructor.cs" />
     <Compile Include="Components\ResolvableDependency.cs" />
     <Compile Include="Components\ReviewerRepository.cs" />
     <Compile Include="Components\ServiceImplGeneric.cs" />
@@ -616,7 +618,7 @@
     <Compile Include="ComponentsWithAttribute\HasUserAttributeNonRegister.cs" />
     <Compile Include="ComponentsWithAttribute\HasUserAttributeRegister.cs" />
     <Compile Include="ComponentsWithAttribute\UserAttribute.cs" />
-    <Compile Include="Components\ArrayDepAsConstructor.cs" />
+    <Compile Include="Components\ReadOnlyCollectionDepAsConstructor.cs" />
     <Compile Include="Components\ArrayDepAsProperty.cs" />
     <Compile Include="Components\CollectionDepAsConstructor.cs" />
     <Compile Include="Components\CollectionDepAsProperty.cs" />

src/Castle.Windsor.Tests/Components/ArrayDepAsConstructor.cs

@@ -1,4 +1,4 @@
-// Copyright 2004-2011 Castle Project - http://www.castleproject.org/
+// Copyright 2004-2013 Castle Project - http://www.castleproject.org/
 // 
 // Licensed under the Apache License, Version 2.0 (the "License");
 // you may not use this file except in compliance with the License.

src/Castle.Windsor.Tests/Components/ReadOnlyCollectionDepAsConstructor.cs

@@ -0,0 +1,35 @@
+// Copyright 2004-2013 Castle Project - http://www.castleproject.org/
+// 
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+// 
+//     http://www.apache.org/licenses/LICENSE-2.0
+// 
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+namespace CastleTests.Components
+{
+#if DOTNET45
+	using System.Collections.Generic;
+
+	public class ReadOnlyCollectionDepAsConstructor
+	{
+		private readonly IReadOnlyCollection<IEmptyService> services;
+
+		public ReadOnlyCollectionDepAsConstructor(IReadOnlyCollection<IEmptyService> services)
+		{
+			this.services = services;
+		}
+
+		public IReadOnlyCollection<IEmptyService> Services
+		{
+			get { return services; }
+		}
+	}
+#endif
+}

src/Castle.Windsor.Tests/Components/ReadOnlyListDepAsConstructor.cs

@@ -0,0 +1,35 @@
+// Copyright 2004-2013 Castle Project - http://www.castleproject.org/
+// 
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+// 
+//     http://www.apache.org/licenses/LICENSE-2.0
+// 
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+
+namespace CastleTests.Components
+{
+	using System.Collections.Generic;
+
+#if DOTNET45
+	public class ReadOnlyListDepAsConstructor
+	{
+		private readonly IReadOnlyList<IEmptyService> services;
+
+		public ReadOnlyListDepAsConstructor(IReadOnlyList<IEmptyService> services)
+		{
+			this.services = services;
+		}
+
+		public IReadOnlyList<IEmptyService> Services
+		{
+			get { return services; }
+		}
+	}
+#endif
+}

src/Castle.Windsor.Tests/SpecializedResolvers/CollectionResolverTestCase.cs

@@ -17,6 +17,7 @@ namespace Castle.MicroKernel.Tests.SpecializedResolvers
 	using System;
 	using System.Linq;
 
+	using Castle.Core.Internal;
 	using Castle.MicroKernel.Handlers;
 	using Castle.MicroKernel.Registration;
 	using Castle.MicroKernel.Resolvers.SpecializedResolvers;
@@ -80,6 +81,43 @@ public void DependencyOnArrayOfServices_OnConstructor()
 			}
 		}
 
+#if DOTNET45
+		[Test]
+		public void DependencyOn_Readonly_collection_OfServices_OnConstructor()
+		{
+			Container.Register(Component.For<IEmptyService>().ImplementedBy<EmptyServiceA>(),
+							   Component.For<IEmptyService>().ImplementedBy<EmptyServiceB>(),
+							   Component.For<ReadOnlyCollectionDepAsConstructor>());
+
+			var component = Container.Resolve<ReadOnlyCollectionDepAsConstructor>();
+
+			Assert.IsNotNull(component.Services);
+			Assert.AreEqual(2, component.Services.Count);
+			foreach (var service in component.Services)
+			{
+				Assert.IsNotNull(service);
+			}
+		}
+
+
+		[Test]
+		public void DependencyOn_Readonly_list_OfServices_OnConstructor()
+		{
+			Container.Register(Component.For<IEmptyService>().ImplementedBy<EmptyServiceA>(),
+							   Component.For<IEmptyService>().ImplementedBy<EmptyServiceB>(),
+							   Component.For<ReadOnlyListDepAsConstructor>());
+
+			var component = Container.Resolve<ReadOnlyListDepAsConstructor>();
+
+			Assert.IsNotNull(component.Services);
+			Assert.AreEqual(2, component.Services.Count);
+			foreach (var service in component.Services)
+			{
+				Assert.IsNotNull(service);
+			}
+		}
+#endif
+
 		[Test]
 		public void DependencyOnArrayOfServices_OnConstructor_empty_allowed_empty_provided()
 		{

src/Castle.Windsor/Core/Internal/ReflectionUtil.cs

@@ -28,6 +28,11 @@ namespace Castle.Core.Internal
 
 	public static class ReflectionUtil
 	{
+		public static readonly Type[] OpenGenericArrayInterfaces = typeof(object[]).GetInterfaces()
+			.Where(i => i.IsGenericType)
+			.Select(i => i.GetGenericTypeDefinition())
+			.ToArray();
+
 #if !(DOTNET35 || SILVERLIGHT)
 		private static readonly ConcurrentDictionary<ConstructorInfo, Func<object[], object>> factories =
 			new ConcurrentDictionary<ConstructorInfo, Func<object[], object>>();
@@ -210,9 +215,7 @@ public static Type GetCompatibleArrayItemType(this Type type)
 				return null;
 			}
 			var openGeneric = type.GetGenericTypeDefinition();
-			if (openGeneric == typeof(IList<>) ||
-			    openGeneric == typeof(ICollection<>) ||
-			    openGeneric == typeof(IEnumerable<>))
+			if (OpenGenericArrayInterfaces.Contains(openGeneric))
 			{
 				return type.GetGenericArguments()[0];
 			}