Implement IncludeVirtualMembers in MembersConfig (#27)

* Implement IncludeVirtualMembers

* Add tests for MemberProvider

---------

Co-authored-by: Empiree <Empiree@users.noreply.github.com>

src/Dumpify.Tests/Providers/MemberProviderTests.cs

@@ -0,0 +1,24 @@
+namespace Dumpify.Tests.Providers;
+
+public class MemberProviderTests
+{
+ [Theory]
+ [InlineData(true)]
+ [InlineData(false)]
+ public void IncludeVirtualMembers(bool includeVirtualMembers)
+ {
+ var membersConfig = new MembersConfig { IncludeVirtualMembers = includeVirtualMembers };
+ var testClass = new ClassWithVirtualProperty();
+
+ var output = testClass.DumpText(members: membersConfig);
+ var isContainsVirtualProperty = output.Contains(testClass.Bar);
+
+ includeVirtualMembers.Should().Be(isContainsVirtualProperty);
+ }
+
+ private class ClassWithVirtualProperty
+ {
+ public string Foo { get; set; } = "Oleg";
+ public virtual string Bar { get; set; } = "Hello";
+ }
+}

src/Dumpify/Config/MembersConfig.cs

@@ -4,6 +4,7 @@ public class MembersConfig
 {
  public bool IncludePublicMembers { get; set; } = true;
  public bool IncludeNonPublicMembers { get; set; } = false;
+ public bool IncludeVirtualMembers { get; set; } = true;
  public bool IncludeProperties { get; set; } = true;
  public bool IncludeFields { get; set; } = false;
 }

src/Dumpify/Descriptors/ValueProviders/MemberProvider.cs

@@ -8,21 +8,24 @@ internal sealed record MemberProvider : IMemberProvider
  private readonly bool _includeFields;
  private readonly bool _includePublicMembers;
  private readonly bool _includeNonPublicMembers;
+ private readonly bool _includeVirtualMembers;
 
  public MemberProvider()
- : this(true, false, true, false) { }
+ : this(true, false, true, false, true) { }
 
  public MemberProvider(
  bool includeProperties,
  bool includeFields,
  bool includePublicMembers,
- bool includeNonPublicMembers
+ bool includeNonPublicMembers,
+ bool includeVirtualMembers
  )
  {
  _includeProperties = includeProperties;
  _includeFields = includeFields;
  _includePublicMembers = includePublicMembers;
  _includeNonPublicMembers = includeNonPublicMembers;
+ _includeVirtualMembers = includeVirtualMembers;
  }
 
  public IEnumerable<IValueProvider> GetMembers(Type type)
@@ -38,10 +41,14 @@ public IEnumerable<IValueProvider> GetMembers(Type type)
  {
  var properties = type.GetProperties(flags)
  .Where(p => p.GetIndexParameters().Length == 0)
- .Where(p => p.GetMethod is not null)
- .Select(p => new PropertyValueProvider(p));
+ .Where(p => p.GetMethod is not null);
 
- members = members.Concat(properties);
+ if (!_includeVirtualMembers)
+ properties = properties.Where(p => !IsVirtualProperty(p));
+
+ var providers = properties.Select(p => new PropertyValueProvider(p));
+
+ members = members.Concat(providers);
  }
 
  if (_includeFields)
@@ -62,16 +69,21 @@ public bool Equals(IMemberProvider? provider)
  }
 
  return _includeProperties == other._includeProperties
- && _includeFields == other._includeFields
- && _includePublicMembers == other._includePublicMembers
- && _includeNonPublicMembers == other._includeNonPublicMembers;
+ && _includeFields == other._includeFields
+ && _includePublicMembers == other._includePublicMembers
+ && _includeNonPublicMembers == other._includeNonPublicMembers
+ && _includeVirtualMembers == other._includeVirtualMembers;
  }
 
  public override int GetHashCode() =>
  (
  _includeProperties,
  _includeFields,
  _includePublicMembers,
- _includeNonPublicMembers
+ _includeNonPublicMembers,
+ _includeVirtualMembers
  ).GetHashCode();
+
+ private bool IsVirtualProperty(PropertyInfo propertyInfo) =>
+ propertyInfo.GetAccessors().Any(accessor => accessor.IsVirtual);
 }

src/Dumpify/Extensions/DumpExtensions.cs

@@ -179,7 +179,8 @@ public static string DumpText<T>(
  membersConfig.IncludeProperties,
  membersConfig.IncludeFields,
  membersConfig.IncludePublicMembers,
- membersConfig.IncludeNonPublicMembers
+ membersConfig.IncludeNonPublicMembers,
+ membersConfig.IncludeVirtualMembers
  ),
  TypeNameProvider = new TypeNameProvider(
  typeNamingConfig.UseAliases,