Merge pull request #36 from omuleanu/dev

xml docs, nunit adapter update
omuleanu · May 27, 2024 · 77551ca · 77551ca
2 parents c39e8a7 + 7ead111
commit 77551ca
Show file tree

Hide file tree

Showing 13 changed files with 153 additions and 3 deletions.
diff --git a/Tests/Tests.csproj b/Tests/Tests.csproj
@@ -1,6 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <Project ToolsVersion="12.0" DefaultTargets="Build" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
-  <Import Project="..\packages\NUnitTestAdapter.2.2.0\build\NUnitTestAdapter.props" Condition="Exists('..\packages\NUnitTestAdapter.2.2.0\build\NUnitTestAdapter.props')" />
+  <Import Project="..\packages\NUnitTestAdapter.2.3.0\build\NUnitTestAdapter.props" Condition="Exists('..\packages\NUnitTestAdapter.2.3.0\build\NUnitTestAdapter.props')" />
   <PropertyGroup>
     <Configuration Condition=" '$(Configuration)' == '' ">Debug</Configuration>
     <Platform Condition=" '$(Platform)' == '' ">AnyCPU</Platform>
@@ -142,7 +142,7 @@
     <PropertyGroup>
       <ErrorText>This project references NuGet package(s) that are missing on this computer. Use NuGet Package Restore to download them.  For more information, see http://go.microsoft.com/fwlink/?LinkID=322105. The missing file is {0}.</ErrorText>
     </PropertyGroup>
-    <Error Condition="!Exists('..\packages\NUnitTestAdapter.2.2.0\build\NUnitTestAdapter.props')" Text="$([System.String]::Format('$(ErrorText)', '..\packages\NUnitTestAdapter.2.2.0\build\NUnitTestAdapter.props'))" />
+    <Error Condition="!Exists('..\packages\NUnitTestAdapter.2.3.0\build\NUnitTestAdapter.props')" Text="$([System.String]::Format('$(ErrorText)', '..\packages\NUnitTestAdapter.2.3.0\build\NUnitTestAdapter.props'))" />
   </Target>
   <!-- To modify your build process, add your task inside one of the targets below and uncomment it. 
        Other similar extension points exist, see Microsoft.Common.targets.

diff --git a/Tests/packages.config b/Tests/packages.config
@@ -1,5 +1,5 @@
 <?xml version="1.0" encoding="utf-8"?>
 <packages>
   <package id="NUnit" version="2.6.4" targetFramework="net45" />
-  <package id="NUnitTestAdapter" version="2.2.0" targetFramework="net461" />
+  <package id="NUnitTestAdapter" version="2.3.0" targetFramework="net461" />
 </packages>
diff --git a/ValueInjecter/Injections/FlatLoopInjection.cs b/ValueInjecter/Injections/FlatLoopInjection.cs
@@ -13,6 +13,9 @@ namespace Omu.ValueInjecter.Injections
     /// </summary>
     public class FlatLoopInjection : ValueInjection
     {
+        /// <summary>
+        /// <inheritdoc/>
+        /// </summary>        
         protected override void Inject(object source, object target)
         {
             var targetProps = target.GetType().GetProps();
@@ -22,16 +25,26 @@ protected override void Inject(object source, object target)
             }
         }
 
+        /// <summary>
+        /// match properties
+        /// </summary>        
         protected virtual bool Match(string propName, PropertyInfo unflatProp, PropertyInfo targetFlatProp)
         {
             return unflatProp.PropertyType == targetFlatProp.PropertyType && propName == unflatProp.Name && unflatProp.GetGetMethod() != null;
         }
 
+
+        /// <summary>
+        /// set value in the target property
+        /// </summary>
         protected virtual void SetValue(object source, object target, PropertyInfo sp, PropertyInfo tp)
         {
             tp.SetValue(target, sp.GetValue(source, null), null);
         }
 
