Merge pull request #5 from darthfabar/development

[FIX] filter framework type properties

Author: darthfabar

Source/ContractModelsAttributeCheck/AttributeChecker.cs

@@ -137,6 +137,7 @@ private void SearchPropertyForUsedTypes(DistinctTypeList distinctTypeList, Prope
             _ = ShouldSkipType(distinctTypeList, currentType) ||
                 SearchForTypesWhenItIsAnArray(currentType, distinctTypeList) ||
                 SearchForTypesWhenItIsAGenericType(currentType, distinctTypeList) ||
+                ShouldSkipFrameworkType(currentType) ||
                 SearchForTypesWhenItIsAClass(currentType, distinctTypeList);
         }
 

Tests/ContractModelsAttributeCheck.Test/AttributeCheckerTests.cs

@@ -96,6 +96,16 @@ public void Array_Should_Not_Add_ArrayProperties()
             missingAttributes.Count.Should().Be(0);
         }
 
+        [Fact]
+        public void Should_Not_Contain_Uri_Properties()
+        {
+            var results = _attributeChecker.CheckPropertiesForAttributes(typeof(TestClassWithSystemTypeProperty), _attributes);
+
+            var missingAttributes = results.Where(w => !w.HasRequiredAttribute).ToList();
+            missingAttributes.Count.Should().Be(1);
+
+        }
+
         [Fact]
         public void Validate_DistinctTypeList()
         {

Tests/ContractModelsAttributeCheck.Test/TestTypes/TestClassWithSystemTypeProperty.cs

@@ -0,0 +1,10 @@
+#nullable disable
+using System;
+
+namespace ContractModelsAttributeCheck.Test.TestTypes
+{
+    public class TestClassWithSystemTypeProperty
+    {
+        public Uri Url { get; set; }
+    }
+}

Tests/ContractModelsAttributeCheck.Test/TestTypes/TestClassWithoutAttributes.cs

@@ -1,4 +1,5 @@
 #nullable disable
+using System;
 using System.Collections.Generic;
 
 namespace ContractModelsAttributeCheck.Test.TestTypes