Make new validations consistent with System.ComponentModel.DataAnnotations behavior

src/Http/Http/perf/Microbenchmarks/ValidatableTypesBenchmark.cs

@@ -244,6 +244,7 @@ public IEnumerable<ValidationResult> Validate(ValidationContext validationContex
 
     private class MockValidatableTypeInfo(Type type, ValidatablePropertyInfo[] members) : ValidatableTypeInfo(type, members)
     {
+        protected override ValidationAttribute[] GetValidationAttributes() => [];
     }
 
     private class MockValidatablePropertyInfo(

src/Validation/gen/Emitters/ValidationsGenerator.Emitter.cs

@@ -81,7 +81,16 @@ public GeneratedValidatablePropertyInfo(
         public GeneratedValidatableTypeInfo(
             [param: global::System.Diagnostics.CodeAnalysis.DynamicallyAccessedMembers(global::System.Diagnostics.CodeAnalysis.DynamicallyAccessedMemberTypes.Interfaces)]
             global::System.Type type,
-            ValidatablePropertyInfo[] members) : base(type, members) { }
+            ValidatablePropertyInfo[] members) : base(type, members)
+        {
+            Type = type;
+        }
+
+        [global::System.Diagnostics.CodeAnalysis.DynamicallyAccessedMembers(global::System.Diagnostics.CodeAnalysis.DynamicallyAccessedMemberTypes.Interfaces)]
+        internal global::System.Type Type { get; }
+
+        protected override global::System.ComponentModel.DataAnnotations.ValidationAttribute[] GetValidationAttributes()
+            => ValidationAttributeCache.GetValidationAttributes(Type, null);
     }
 
     {{GeneratedCodeAttribute}}
@@ -127,46 +136,60 @@ private sealed record CacheKey(
             [param: global::System.Diagnostics.CodeAnalysis.DynamicallyAccessedMembers(global::System.Diagnostics.CodeAnalysis.DynamicallyAccessedMemberTypes.PublicProperties | global::System.Diagnostics.CodeAnalysis.DynamicallyAccessedMemberTypes.PublicConstructors)]
             [property: global::System.Diagnostics.CodeAnalysis.DynamicallyAccessedMembers(global::System.Diagnostics.CodeAnalysis.DynamicallyAccessedMemberTypes.PublicProperties | global::System.Diagnostics.CodeAnalysis.DynamicallyAccessedMemberTypes.PublicConstructors)]
             global::System.Type ContainingType,
-            string PropertyName);
+            string? PropertyName);
         private static readonly global::System.Collections.Concurrent.ConcurrentDictionary<CacheKey, global::System.ComponentModel.DataAnnotations.ValidationAttribute[]> _cache = new();
 
         public static global::System.ComponentModel.DataAnnotations.ValidationAttribute[] GetValidationAttributes(
             [global::System.Diagnostics.CodeAnalysis.DynamicallyAccessedMembers(global::System.Diagnostics.CodeAnalysis.DynamicallyAccessedMemberTypes.PublicProperties | global::System.Diagnostics.CodeAnalysis.DynamicallyAccessedMemberTypes.PublicConstructors)]
             global::System.Type containingType,
-            string propertyName)
+            string? propertyName)
         {
             var key = new CacheKey(containingType, propertyName);
             return _cache.GetOrAdd(key, static k =>
             {
                 var results = new global::System.Collections.Generic.List<global::System.ComponentModel.DataAnnotations.ValidationAttribute>();
 
-                // Get attributes from the property
-                var property = k.ContainingType.GetProperty(k.PropertyName);
-                if (property != null)
+                if (k.PropertyName is not null)
                 {
-                    var propertyAttributes = global::System.Reflection.CustomAttributeExtensions
-                        .GetCustomAttributes<global::System.ComponentModel.DataAnnotations.ValidationAttribute>(property, inherit: true);
-
-                    results.AddRange(propertyAttributes);
-                }
+                    // Get attributes from the property
+                    var property = k.ContainingType.GetProperty(k.PropertyName);
+                    if (property != null)
+                    {
+                        var propertyAttributes = global::System.Reflection.CustomAttributeExtensions
+                            .GetCustomAttributes<global::System.ComponentModel.DataAnnotations.ValidationAttribute>(property, inherit: true);
 
-                // Check constructors for parameters that match the property name
-                // to handle record scenarios
-                foreach (var constructor in k.ContainingType.GetConstructors())
-                {
-                    // Look for parameter with matching name (case insensitive)
-                    var parameter = global::System.Linq.Enumerable.FirstOrDefault(
-                        constructor.GetParameters(),
-                        p => string.Equals(p.Name, k.PropertyName, global::System.StringComparison.OrdinalIgnoreCase));
+                        results.AddRange(propertyAttributes);
+                    }
 
-                    if (parameter != null)
+                    // Check constructors for parameters that match the property name
+                    // to handle record scenarios
+                    foreach (var constructor in k.ContainingType.GetConstructors())
                     {
-                        var paramAttributes = global::System.Reflection.CustomAttributeExtensions
-                            .GetCustomAttributes<global::System.ComponentModel.DataAnnotations.ValidationAttribute>(parameter, inherit: true);
+                        // Look for parameter with matching name (case insensitive)
+                        var parameter = global::System.Linq.Enumerable.FirstOrDefault(
+                            constructor.GetParameters(),
+                            p => string.Equals(p.Name, k.PropertyName, global::System.StringComparison.OrdinalIgnoreCase));
 
-                        results.AddRange(paramAttributes);
+                        if (parameter != null)
+                        {
+                            var paramAttributes = global::System.Reflection.CustomAttributeExtensions
+                                .GetCustomAttributes<global::System.ComponentModel.DataAnnotations.ValidationAttribute>(parameter, inherit: true);
 
-                        break;
+                            results.AddRange(paramAttributes);
+
+                            break;
+                        }
+                    }
+                }
+                else
+                {
+                    // Get attributes from the type itself and its super types
+                    foreach (var attr in k.ContainingType.GetCustomAttributes(typeof(global::System.ComponentModel.DataAnnotations.ValidationAttribute), true))
+                    {
+                        if (attr is global::System.ComponentModel.DataAnnotations.ValidationAttribute validationAttribute)
+                        {
+                            results.Add(validationAttribute);
+                        }
                     }
                 }
 

src/Validation/gen/Parsers/ValidationsGenerator.TypesParser.cs

@@ -82,6 +82,8 @@ internal bool TryExtractValidatableType(ITypeSymbol incomingTypeSymbol, WellKnow
 
         visitedTypes.Add(typeSymbol);
 
+        var hasValidationAttributes = HasValidationAttributes(typeSymbol, wellKnownTypes);
+
         // Extract validatable types discovered in base types of this type and add them to the top-level list.
         var current = typeSymbol.BaseType;
         var hasValidatableBaseType = false;
@@ -107,7 +109,7 @@ internal bool TryExtractValidatableType(ITypeSymbol incomingTypeSymbol, WellKnow
         }
 
         // No validatable members or derived types found, so we don't need to add this type.
-        if (members.IsDefaultOrEmpty && !hasValidatableBaseType && !hasValidatableDerivedTypes)
+        if (members.IsDefaultOrEmpty && !hasValidationAttributes && !hasValidatableBaseType && !hasValidatableDerivedTypes)
         {
             return false;
         }
@@ -283,4 +285,20 @@ internal static ImmutableArray<ValidationAttribute> ExtractValidationAttributes(
                 NamedArguments: attribute.NamedArguments.ToDictionary(namedArgument => namedArgument.Key, namedArgument => namedArgument.Value.ToCSharpString()),
                 IsCustomValidationAttribute: SymbolEqualityComparer.Default.Equals(attribute.AttributeClass, wellKnownTypes.Get(WellKnownTypeData.WellKnownType.System_ComponentModel_DataAnnotations_CustomValidationAttribute))))];
     }
+
+    internal static bool HasValidationAttributes(ISymbol symbol, WellKnownTypes wellKnownTypes)
+    {
+        var validationAttributeSymbol = wellKnownTypes.Get(WellKnownTypeData.WellKnownType.System_ComponentModel_DataAnnotations_ValidationAttribute);
+
+        foreach (var attribute in symbol.GetAttributes())
+        {
+            if (attribute.AttributeClass is not null &&
+                attribute.AttributeClass.ImplementsValidationAttribute(validationAttributeSymbol))
+            {
+                return true;
+            }
+        }
+
+        return false;
+    }
 }