+        /// <summary>
+        /// execute injection on all properties
+        /// </summary>
         protected void Execute(PropertyInfo tp, object source, object target)
         {
             if (tp.CanWrite && tp.GetSetMethod() != null)

diff --git a/ValueInjecter/Injections/INoSourceInjection.cs b/ValueInjecter/Injections/INoSourceInjection.cs
@@ -1,7 +1,15 @@
 namespace Omu.ValueInjecter.Injections
 {
+    /// <summary>
+    /// injection without source
+    /// </summary>
     public interface INoSourceInjection
     {
+        /// <summary>
+        /// inject value into target object
+        /// </summary>
+        /// <param name="target"></param>
+        /// <returns></returns>
         object Map(object target);
     }
 }
diff --git a/ValueInjecter/Injections/KnownTargetInjection.cs b/ValueInjecter/Injections/KnownTargetInjection.cs
@@ -1,7 +1,17 @@
 namespace Omu.ValueInjecter.Injections
 {
+    /// <summary>
+    /// known target type injection
+    /// </summary>
+    /// <typeparam name="TTarget"></typeparam>
     public abstract class KnownTargetInjection<TTarget> : IValueInjection
     {
+        /// <summary>
+        /// set values into target object based on source object
+        /// </summary>
+        /// <param name="source"></param>
+        /// <param name="target"></param>
+        /// <returns></returns>
         public object Map(object source, object target)
         {
             var theTarget = (TTarget) target;
@@ -10,6 +20,11 @@ public object Map(object source, object target)
             return target;
         }
 
+        /// <summary>
+        /// set values into target object based on source object
+        /// </summary>
+        /// <param name="source"></param>
+        /// <param name="target"></param>
         protected abstract void Inject(object source, ref TTarget target);
     }
 }
diff --git a/ValueInjecter/Injections/LoopInjection.cs b/ValueInjecter/Injections/LoopInjection.cs
@@ -12,31 +12,55 @@ namespace Omu.ValueInjecter.Injections
     /// </summary>
     public class LoopInjection : PropertyInjection
     {
+        /// <summary>
+        /// 
+        /// </summary>
         public LoopInjection()
         {
         }
 
+        /// <summary>
+        /// ctor with ignored props setter
+        /// </summary>
+        /// <param name="ignoredProps"></param>
         public LoopInjection(string[] ignoredProps)
             : base(ignoredProps)
         {
         }
 
+        /// <summary>
+        /// get target property name based on source property name
+        /// </summary>
+        /// <param name="sourceName"></param>
+        /// <returns></returns>
         protected virtual string GetTargetProp(string sourceName)
         {
             return sourceName;
         }
 
+        /// <summary>
+        /// determine if types are matching
+        /// </summary>
+        /// <param name="source"></param>
+        /// <param name="target"></param>
+        /// <returns></returns>
         protected virtual bool MatchTypes(Type source, Type target)
         {
             return source == target;
         }
 
+        /// <summary>
+        /// set target property value 
+        /// </summary>
         protected virtual void SetValue(object source, object target, PropertyInfo sp, PropertyInfo tp)
         {
             var val = sp.GetValue(source, null);
             tp.SetValue(target, val, null);
         }
 
+        /// <summary>
+        /// <inheritdoc/>
+        /// </summary>
         protected override void Execute(PropertyInfo sp, object source, object target)
         {
             if (sp.CanRead && sp.GetGetMethod() != null && (ignoredProps == null || !ignoredProps.Contains(sp.Name)))

diff --git a/ValueInjecter/Injections/NoSourceInjection.cs b/ValueInjecter/Injections/NoSourceInjection.cs
@@ -5,12 +5,21 @@
     ///</summary>
     public abstract class NoSourceInjection : INoSourceInjection
     {
+        /// <summary>
+        /// map target
+        /// </summary>
+        /// <param name="target"></param>
+        /// <returns></returns>
         public object Map(object target)
         {
             Inject(target);
             return target;
         }
 
+        /// <summary>
+        /// inject value into target object
+        /// </summary>
+        /// <param name="target"></param>
         protected abstract void Inject(object target);
     }
 }
diff --git a/ValueInjecter/Injections/PropertyInjection.cs b/ValueInjecter/Injections/PropertyInjection.cs
@@ -5,19 +5,35 @@
 
 namespace Omu.ValueInjecter.Injections
 {
+    /// <summary>
+    /// Property injection
+    /// </summary>
     public class PropertyInjection : ValueInjection
     {
+        /// <summary>
+        ///  properties to ignore
+        /// </summary>
         protected string[] ignoredProps;
 
+        /// <summary>
+        /// 
+        /// </summary>
         public PropertyInjection()
         {
         }
 
+        /// <summary>
+        /// ctor with properties to ignore setter
+        /// </summary>
+        /// <param name="ignoredProps"></param>
         public PropertyInjection(string[] ignoredProps)
         {
             this.ignoredProps = ignoredProps;
         }
 
+        /// <summary>
+        /// <inheritdoc/>
+        /// </summary>
         protected override void Inject(object source, object target)
         {
             var sourceProps = source.GetType().GetProps();
@@ -27,6 +43,9 @@ protected override void Inject(object source, object target)
             }
         }
 
+        /// <summary>
+        /// <inheritdoc/>
+        /// </summary>
         protected virtual void Execute(PropertyInfo sp, object source, object target)
         {
             if (sp.CanRead && sp.GetGetMethod() != null && (ignoredProps == null || !ignoredProps.Contains(sp.Name)))

diff --git a/ValueInjecter/Injections/UnflatLoopInjection.cs b/ValueInjecter/Injections/UnflatLoopInjection.cs
@@ -14,8 +14,14 @@ namespace Omu.ValueInjecter.Injections
     /// </summary>
     public class UnflatLoopInjection : ValueInjection
     {
+        /// <summary>
+        /// 
+        /// </summary>
         protected Func<PropertyInfo, object, object> activator;
 
+        /// <summary>
+        /// 
+        /// </summary>
         public UnflatLoopInjection()
         {
         }
@@ -29,6 +35,11 @@ public UnflatLoopInjection(Func<PropertyInfo, object, object> activator)
             this.activator = activator;
         }
 
+        /// <summary>
+        /// inject values from source to target
+        /// </summary>
+        /// <param name="source"></param>
+        /// <param name="target"></param>
         protected override void Inject(object source, object target)
         {
             var sourceProps = source.GetType().GetProps();
@@ -38,16 +49,36 @@ protected override void Inject(object source, object target)
             }
         }
 
+        /// <summary>
+        /// check if properties match
+        /// </summary>
+        /// <param name="propName"></param>
+        /// <param name="unflatProp"></param>
+        /// <param name="sourceFlatProp"></param>
+        /// <returns></returns>
         protected virtual bool Match(string propName, PropertyInfo unflatProp, PropertyInfo sourceFlatProp)
         {
             return unflatProp.PropertyType == sourceFlatProp.PropertyType && propName == unflatProp.Name && unflatProp.GetSetMethod() != null;
         }
 
+        /// <summary>
+        /// set property value from source to target
+        /// </summary>
+        /// <param name="source"></param>
+        /// <param name="target"></param>
+        /// <param name="sp"></param>
+        /// <param name="tp"></param>
         protected virtual void SetValue(object source, object target, PropertyInfo sp, PropertyInfo tp)
         {
             tp.SetValue(target, sp.GetValue(source, null), null);
         }
 
+        /// <summary>
+        /// execute property injection from source to target object
+        /// </summary>
+        /// <param name="sp"></param>
+        /// <param name="source"></param>
+        /// <param name="target"></param>
         protected virtual void Execute(PropertyInfo sp, object source, object target)
         {
             if (sp.CanRead && sp.GetGetMethod() != null)

diff --git a/ValueInjecter/Mapper.cs b/ValueInjecter/Mapper.cs
@@ -2,6 +2,9 @@
 
 namespace Omu.ValueInjecter
 {
+    /// <summary>
+    /// mapper instance class
+    /// </summary>
     public class Mapper
     {
         /// <summary>

diff --git a/ValueInjecter/MapperInstance.cs b/ValueInjecter/MapperInstance.cs
@@ -3,6 +3,9 @@
 
 namespace Omu.ValueInjecter
 {
+    /// <summary>
+    /// mapper instance class
+    /// </summary>
     public class MapperInstance
     {
         /// <summary>

diff --git a/ValueInjecter/Utils/Extensions.cs b/ValueInjecter/Utils/Extensions.cs
@@ -5,6 +5,9 @@
 
 namespace Omu.ValueInjecter.Utils
 {
+    /// <summary>
+    /// ValueInjecter extensions
+    /// </summary>
     public static class Extensions
     {
         /// <summary>

diff --git a/ValueInjecter/Utils/StrUtil.cs b/ValueInjecter/Utils/StrUtil.cs
@@ -2,19 +2,41 @@
 
 namespace Omu.ValueInjecter.Utils
 {
+    /// <summary>
+    /// String utilities 
+    /// </summary>
     public static class StrUtil
     {
+        /// <summary>
+        /// remove string prefix
+        /// </summary>
+        /// <param name="o"></param>
+        /// <param name="prefix"></param>
+        /// <param name="comparison"></param>
+        /// <returns></returns>
         public static string RemovePrefix(string o, string prefix, StringComparison comparison)
         {
             if (prefix == null) return o;
             return !o.StartsWith(prefix, comparison) ? o : o.Remove(0, prefix.Length);
         }
 
+        /// <summary>
+        /// remove string prefix
+        /// </summary>
+        /// <param name="o"></param>
+        /// <param name="prefix"></param>
+        /// <returns></returns>
         public static string RemovePrefix(string o, string prefix)
         {
             return RemovePrefix(o, prefix, StringComparison.Ordinal);
         }
 
+        /// <summary>
+        /// remove string suffix
+        /// </summary>
+        /// <param name="o"></param>
+        /// <param name="suffix"></param>
+        /// <returns></returns>
         public static string RemoveSuffix(string o, string suffix)
         {
             if (suffix == null) return o;