Renamed auto-data attributes (#3)

* Renamed auto arguments attribute

* Renamed base auto data source attribute

* Renamed class auto data attribute

* Renamed member auto data attribute

* Renamed base auto data-source attribute

* Renamed auto data attribute

* Refactored DataGeneratorMetadataHelper

* Renamed dependency constraints tests to not reference xUnit 3

* Removed unused enumerable extensions

* Removed redundant logo images

* Removed unused value task extensions

* Removed unused value task extensions

* Refactored for clarity

* Adjusted names of derived data source attributes

* Renamed composite data source attribute

* Renamed test types

Author: aivascu

…ture.TUnit/ArgumentsAutoDataAttribute.cs …oFixture.TUnit/AutoArgumentsAttribute.cssrc/AutoFixture.TUnit/ArgumentsAutoDataAttribute.cs renamed to src/AutoFixture.TUnit/AutoArgumentsAttribute.cs src/AutoFixture.TUnit/ArgumentsAutoDataAttribute.cs renamed to src/AutoFixture.TUnit/AutoArgumentsAttribute.cs

@@ -9,24 +9,24 @@ namespace AutoFixture.TUnit;
 /// </summary>
 [SuppressMessage("Microsoft.Performance", "CA1813:AvoidUnsealedAttributes",
     Justification = "This attribute is the root of a potential attribute hierarchy.")]
-public class ArgumentsAutoDataAttribute : AutoFixtureDataSourceAttribute
+public class AutoArgumentsAttribute : BaseDataSourceAttribute
 {
     /// <summary>
-    /// Initializes a new instance of the <see cref="ArgumentsAutoDataAttribute" /> class.
+    /// Initializes a new instance of the <see cref="AutoArgumentsAttribute" /> class.
     /// </summary>
     /// <param name="values">The data values to pass to the theory.</param>
-    public ArgumentsAutoDataAttribute(params object?[] values)
+    public AutoArgumentsAttribute(params object?[] values)
         : this(() => new Fixture(), values)
     {
     }
 
     /// <summary>
-    /// Initializes a new instance of the <see cref="ArgumentsAutoDataAttribute" /> class.
+    /// Initializes a new instance of the <see cref="AutoArgumentsAttribute" /> class.
     /// </summary>
     /// <param name="fixtureFactory">The fixture factory.</param>
     /// <param name="values">The data values to pass to the theory.</param>
     /// <exception cref="ArgumentNullException"></exception>
-    protected ArgumentsAutoDataAttribute(Func<IFixture> fixtureFactory, params object?[]? values)
+    protected AutoArgumentsAttribute(Func<IFixture> fixtureFactory, params object?[]? values)
     {
         this.FixtureFactory = fixtureFactory ?? throw new ArgumentNullException(nameof(fixtureFactory));
         this.Values = values ?? [null];

…oFixture.TUnit/ClassAutoDataAttribute.cs …re.TUnit/AutoClassDataSourceAttribute.cssrc/AutoFixture.TUnit/ClassAutoDataAttribute.cs renamed to src/AutoFixture.TUnit/AutoClassDataSourceAttribute.cs src/AutoFixture.TUnit/ClassAutoDataAttribute.cs renamed to src/AutoFixture.TUnit/AutoClassDataSourceAttribute.cs

@@ -10,20 +10,20 @@ namespace AutoFixture.TUnit;
 /// </summary>
 [SuppressMessage("Microsoft.Performance", "CA1813:AvoidUnsealedAttributes",
     Justification = "This attribute is the root of a potential attribute hierarchy.")]
-public class ClassAutoDataAttribute : AutoFixtureDataSourceAttribute
+public class AutoClassDataSourceAttribute : BaseDataSourceAttribute
 {
     /// <summary>
-    /// Initializes a new instance of the <see cref="ClassAutoDataAttribute"/> class.
+    /// Initializes a new instance of the <see cref="AutoClassDataSourceAttribute"/> class.
     /// </summary>
     /// <param name="sourceType">The type of the class that provides the data.</param>
     /// <param name="parameters">The parameters passed to the data provider class constructor.</param>
-    public ClassAutoDataAttribute(Type sourceType, params object?[] parameters)
+    public AutoClassDataSourceAttribute(Type sourceType, params object?[] parameters)
         : this(() => new Fixture(), sourceType, parameters)
     {
     }
 
     /// <summary>
-    /// Initializes a new instance of the <see cref="ClassAutoDataAttribute"/> class.
+    /// Initializes a new instance of the <see cref="AutoClassDataSourceAttribute"/> class.
     /// </summary>
     /// <param name="fixtureFactory">The fixture factory that provides missing data from <paramref name="sourceType"/>.</param>
     /// <param name="sourceType">The type of the class that provides the data.</param>
@@ -38,7 +38,7 @@ public ClassAutoDataAttribute(Type sourceType, params object?[] parameters)
     /// The missing arguments for the test are being supplied from the Fixture instance.
     /// <code>
     /// [Test]
-    /// [ClassAutoData(typeof(MyTestData))]
+    /// [AutoClassDataSource(typeof(MyTestData))]
     /// public void ClassDataSuppliesExtraValues(int sum, int[] numbers, Person client)
     /// {
     ///     var actual = numbers.Sum(x => x);
@@ -60,7 +60,7 @@ public ClassAutoDataAttribute(Type sourceType, params object?[] parameters)
     /// }
     /// </code>
     /// </example>
-    protected ClassAutoDataAttribute(Func<IFixture> fixtureFactory, Type sourceType, params object?[] parameters)
+    protected AutoClassDataSourceAttribute(Func<IFixture> fixtureFactory, Type sourceType, params object?[] parameters)
     {
         this.FixtureFactory = fixtureFactory ?? throw new ArgumentNullException(nameof(fixtureFactory));
         this.SourceType = sourceType ?? throw new ArgumentNullException(nameof(sourceType));

…c/AutoFixture.TUnit/AutoDataAttribute.cs …Fixture.TUnit/AutoDataSourceAttribute.cssrc/AutoFixture.TUnit/AutoDataAttribute.cs renamed to src/AutoFixture.TUnit/AutoDataSourceAttribute.cs src/AutoFixture.TUnit/AutoDataAttribute.cs renamed to src/AutoFixture.TUnit/AutoDataSourceAttribute.cs

@@ -9,29 +9,29 @@ namespace AutoFixture.TUnit;
 /// </summary>
 [SuppressMessage("Microsoft.Performance", "CA1813:AvoidUnsealedAttributes",
     Justification = "This attribute is the root of a potential attribute hierarchy.")]
-public class AutoDataAttribute : AutoFixtureDataSourceAttribute
+public class AutoDataSourceAttribute : BaseDataSourceAttribute
 {
     /// <summary>
-    /// Initializes a new instance of the <see cref="AutoDataAttribute" /> class.
+    /// Initializes a new instance of the <see cref="AutoDataSourceAttribute" /> class.
     /// </summary>
     /// <remarks>
     /// <para>
     /// This constructor overload initializes the <see cref="Fixture" /> to an instance of
     /// <see cref="Fixture" />.
     /// </para>
     /// </remarks>
-    public AutoDataAttribute()
+    public AutoDataSourceAttribute()
         : this(() => new Fixture())
     {
     }
 
     /// <summary>
-    /// Initializes a new instance of the <see cref="AutoDataAttribute" /> class
+    /// Initializes a new instance of the <see cref="AutoDataSourceAttribute" /> class
     /// with the supplied <paramref name="fixtureFactory" />. Fixture will be created
     /// on demand using the provided factory.
     /// </summary>
     /// <param name="fixtureFactory">The fixture factory used to construct the fixture.</param>
-    protected AutoDataAttribute(Func<IFixture> fixtureFactory)
+    protected AutoDataSourceAttribute(Func<IFixture> fixtureFactory)
     {
         this.FixtureFactory = fixtureFactory ?? throw new ArgumentNullException(nameof(fixtureFactory));
     }

src/AutoFixture.TUnit/AutoFixtureLogo200x200.png

@@ -13,49 +13,49 @@ namespace AutoFixture.TUnit;
 /// </summary>
 [SuppressMessage("Microsoft.Performance", "CA1813:AvoidUnsealedAttributes",
     Justification = "This attribute is the root of a potential attribute hierarchy.")]
-public class MemberAutoDataAttribute : AutoFixtureDataSourceAttribute
+public class AutoMemberDataSourceAttribute : BaseDataSourceAttribute
 {
     /// <summary>
-    /// Initializes a new instance of the <see cref="MemberAutoDataAttribute" /> class.
+    /// Initializes a new instance of the <see cref="AutoMemberDataSourceAttribute" /> class.
     /// </summary>
     /// <param name="memberName">The name of the public static member on the test class that will provide the test data.</param>
     /// <param name="parameters">The parameters for the member (only supported for methods; ignored for everything else).</param>
-    public MemberAutoDataAttribute(string memberName, params object?[] parameters)
+    public AutoMemberDataSourceAttribute(string memberName, params object?[] parameters)
         : this(() => new Fixture(), memberType: null, memberName, parameters)
     {
     }
 
     /// <summary>
-    /// Initializes a new instance of the <see cref="MemberAutoDataAttribute" /> class.
+    /// Initializes a new instance of the <see cref="AutoMemberDataSourceAttribute" /> class.
     /// </summary>
     /// <param name="memberType">The type declaring the source member.</param>
     /// <param name="memberName">The name of the public static member on the test class that will provide the test data.</param>
     /// <param name="parameters">The parameters for the member (only supported for methods; ignored for everything else).</param>
-    public MemberAutoDataAttribute(Type? memberType, string memberName, params object?[] parameters)
+    public AutoMemberDataSourceAttribute(Type? memberType, string memberName, params object?[] parameters)
         : this(() => new Fixture(), memberType, memberName, parameters)
     {
     }
 
     /// <summary>
-    /// Initializes a new instance of the <see cref="MemberAutoDataAttribute" /> class.
+    /// Initializes a new instance of the <see cref="AutoMemberDataSourceAttribute" /> class.
     /// </summary>
     /// <param name="fixtureFactory">The fixture factory delegate.</param>
     /// <param name="memberName">The name of the public static member on the test class that will provide the test data.</param>
     /// <param name="parameters">The parameters for the member (only supported for methods; ignored for everything else).</param>
-    protected MemberAutoDataAttribute(Func<IFixture> fixtureFactory, string memberName, params object?[] parameters)
+    protected AutoMemberDataSourceAttribute(Func<IFixture> fixtureFactory, string memberName, params object?[] parameters)
         : this(fixtureFactory, memberType: null, memberName, parameters)
     {
     }
 
     /// <summary>
-    /// Initializes a new instance of the <see cref="MemberAutoDataAttribute" /> class.
+    /// Initializes a new instance of the <see cref="AutoMemberDataSourceAttribute" /> class.
     /// </summary>
     /// <param name="fixtureFactory">The fixture factory delegate.</param>
     /// <param name="memberType">The type declaring the source member.</param>
     /// <param name="memberName">The name of the public static member on the test class that will provide the test data.</param>
     /// <param name="parameters">The parameters for the member (only supported for methods; ignored for everything else).</param>
     /// <exception cref="ArgumentNullException">Thrown when arguments are null.</exception>
-    protected MemberAutoDataAttribute(Func<IFixture> fixtureFactory, Type? memberType, string memberName, params object?[]? parameters)
+    protected AutoMemberDataSourceAttribute(Func<IFixture> fixtureFactory, Type? memberType, string memberName, params object?[]? parameters)
     {
         this.FixtureFactory = fixtureFactory ?? throw new ArgumentNullException(nameof(fixtureFactory));
         this.MemberName = memberName ?? throw new ArgumentNullException(nameof(memberName));

…Fixture.TUnit/MemberAutoDataAttribute.cs …e.TUnit/AutoMemberDataSourceAttribute.cssrc/AutoFixture.TUnit/MemberAutoDataAttribute.cs renamed to src/AutoFixture.TUnit/AutoMemberDataSourceAttribute.cs src/AutoFixture.TUnit/MemberAutoDataAttribute.cs renamed to src/AutoFixture.TUnit/AutoMemberDataSourceAttribute.cs

@@ -5,22 +5,24 @@ namespace AutoFixture.TUnit;
 /// <summary>
 /// Base class for data sources that provide AutoFixture test data for TUnit data driven tests.
 /// </summary>
-public abstract class AutoFixtureDataSourceAttribute : NonTypedDataSourceGeneratorAttribute, IDataSource
+public abstract class BaseDataSourceAttribute : NonTypedDataSourceGeneratorAttribute, IDataSource
 {
     /// <summary>
     /// Returns the test data provided by the source.
     /// </summary>
-    /// <param name="dataGeneratorMetadata"></param>
-    /// <returns></returns>
+    /// <param name="dataGeneratorMetadata">
+    /// The metadata for the target method for which to provide the arguments.
+    /// </param>
+    /// <returns>
+    /// Returns a sequence of argument collections, where each collection
+    /// is an array of objects representing the arguments for a test method.
+    /// </returns>
     public abstract IEnumerable<object?[]?> GetData(DataGeneratorMetadata dataGeneratorMetadata);
 
     /// <inheritdoc />
     public override IEnumerable<Func<object?[]>> GenerateDataSources(DataGeneratorMetadata dataGeneratorMetadata)
     {
-        if (dataGeneratorMetadata is null)
-        {
-            throw new ArgumentNullException(nameof(dataGeneratorMetadata));
-        }
+        if (dataGeneratorMetadata is null) throw new ArgumentNullException(nameof(dataGeneratorMetadata));
 
         return GetTestDataEnumerable();
 
@@ -35,22 +37,15 @@ public abstract class AutoFixtureDataSourceAttribute : NonTypedDataSourceGenerat
             }
 
             var enumerable = this.GetData(dataGeneratorMetadata)
-                             ?? throw new InvalidOperationException("The source member yielded no test data.");
+                ?? throw new InvalidOperationException("The source member yielded no test data.");
 
             foreach (var testData in enumerable)
             {
-                if (testData is null)
-                {
-                    throw new InvalidOperationException("The source member yielded a null test data.");
-                }
-
-                if (testData.Length > parameters.Length)
-                {
-                    throw new InvalidOperationException("The number of arguments provided exceeds the number of parameters.");
-                }
+                if (testData is null) throw new InvalidOperationException("The source member yielded a null test data.");
+                if (testData.Length > parameters.Length) throw new InvalidOperationException("The number of arguments provided exceeds the number of parameters.");
 
                 yield return () => testData;
             }
         }
     }
-}
+}

….TUnit/AutoFixtureDataSourceAttribute.cs …Fixture.TUnit/BaseDataSourceAttribute.cssrc/AutoFixture.TUnit/AutoFixtureDataSourceAttribute.cs renamed to src/AutoFixture.TUnit/BaseDataSourceAttribute.cs src/AutoFixture.TUnit/AutoFixtureDataSourceAttribute.cs renamed to src/AutoFixture.TUnit/BaseDataSourceAttribute.cs

@@ -8,32 +8,32 @@ namespace AutoFixture.TUnit;
 /// </summary>
 [SuppressMessage("Microsoft.Performance", "CA1813:AvoidUnsealedAttributes",
     Justification = "This attribute is the root of a potential attribute hierarchy.")]
-public class CompositeDataAttribute : AutoFixtureDataSourceAttribute
+public class CompositeDataSourceAttribute : BaseDataSourceAttribute
 {
-    private readonly AutoFixtureDataSourceAttribute[] attributes;
+    private readonly BaseDataSourceAttribute[] attributes;
 
     /// <summary>
-    /// Initializes a new instance of the <see cref="CompositeDataAttribute"/> class.
+    /// Initializes a new instance of the <see cref="CompositeDataSourceAttribute"/> class.
     /// </summary>
     /// <param name="attributes">The attributes representing a data source for a data theory.</param>
-    public CompositeDataAttribute(IEnumerable<AutoFixtureDataSourceAttribute> attributes)
-        : this(attributes as AutoFixtureDataSourceAttribute[] ?? attributes.ToArray())
+    public CompositeDataSourceAttribute(IEnumerable<BaseDataSourceAttribute> attributes)
+        : this(attributes as BaseDataSourceAttribute[] ?? attributes.ToArray())
     {
     }
 
     /// <summary>
-    /// Initializes a new instance of the <see cref="CompositeDataAttribute"/> class.
+    /// Initializes a new instance of the <see cref="CompositeDataSourceAttribute"/> class.
     /// </summary>
     /// <param name="attributes">The attributes representing a data source for a data theory.</param>
-    public CompositeDataAttribute(params AutoFixtureDataSourceAttribute[] attributes)
+    public CompositeDataSourceAttribute(params BaseDataSourceAttribute[] attributes)
     {
         this.attributes = attributes ?? throw new ArgumentNullException(nameof(attributes));
     }
 
     /// <summary>
     /// Gets the attributes supplied through one of the constructors.
     /// </summary>
-    public IReadOnlyList<AutoFixtureDataSourceAttribute> Attributes => Array.AsReadOnly(this.attributes);
+    public IReadOnlyList<BaseDataSourceAttribute> Attributes => Array.AsReadOnly(this.attributes);
 
     /// <inheritdoc />
     public override IEnumerable<object?[]> GetData(DataGeneratorMetadata dataGeneratorMetadata)

…oFixture.TUnit/CompositeDataAttribute.cs …re.TUnit/CompositeDataSourceAttribute.cssrc/AutoFixture.TUnit/CompositeDataAttribute.cs renamed to src/AutoFixture.TUnit/CompositeDataSourceAttribute.cs src/AutoFixture.TUnit/CompositeDataAttribute.cs renamed to src/AutoFixture.TUnit/CompositeDataSourceAttribute.cs

@@ -4,7 +4,7 @@ namespace AutoFixture.TUnit;
 
 /// <summary>
 /// Base class for customizing parameters in methods decorated with
-/// <see cref="AutoDataAttribute"/>.
+/// <see cref="AutoDataSourceAttribute"/>.
 /// </summary>
 [AttributeUsage(AttributeTargets.Parameter, AllowMultiple = true)]
 public abstract class CustomizeAttribute : Attribute, IParameterCustomizationSource

src/AutoFixture.TUnit/CustomizeAttribute.cs

@@ -4,7 +4,7 @@
 namespace AutoFixture.TUnit;
 
 /// <summary>
-/// An attribute that can be applied to parameters in an <see cref="AutoDataAttribute"/>-driven
+/// An attribute that can be applied to parameters in an <see cref="AutoDataSourceAttribute"/>-driven
 /// Theory to indicate that the parameter value should be created using a constructor with one
 /// or more array arguments, if applicable.
 /// </summary>
@@ -22,10 +22,7 @@ public sealed class FavorArraysAttribute : CustomizeAttribute
     /// </returns>
     public override ICustomization GetCustomization(ParameterInfo parameter)
     {
-        if (parameter is null)
-        {
-            throw new ArgumentNullException(nameof(parameter));
-        }
+        if (parameter is null) throw new ArgumentNullException(nameof(parameter));
 
         return new ConstructorCustomization(parameter.ParameterType, new ArrayFavoringConstructorQuery());
     }

src/AutoFixture.TUnit/FavorArraysAttribute.cs

@@ -4,7 +4,7 @@
 namespace AutoFixture.TUnit;
 
 /// <summary>
-/// An attribute that can be applied to parameters in an <see cref="AutoDataAttribute"/>-driven
+/// An attribute that can be applied to parameters in an <see cref="AutoDataSourceAttribute"/>-driven
 /// Theory to indicate that the parameter value should be created using a constructor with one
 /// or more <see cref="IEnumerable{T}" /> arguments, if applicable.
 /// </summary>
@@ -22,10 +22,7 @@ public sealed class FavorEnumerablesAttribute : CustomizeAttribute
     /// </returns>
     public override ICustomization GetCustomization(ParameterInfo parameter)
     {
-        if (parameter == null)
-        {
-            throw new ArgumentNullException(nameof(parameter));
-        }
+        if (parameter is null) throw new ArgumentNullException(nameof(parameter));
 
         return new ConstructorCustomization(parameter.ParameterType, new EnumerableFavoringConstructorQuery());
     }