Merge pull request #641 from stakx/callbase

Don't ignore non-public interfaces unconditionally

CHANGELOG.md

@@ -15,6 +15,9 @@ The format is loosely based on [Keep a Changelog](http://keepachangelog.com/en/1
 * Update package reference to `Castle.Core` (DynamicProxy) from version 4.3.0 to 4.3.1 (@stakx, #635)
 * Floating point values are formatted with higher precision (satisfying round-tripping) in diagnostic messages (@stakx, #637)
 
+#### Fixed
+
+* `CallBase` disregarded for some base methods from non-public interfaces (@stakx, #641)
 
 #### Obsoleted
 

Source/Mock.Generic.cs

@@ -69,7 +69,7 @@ static Mock()
 			inheritedInterfaces =
 				typeof(T)
 				.GetInterfaces()
-				.Where(i => { var it = i.GetTypeInfo(); return it.IsPublic || it.IsNestedPublic && !it.IsImport; })
+				.Where(i => ProxyFactory.Instance.IsTypeVisible(i) && !i.GetTypeInfo().IsImport)
 				.ToArray();
 
 			serialNumberCounter = 0;

Source/ProxyFactories/CastleProxyFactory.cs

@@ -122,6 +122,11 @@ public override bool IsMethodVisible(MethodInfo method, out string messageIfNotV
 			return ProxyUtil.IsAccessible(method, out messageIfNotVisible);
 		}
 
+		public override bool IsTypeVisible(Type type)
+		{
+			return ProxyUtil.IsAccessible(type);
+		}
+
 		/// <inheritdoc />
 		public override Type GetDelegateProxyInterface(Type delegateType, out MethodInfo delegateInterfaceMethod)
 		{

Source/ProxyFactories/ProxyFactory.cs

@@ -54,6 +54,8 @@ internal abstract class ProxyFactory
 
 		public abstract bool IsMethodVisible(MethodInfo method, out string messageIfNotVisible);
 
+		public abstract bool IsTypeVisible(Type type);
+
 		/// <summary>
 		/// Gets an autogenerated interface with a method on it that matches the signature of the specified
 		/// <paramref name="delegateType"/>.