Fixed #199

unitycontainer · Mar 10, 2023 · 3353ed1 · 3353ed1
1 parent 276b202
commit 3353ed1
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 10 deletions.
diff --git a/src/Dependency/Resolution/Overrides/FieldOverride.cs b/src/Dependency/Resolution/Overrides/FieldOverride.cs
@@ -23,12 +23,12 @@ public class FieldOverride : ResolverOverride,
         /// <summary>
         /// Create an instance of <see cref="FieldOverride"/>.
         /// </summary>
-        /// <param name="fieldName">The Field name.</param>
-        /// <param name="fieldValue">InjectionParameterValue to use for the Field.</param>
-        public FieldOverride(string fieldName, object fieldValue)
-            : base(fieldName)
+        /// <param name="name">The Field name.</param>
+        /// <param name="value">InjectionParameterValue to use for the Field.</param>
+        public FieldOverride(string name, object value)
+            : base(name ?? throw new ArgumentNullException(nameof(name), "Must provide a name of the field to override"))
         {
-            Value = fieldValue;
+            Value = value;
         }
 
         #endregion

diff --git a/src/Dependency/Resolution/Overrides/PropertyOverride.cs b/src/Dependency/Resolution/Overrides/PropertyOverride.cs
@@ -1,5 +1,6 @@
 using System;
 using System.Reflection;
+using System.Xml.Linq;
 
 namespace Unity.Resolution
 {
@@ -23,12 +24,12 @@ public class PropertyOverride : ResolverOverride,
         /// <summary>
         /// Create an instance of <see cref="PropertyOverride"/>.
         /// </summary>
-        /// <param name="propertyName">The property name.</param>
-        /// <param name="propertyValue">InjectionParameterValue to use for the property.</param>
-        public PropertyOverride(string propertyName, object propertyValue)
-            : base(propertyName)
+        /// <param name="name">The property name.</param>
+        /// <param name="value">InjectionParameterValue to use for the property.</param>
+        public PropertyOverride(string name, object value)
+            : base(name ?? throw new ArgumentNullException(nameof(name), "Must provide a name of the property to override"))
         {
-            Value = propertyValue;
+            Value = value;
         }
 
         #endregion