Merge pull request #27 from atc-net/fix/memberdata-removes-duplicate-values

MemberAutoNSubstituteDataAttribute removes parameters with same values

Author: rickykaare

src/Atc.Test/MemberAutoNSubstituteDataAttribute.cs

@@ -33,7 +33,7 @@ protected override object[] ConvertDataItem(MethodInfo testMethod, object item)
         var fixture = FixtureFactory.Create();
 
         return values
-            .Union(testMethod
+            .Concat(testMethod
                 .GetParameters()
                 .Skip(values.Length)
                 .Select(p => GetSpecimen(fixture, p)))

test/Atc.Test.Tests/MemberAutoNSubstituteDataAttributeTests.cs

@@ -2,16 +2,16 @@ namespace Atc.Test.Tests;
 
 public class MemberAutoNSubstituteDataAttributeTests
 {
-    public static readonly IEnumerable<object[]> TestData = new[]
+    public static readonly IEnumerable<object[]> MemberData = new[]
     {
         new object[] { SampleEnum.One },
         new object[] { SampleEnum.Two },
         new object[] { SampleEnum.Three },
     };
 
     [Theory]
-    [MemberAutoNSubstituteData(nameof(TestData))]
-    public void MemberAutoNSubstituteData_Should_Call_For_Each_Value(
+    [MemberAutoNSubstituteData(nameof(MemberData))]
+    public void MemberAutoNSubstituteData_Should_Call_For_MemberData(
         SampleEnum value,
         [Frozen] ISampleInterface interfaceType,
         SampleClass concreteType,
@@ -25,4 +25,29 @@ public void MemberAutoNSubstituteData_Should_Call_For_Each_Value(
         dependantType.Should().NotBeNull();
         dependantType.Dependency.Should().Be(interfaceType);
     }
+
+    public static readonly IEnumerable<object[]> MemberDataMultipleValues = new[]
+    {
+        new object[] { SampleEnum.One, SampleEnum.Two },
+        new object[] { SampleEnum.One, SampleEnum.One },
+    };
+
+    [Theory]
+    [MemberAutoNSubstituteData(nameof(MemberDataMultipleValues))]
+    public void MemberAutoNSubstituteData_Should_Call_For_Each_MemderData_Values(
+        SampleEnum firstValue,
+        SampleEnum secondValue,
+        [Frozen] ISampleInterface interfaceType,
+        SampleClass concreteType,
+        SampleDependantClass dependantType)
+    {
+        firstValue.Should().BeOneOf(SampleEnum.One, SampleEnum.Two, SampleEnum.Three);
+        secondValue.Should().BeOneOf(SampleEnum.One, SampleEnum.Two, SampleEnum.Three);
+        interfaceType.Should().NotBeNull();
+        interfaceType.IsSubstitute().Should().BeTrue();
+        concreteType.Should().NotBeNull();
+        concreteType.IsSubstitute().Should().BeFalse();
+        dependantType.Should().NotBeNull();
+        dependantType.Dependency.Should().Be(interfaceType);
+    }
 }