Fixes #95, refactored

Author: i4004

src/Simplify.Web.Tests/Model/Validation/ValidationAttributesExecutorTests.cs

@@ -4,7 +4,8 @@
 using Simplify.Web.Model.Validation;
 using Simplify.Web.Tests.Model.Validation.Attributes;
 using Simplify.Web.Tests.TestEntities;
-using Simplify.Web.Tests.TestEntities.HierarchyValidation;
+using Simplify.Web.Tests.TestEntities.Inheritance;
+using Simplify.Web.Tests.TestEntities.Nesting;
 
 namespace Simplify.Web.Tests.Model.Validation
 {
@@ -35,17 +36,16 @@ public void Validate_ModelWithOnePropertyAndOneValidateAttribute_AttributeValida
 		}
 
 		[Test]
-		public void Validate_HierarchicalModel_NestedNullAttributeException()
+		public void Validate_NestedProperties_NestedNullAttributeException()
 		{
 			// Arrange
-			var model = new RootModel
+			var model = new NestingRootModel
 			{
-				//BuiltInType = "test",
-				CustomType = new ChildModel
+				NestedProperty = new NestedModel
 				{
-					//BuiltInType = "test",
-					CustomType = new SubChildModel()
-				}
+					NestedProperty = new SubNestedModel()
+				},
+				TestInt = 1
 			};
 
 			// Act
@@ -54,7 +54,24 @@ public void Validate_HierarchicalModel_NestedNullAttributeException()
 
 			// Assert
 			Assert.That(ex.Message,
-				Does.StartWith($"Required property '{nameof(SubChildModel.BuiltInType)}' is null or empty"));
+				Does.StartWith($"Required property '{nameof(SubNestedModel.BuiltInType)}' is null or empty"));
+		}
+
+		[Test]
+		public void Validate_InheritedProperty_NestedNullAttributeException()
+		{
+			// Arrange
+			var model = new InheritanceRootModel
+			{
+				NestedProperty = new BaseNestedModel()
+			};
+
+			// Act
+			var ex = Assert.Throws<ModelValidationException>(() => _validator.Validate(model, null));
+
+			// Assert
+			Assert.That(ex.Message,
+				Does.StartWith($"Required property '{nameof(BaseNestedModel.BuiltInType)}' is null or empty"));
 		}
 	}
 }

src/Simplify.Web.Tests/TestEntities/HierarchyValidation/BaseModel.cs

@@ -0,0 +1,12 @@
+using Simplify.Web.Model.Validation.Attributes;
+
+#nullable disable
+
+namespace Simplify.Web.Tests.TestEntities.Inheritance
+{
+	public class BaseModel
+	{
+		[Required]
+		public BaseNestedModel NestedProperty { get; set; }
+	}
+}

src/Simplify.Web.Tests/TestEntities/HierarchyValidation/ChildModel.cs

@@ -0,0 +1,12 @@
+using Simplify.Web.Model.Validation.Attributes;
+
+#nullable disable
+
+namespace Simplify.Web.Tests.TestEntities.Inheritance
+{
+	public class BaseNestedModel
+	{
+		[Required]
+		public string BuiltInType { get; set; }
+	}
+}

src/Simplify.Web.Tests/TestEntities/HierarchyValidation/RootModel.cs

@@ -0,0 +1,8 @@
+#nullable disable
+
+namespace Simplify.Web.Tests.TestEntities.Inheritance
+{
+	public class InheritanceRootModel : BaseModel
+	{
+	}
+}

src/Simplify.Web.Tests/TestEntities/HierarchyValidation/SubChildModel.cs

@@ -1,8 +1,8 @@
 using Simplify.Web.Model.Validation.Attributes;
 
-namespace Simplify.Web.Tests.TestEntities.HierarchyValidation
+namespace Simplify.Web.Tests.TestEntities.Nesting
 {
-	public interface ISubChildModel
+	public interface ISubNestedModel
 	{
 		[Required]
 		string BuiltInType { get; set; }

src/Simplify.Web.Tests/TestEntities/Inheritance/BaseModel.cs

@@ -0,0 +1,12 @@
+#nullable disable
+
+using Simplify.Web.Model.Validation.Attributes;
+
+namespace Simplify.Web.Tests.TestEntities.Nesting
+{
+	public class NestedModel
+	{
+		[Required]
+		public ISubNestedModel NestedProperty { get; set; }
+	}
+}

src/Simplify.Web.Tests/TestEntities/Inheritance/BaseNestedModel.cs

@@ -0,0 +1,14 @@
+#nullable disable
+
+using Simplify.Web.Model.Validation.Attributes;
+
+namespace Simplify.Web.Tests.TestEntities.Nesting
+{
+	public class NestingRootModel
+	{
+		public NestedModel NestedProperty { get; set; }
+
+		[Required]
+		public int? TestInt { get; set; }
+	}
+}

src/Simplify.Web.Tests/TestEntities/Inheritance/InheritanceRootModel.cs

@@ -0,0 +1,9 @@
+#nullable disable
+
+namespace Simplify.Web.Tests.TestEntities.Nesting
+{
+	public class SubNestedModel : ISubNestedModel
+	{
+		public string BuiltInType { get; set; }
+	}
+}

src/Simplify.Web.Tests/TestEntities/HierarchyValidation/ISubChildModel.cs renamed to src/Simplify.Web.Tests/TestEntities/Nesting/ISubNestedModel.cs

@@ -28,6 +28,9 @@ private static void Validate(Type type, object? value, IDIResolver resolver)
 		{
 			var properties = type.GetProperties();
 
+			if (type.BaseType != null && type.BaseType != typeof(object))
+				Validate(type.BaseType, value, resolver);
+
 			foreach (var item in properties)
 			{
 				var currentItemValue = item.GetValue(value);