Fixed array support

Stream support
ImmutableDictionary support
Performance improvements

Author: soenneker

src/AutoFakerBinder.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections;
 using System.Collections.Concurrent;
 using System.Collections.Generic;
@@ -128,28 +128,21 @@ internal static void PopulateMembers(object instance, AutoFakerContext context,
 
             if (value != null)
             {
-                try
+                if (!member.IsReadOnly)
                 {
-                    if (!member.IsReadOnly)
+                    member.Setter.Invoke(instance, value);
+                }
+                else
+                {
+                    if (member.IsDictionary)
                     {
-                        member.Setter.Invoke(instance, value);
+                        PopulateDictionary(value, instance, member);
                     }
-                    else
+                    else if (member.IsCollection)
                     {
-                        if (member.IsDictionary)
-                        {
-                            PopulateDictionary(value, instance, member);
-                        }
-                        else if (member.IsCollection)
-                        {
-                            PopulateCollection(value, instance, member);
-                        }
+                        PopulateCollection(value, instance, member);
                     }
                 }
-                catch (Exception e)
-                { // TODO: get rid of this, deal with failing unit
-                    Console.WriteLine(e.ToString());
-                }
             }
 
             // Remove the current type from the type stack so siblings can be created
@@ -187,7 +180,7 @@ private static bool ShouldSkip(AutoMember autoMember, AutoFakerContext context)
         // Inline logic to count matching types in the stack.
         if (autoMember.CachedType.CacheKey is { } cacheKeyValue)
         {
-            int typeCount = 0;
+            var typeCount = 0;
             foreach (int type in typesStack)
             {
                 if (type == cacheKeyValue)
@@ -352,9 +345,7 @@ private static IAutoFakerGenerator GetParameterGenerator(CachedType type, Cached
                 ref readonly CachedField cachedField = ref cachedFields[i];
 
                 // Skip constants, delegates, and backing fields
-                if (cachedField.FieldInfo.IsConstant() ||
-                    cachedField.IsDelegate ||
-                    cachedField.FieldInfo.Name.Contains("k__BackingField"))
+                if (cachedField.FieldInfo.IsConstant() || cachedField.IsDelegate || cachedField.FieldInfo.Name.Contains("k__BackingField"))
                     continue;
 
                 autoMembers.Add(new AutoMember(cachedField, cacheService, autoFakerConfig));
@@ -396,18 +387,43 @@ private static void PopulateCollection(object value, object parent, AutoMember m
 
         object? instance = member.Getter(parent);
 
-        if (instance is IList {IsReadOnly: true})
-            return;
+        switch (instance)
+        {
+            case null:
+                return;
+            case Array destArray:
+            {
+                int itemsToCopy = Math.Min(destArray.Length, collection.Count);
+                var i = 0;
+
+
+                foreach (object? item in collection)
+                {
+                    if (i >= itemsToCopy)
+                        break;
+
+                    // Handles boxing/unboxing and element-type conversion internally
+                    destArray.SetValue(item, i++);
+                }
+
+                return;
+            }
+            case IList {IsReadOnly: true}:
+                return;
+        }
 
         CachedType[] argTypes = member.CachedType.GetAddMethodArgumentTypes();
         CachedMethod? addMethod = GetAddMethod(member.CachedType, argTypes);
 
-        if (instance == null || addMethod == null)
+        if (addMethod == null)
             return;
 
+        var args = new object[1];
+
         foreach (object? item in collection)
         {
-            addMethod.Invoke(instance, [item]);
+            args[0] = item; // boxing avoidance
+            addMethod.Invoke(instance, args);
         }
     }
 

src/AutoFaker{T}.cs

@@ -116,14 +116,14 @@ private AutoFakerContext CreateContext(string? ruleSets)
     /// </summary>
     /// <param name="ruleSets"></param>
     /// <returns></returns>
-    private List<string> ParseRuleSets(string? ruleSets)
+    private HashSet<string> ParseRuleSets(string? ruleSets)
     {
         if (string.IsNullOrWhiteSpace(ruleSets))
         {
             ruleSets = null;
         }
 
-        var validRuleSets = new List<string>();
+        var validRuleSets = new HashSet<string>();
 
         string[] ruleSetArray = ruleSets?.Split(',') ?? [currentRuleSet];
 
@@ -162,7 +162,7 @@ private void PrepareCreate(AutoFakerContext context)
                 context.GenerateName = null;
 
                 // Get the members that should not be set during construction
-                List<string> memberNames = GetRuleSetsMemberNames(context);
+                HashSet<string> memberNames = GetRuleSetsMemberNames(context);
 
                 foreach (string? memberName in TypeProperties.Keys)
                 {
@@ -232,7 +232,7 @@ private void PrepareFinish(AutoFakerContext context)
             {
                 var finalAutoMembers = new List<AutoMember>();
 
-                List<string> memberNames = GetRuleSetsMemberNames(context);
+                HashSet<string> memberNames = GetRuleSetsMemberNames(context);
 
                 for (var i = 0; i < autoMembers.Count; i++)
                 {
@@ -254,17 +254,15 @@ private void PrepareFinish(AutoFakerContext context)
         _finishInitialized = true;
     }
 
-    private List<string> GetRuleSetsMemberNames(AutoFakerContext context)
+    private HashSet<string> GetRuleSetsMemberNames(AutoFakerContext context)
     {
-        var members = new List<string>();
+        var members = new HashSet<string>();
 
         if (context.RuleSets == null)
             return members;
 
-        for (var i = 0; i < context.RuleSets.Count; i++)
+        foreach (string ruleSetName in context.RuleSets)
         {
-            string ruleSetName = context.RuleSets[i];
-
             if (!Actions.TryGetValue(ruleSetName, out Dictionary<string, PopulateAction<TType>>? ruleSet))
             {
                 continue;

src/Context/AutoFakerContext.cs

@@ -49,7 +49,7 @@ public sealed class AutoFakerContext
     /// <summary>
     /// The requested rule sets provided for the generate request.
     /// </summary>
-    public List<string>? RuleSets;
+    public HashSet<string>? RuleSets;
 
     internal readonly Stack<int> TypesStack;
 

src/Context/AutoFakerOverrideContext.cs

@@ -37,7 +37,7 @@ public sealed class AutoFakerOverrideContext
     /// <summary>
     /// The requested rule sets provided for the generate request.
     /// </summary>
-    public List<string>? RuleSets { get; }
+    public HashSet<string>? RuleSets { get; }
 
     internal AutoFakerOverrideContext(AutoFakerContext generateContext)
     {

src/Extensions/TypeArrayExtension.cs

@@ -23,6 +23,8 @@ internal static (CachedType?, GenericCollectionType?) GetTypeOfGenericCollection
                 case "IImmutableList`1":
                 case "ImmutableList`1":
                     return (cachedType, GenericCollectionType.ImmutableList);
+                case "ImmutableDictionary`2":
+                    return (cachedType, GenericCollectionType.ImmutableDictionary);
                 case "SortedList`2":
                     return (cachedType, GenericCollectionType.SortedList);
                 case "Dictionary`2":

src/Generators/AutoFakerGeneratorFactory.cs

@@ -128,6 +128,12 @@ internal static IAutoFakerGenerator CreateGenerator(AutoFakerContext context)
                     return CreateGenericGenerator(CachedTypeService.ReadOnlyDictionaryGenerator.Value, keyType, valueType);
                 }
                 case GenericCollectionType.ImmutableDictionary:
+                {
+                    CachedType keyType = generics[0];
+                    CachedType valueType = generics[1];
+
+                    return CreateGenericGenerator(CachedTypeService.ImmutableDictionaryGenerator.Value, keyType, valueType);
+                }
                 case GenericCollectionType.Dictionary:
                 case GenericCollectionType.SortedList:
                 {

src/Generators/AutoFakerGeneratorMemberOverride.cs

@@ -8,6 +8,7 @@ namespace Soenneker.Utils.AutoBogus.Generators;
 internal sealed class AutoFakerGeneratorMemberOverride<TType, TValue> : AutoFakerGeneratorOverride
 {
     private Type Type { get; }
+
     private readonly CachedType _cachedType;
 
     private string MemberName { get; }

src/Generators/Types/BoolGenerator.cs

@@ -3,8 +3,7 @@
 
 namespace Soenneker.Utils.AutoBogus.Generators.Types;
 
-internal sealed class BoolGenerator
-    : IAutoFakerGenerator
+internal sealed class BoolGenerator : IAutoFakerGenerator
 {
     object IAutoFakerGenerator.Generate(AutoFakerContext context)
     {

src/Generators/Types/ByteGenerator.cs

@@ -3,8 +3,7 @@
 
 namespace Soenneker.Utils.AutoBogus.Generators.Types;
 
-internal sealed class ByteGenerator
-    : IAutoFakerGenerator
+internal sealed class ByteGenerator : IAutoFakerGenerator
 {
     object IAutoFakerGenerator.Generate(AutoFakerContext context)
     {

src/Generators/Types/DataSets/Base/BaseDataSetGenerator.cs

@@ -9,7 +9,7 @@ internal abstract class BaseDataSetGenerator : IAutoFakerGenerator
 {
     public static bool TryCreateGenerator(AutoFakerContext? context, CachedType dataSetType, out BaseDataSetGenerator? generator)
     {
-        generator = default;
+        generator = null;
 
         CachedType cachedDataSetType = context.CacheService.Cache.GetCachedType(typeof(DataSet));