#94 Feature: Also skip members defined via interface

src/AutoBogus.Tests.Models/Complex/IWithCode.cs

@@ -0,0 +1,9 @@
+using System;
+
+namespace AutoBogus.Tests.Models.Complex
+{
+  public interface IWithCode
+  {
+    Guid? Code { get; set; }
+  }
+}

src/AutoBogus.Tests.Models/Complex/Order.cs

@@ -1,9 +1,9 @@
-using System;
+using System;
 using System.Collections.Generic;
 
 namespace AutoBogus.Tests.Models.Complex
 {
-  public sealed class Order
+  public sealed class Order : IWithCode
   {
     public DateTime Timestamp;
 

src/AutoBogus.Tests/AutoFakerFixture.cs

@@ -395,6 +395,18 @@ public void Should_Skip_Configured_Members()
         instance.Discounts.Should().BeNull();
         instance.Items.Should().OnlyContain(i => i.Discounts == null);
       }
+
+      [Fact]
+      public void Should_Skip_Configured_Members_Of_Interface()
+      {
+        var instance = AutoFaker.Generate<Order>(builder =>
+        {
+          builder
+            .WithSkip<IWithCode>(i => i.Code);
+        });
+
+        instance.Code.Should().BeNull();
+      }
     }
 
     public class Behaviors_Types

src/AutoBogus/AutoBinder.cs

@@ -72,7 +72,7 @@ public virtual void PopulateInstance<TType>(object instance, AutoGenerateContext
         {
           // Check if the member has a skip config or the type has already been generated as a parent
           // If so skip this generation otherwise track it for use later in the object tree
-          if (ShouldSkip(member.Type, $"{type.FullName}.{member.Name}", context))
+          if (ShouldSkip(type, member, context))
           {
             continue;
           }
@@ -111,19 +111,26 @@ public virtual void PopulateInstance<TType>(object instance, AutoGenerateContext
       }
     }
 
-    private bool ShouldSkip(Type type, string path, AutoGenerateContext context)
+    private bool ShouldSkip(Type type, AutoMember member, AutoGenerateContext context)
     {
-      // Skip if the type is found
-      if (context.Config.SkipTypes.Contains(type))
+      // Skip if the member type is found
+      if (context.Config.SkipTypes.Contains(member.Type))
       {
         return true;
       }
 
-      // Skip if the path is found
-      if (context.Config.SkipPaths.Contains(path))
+      // Skip if the path is found (both current type and its implemented interfaces)
+      if (context.Config.SkipPaths.Contains($"{type.FullName}.{member.Name}"))
       {
         return true;
       }
+      foreach (var implementedInterfaceType in type.GetInterfaces())
+      {
+        if (context.Config.SkipPaths.Contains($"{implementedInterfaceType.FullName}.{member.Name}"))
+        {
+          return true;
+        }
+      }
 
       //check if tree depth is reached
       var treeDepth = context.Config.TreeDepth.Invoke(context);
@@ -132,8 +139,8 @@ private bool ShouldSkip(Type type, string path, AutoGenerateContext context)
         return true;
 
       // Finally check if the recursive depth has been reached
-      
-      var count = context.TypesStack.Count(t => t == type);
+
+      var count = context.TypesStack.Count(t => t == member.Type);
       var recursiveDepth = context.Config.RecursiveDepth.Invoke(context);
 
       return count >= recursiveDepth;