src/Validation/src/PublicAPI.Unshipped.txt

@@ -1,4 +1,5 @@
 #nullable enable
+abstract Microsoft.Extensions.Validation.ValidatableTypeInfo.GetValidationAttributes() -> System.ComponentModel.DataAnnotations.ValidationAttribute![]!
 Microsoft.Extensions.DependencyInjection.ValidationServiceCollectionExtensions
 Microsoft.Extensions.Validation.IValidatableInfo
 Microsoft.Extensions.Validation.IValidatableInfo.ValidateAsync(object? value, Microsoft.Extensions.Validation.ValidateContext! context, System.Threading.CancellationToken cancellationToken) -> System.Threading.Tasks.Task!

src/Validation/src/ValidatableTypeInfo.cs

@@ -14,7 +14,7 @@ namespace Microsoft.Extensions.Validation;
 public abstract class ValidatableTypeInfo : IValidatableInfo
 {
     private readonly int _membersCount;
-    private readonly List<Type> _subTypes;
+    private readonly List<Type> _superTypes;
 
     /// <summary>
     /// Creates a new instance of <see cref="ValidatableTypeInfo"/>.
@@ -28,9 +28,15 @@ protected ValidatableTypeInfo(
         Type = type;
         Members = members;
         _membersCount = members.Count;
-        _subTypes = type.GetAllImplementedTypes();
+        _superTypes = type.GetAllImplementedTypes();
     }
 
+    /// <summary>
+    /// Gets the validation attributes for this member.
+    /// </summary>
+    /// <returns>An array of validation attributes to apply to this member.</returns>
+    protected abstract ValidationAttribute[] GetValidationAttributes();
+
     /// <summary>
     /// The type being validated.
     /// </summary>
@@ -59,75 +65,139 @@ public virtual async Task ValidateAsync(object? value, ValidateContext context,
         }
 
         var originalPrefix = context.CurrentValidationPath;
+        var originalErrorCount = context.ValidationErrors?.Count ?? 0;
 
         try
         {
+            // First validate direct members
+            await ValidateMembersAsync(value, context, cancellationToken);
+
             var actualType = value.GetType();
 
-            // First validate members
-            for (var i = 0; i < _membersCount; i++)
+            // Then validate inherited members
+            foreach (var superTypeInfo in GetSuperTypeInfos(actualType, context))
+            {
+                await superTypeInfo.ValidateMembersAsync(value, context, cancellationToken);
+            }
+
+            // If any property-level validation errors were found, return early
+            if (context.ValidationErrors is not null && context.ValidationErrors.Count > originalErrorCount)
+            {
+                return;
+            }
+
+            // Validate type-level attributes
+            ValidateTypeAttributes(value, context);
+
+            // If any type-level attribute errors were found, return early
+            if (context.ValidationErrors is not null && context.ValidationErrors.Count > originalErrorCount)
+            {
+                return;
+            }
+
+            // Finally validate IValidatableObject if implemented
+            ValidateValidatableObjectInterface(value, context);
+        }
+        finally
+        {
+            context.CurrentValidationPath = originalPrefix;
+        }
+    }
+
+    private async Task ValidateMembersAsync(object? value, ValidateContext context, CancellationToken cancellationToken)
+    {
+        var originalPrefix = context.CurrentValidationPath;
+
+        for (var i = 0; i < _membersCount; i++)
+        {
+            try
             {
                 await Members[i].ValidateAsync(value, context, cancellationToken);
+
+            }
+            finally
+            {
                 context.CurrentValidationPath = originalPrefix;
             }
+        }
+    }
+
+    private void ValidateTypeAttributes(object? value, ValidateContext context)
+    {
+        var validationAttributes = GetValidationAttributes();
+        var errorPrefix = context.CurrentValidationPath;
 
-            // Then validate sub-types if any
-            foreach (var subType in _subTypes)
+        for (var i = 0; i < validationAttributes.Length; i++)
+        {
+            var attribute = validationAttributes[i];
+            var result = attribute.GetValidationResult(value, context.ValidationContext);
+            if (result is not null && result != ValidationResult.Success && result.ErrorMessage is not null)
             {
-                // Check if the actual type is assignable to the sub-type
-                // and validate it if it is
-                if (subType.IsAssignableFrom(actualType))
+                // Create a validation error for each member name that is provided
+                foreach (var memberName in result.MemberNames)
                 {
-                    if (context.ValidationOptions.TryGetValidatableTypeInfo(subType, out var subTypeInfo))
-                    {
-                        await subTypeInfo.ValidateAsync(value, context, cancellationToken);
-                        context.CurrentValidationPath = originalPrefix;
-                    }
+                    var key = string.IsNullOrEmpty(errorPrefix) ? memberName : $"{errorPrefix}.{memberName}";
+                    context.AddOrExtendValidationError(memberName, key, result.ErrorMessage, value);
+                }
+
+                if (!result.MemberNames.Any())
+                {
+                    // If no member names are specified, then treat this as a top-level error
+                    context.AddOrExtendValidationError(string.Empty, errorPrefix, result.ErrorMessage, value);
                 }
             }
+        }
+    }
 
-            // Finally validate IValidatableObject if implemented
-            if (Type.ImplementsInterface(typeof(IValidatableObject)) && value is IValidatableObject validatable)
+    private void ValidateValidatableObjectInterface(object? value, ValidateContext context)
+    {
+        if (Type.ImplementsInterface(typeof(IValidatableObject)) && value is IValidatableObject validatable)
+        {
+            // Important: Set the DisplayName to the type name for top-level validations
+            // and restore the original validation context properties
+            var originalDisplayName = context.ValidationContext.DisplayName;
+            var originalMemberName = context.ValidationContext.MemberName;
+            var errorPrefix = context.CurrentValidationPath;
+
+            // Set the display name to the class name for IValidatableObject validation
+            context.ValidationContext.DisplayName = Type.Name;
+            context.ValidationContext.MemberName = null;
+
+            var validationResults = validatable.Validate(context.ValidationContext);
+            foreach (var validationResult in validationResults)
             {
-                // Important: Set the DisplayName to the type name for top-level validations
-                // and restore the original validation context properties
-                var originalDisplayName = context.ValidationContext.DisplayName;
-                var originalMemberName = context.ValidationContext.MemberName;
-
-                // Set the display name to the class name for IValidatableObject validation
-                context.ValidationContext.DisplayName = Type.Name;
-                context.ValidationContext.MemberName = null;
-
-                var validationResults = validatable.Validate(context.ValidationContext);
-                foreach (var validationResult in validationResults)
+                if (validationResult != ValidationResult.Success && validationResult.ErrorMessage is not null)
                 {
-                    if (validationResult != ValidationResult.Success && validationResult.ErrorMessage is not null)
+                    // Create a validation error for each member name that is provided
+                    foreach (var memberName in validationResult.MemberNames)
                     {
-                        // Create a validation error for each member name that is provided
-                        foreach (var memberName in validationResult.MemberNames)
-                        {
-                            var key = string.IsNullOrEmpty(originalPrefix) ?
-                                memberName :
-                                $"{originalPrefix}.{memberName}";
-                            context.AddOrExtendValidationError(memberName, key, validationResult.ErrorMessage, value);
-                        }
-
-                        if (!validationResult.MemberNames.Any())
-                        {
-                            // If no member names are specified, then treat this as a top-level error
-                            context.AddOrExtendValidationError(string.Empty, string.Empty, validationResult.ErrorMessage, value);
-                        }
+                        var key = string.IsNullOrEmpty(errorPrefix) ? memberName : $"{errorPrefix}.{memberName}";
+                        context.AddOrExtendValidationError(memberName, key, validationResult.ErrorMessage, value);
                     }
-                }
 
-                // Restore the original validation context properties
-                context.ValidationContext.DisplayName = originalDisplayName;
-                context.ValidationContext.MemberName = originalMemberName;
+                    if (!validationResult.MemberNames.Any())
+                    {
+                        // If no member names are specified, then treat this as a top-level error
+                        context.AddOrExtendValidationError(string.Empty, string.Empty, validationResult.ErrorMessage, value);
+                    }
+                }
             }
+
+            // Restore the original validation context properties
+            context.ValidationContext.DisplayName = originalDisplayName;
+            context.ValidationContext.MemberName = originalMemberName;
         }
-        finally
+    }
+
+    private IEnumerable<ValidatableTypeInfo> GetSuperTypeInfos(Type actualType, ValidateContext context)
+    {
+        foreach (var superType in _superTypes.Where(t => t.IsAssignableFrom(actualType)))
         {
-            context.CurrentValidationPath = originalPrefix;
+            if (context.ValidationOptions.TryGetValidatableTypeInfo(superType, out var found)
+                && found is ValidatableTypeInfo superTypeInfo)
+            {
+                yield return superTypeInfo;
+            }
         }
     }
 }