Merge pull request #13 from jpdillingham/dev

Added test to check behavior when duplicate attributes are added to properties

Author: jpdillingham

Utility.CommandLine.Arguments.Tests/Arguments.cs

@@ -189,6 +189,18 @@ public void ParseCaseSensitive()
             Assert.False(test.ContainsKey("C"));
         }
 
+        /// <summary>
+        ///     Tests the <see cref="Utility.CommandLine.Arguments.Parse(string)"/> method with a string coning a single operand
+        ///     which contains a dash.
+        /// </summary>
+        [Fact]
+        public void ParseDashedOperand()
+        {
+            CommandLine.Arguments test = CommandLine.Arguments.Parse("hello-world");
+
+            Assert.Equal("hello-world", test.OperandList[0]);
+        }
+
         /// <summary>
         ///     Tests the <see cref="Utility.CommandLine.Arguments.Parse(string)"/> method with an empty string.
         /// </summary>
@@ -309,18 +321,6 @@ public void ParseOperands()
             Assert.Equal("four", test.OperandList[2]);
         }
 
-        /// <summary>
-        ///     Tests the <see cref="Utility.CommandLine.Arguments.Parse(string)"/> method with a string coning a single operand
-        ///     which contains a dash.
-        /// </summary>
-        [Fact]
-        public void ParseDashedOperand()
-        {
-            CommandLine.Arguments test = CommandLine.Arguments.Parse("hello-world");
-
-            Assert.Equal("hello-world", test.OperandList[0]);
-        }
-
         /// <summary>
         ///     Tests the <see cref="Utility.CommandLine.Arguments.Parse(string)"/> method with an explicit command line string
         ///     containing only short parameters.
@@ -443,6 +443,35 @@ public void PopulateDictionary()
             Assert.Equal(1, Integer);
         }
 
+        /// <summary>
+        ///     Tests the <see cref="Utility.CommandLine.Arguments.Populate(Type, string)"/> method with an explicit command line
+        ///     string and with a class containing duplicate properties.
+        /// </summary>
+        [Fact]
+        public void PopulateDuplicateProperties()
+        {
+            Exception ex = Record.Exception(() => CommandLine.Arguments.Populate(typeof(TestClassDuplicateProperties), "--hello world"));
+
+            Assert.Null(ex);
+
+            Assert.Equal("world", TestClassDuplicateProperties.Test1);
+            Assert.Equal(default(string), TestClassDuplicateProperties.Test2);
+        }
+
+        /// <summary>
+        ///     Tests the <see cref="Utility.CommandLine.Arguments.Populate(Type, string)"/> method with a Type external to the
+        ///     calling class and with an explicit string.
+        /// </summary>
+        [Fact]
+        public void PopulateExternalClass()
+        {
+            CommandLine.Arguments.Populate(typeof(TestClassPublicProperties), "--test test! operand1 operand2");
+
+            Assert.Equal("test!", TestClassPublicProperties.Test);
+            Assert.Equal("operand1", TestClassPublicProperties.Operands[0]);
+            Assert.Equal("operand2", TestClassPublicProperties.Operands[1]);
+        }
+
         /// <summary>
         ///     Tests the <see cref="Utility.CommandLine.Arguments.Populate(Type, string)"/> method with an explicit command line
         ///     string containing two operands.
@@ -506,20 +535,6 @@ public void PopulateTypeMismatch()
             Assert.IsType<ArgumentException>(ex);
         }
 
-        /// <summary>
-        ///     Tests the <see cref="Utility.CommandLine.Arguments.Populate(Type, string)"/> method with a Type external to the
-        ///     calling class and with an explicit string.
-        /// </summary>
-        [Fact]
-        public void PopulateExternalClass()
-        {
-            CommandLine.Arguments.Populate(typeof(TestClassPublicProperties), "--test test! operand1 operand2");
-
-            Assert.Equal("test!", TestClassPublicProperties.Test);
-            Assert.Equal("operand1", TestClassPublicProperties.Operands[0]);
-            Assert.Equal("operand2", TestClassPublicProperties.Operands[1]);
-        }
-
         #endregion Public Methods
     }
 
@@ -543,6 +558,52 @@ public void Constructor()
         #endregion Public Methods
     }
 
+    /// <summary>
+    ///     Unit tests for the <see cref="CommandLine.Arguments"/> class.
+    /// </summary>
+    /// <remarks>Used to facilitate testing of a class with duplicate properties.</remarks>
+    public class TestClassDuplicateProperties
+    {
+        #region Public Properties
+
+        /// <summary>
+        ///     Gets or sets a test property.
+        /// </summary>
+        [CommandLine.Argument('h', "hello")]
+        public static string Test1 { get; set; }
+
+        /// <summary>
+        ///     Gets or sets a test property.
+        /// </summary>
+        [CommandLine.Argument('h', "hello")]
+        public static string Test2 { get; set; }
+
+        #endregion Public Properties
+    }
+
+    /// <summary>
+    ///     Unit tests for the <see cref="CommandLine.Arguments"/> class.
+    /// </summary>
+    /// <remarks>Used to facilitate testing of a class with public properties.</remarks>
+    public class TestClassPublicProperties
+    {
+        #region Public Properties
+
+        /// <summary>
+        ///     Gets or sets a test property.
+        /// </summary>
+        [CommandLine.Operands]
+        public static string[] Operands { get; set; }
+
+        /// <summary>
+        ///     Gets or sets a test property.
+        /// </summary>
+        [CommandLine.Argument('t', "test")]
+        public static string Test { get; set; }
+
+        #endregion Public Properties
+    }
+
     /// <summary>
     ///     Unit tests for the <see cref="CommandLine.Arguments"/> class.
     /// </summary>
@@ -552,15 +613,15 @@ public void Constructor()
     [Collection("Arguments")]
     public class TestClassWithArrayOperands
     {
-        #region Private Properties
+        #region Public Properties
 
         /// <summary>
         ///     Gets or sets a test property.
         /// </summary>
         [CommandLine.Operands]
         public static string[] Operands { get; set; }
 
-        #endregion Private Properties
+        #endregion Public Properties
 
         #region Public Methods
 
@@ -642,23 +703,4 @@ public void Populate()
 
         #endregion Public Methods
     }
-
-    /// <summary>
-    ///     Unit tests for the <see cref="CommandLine.Arguments"/> class.
-    /// </summary>
-    /// <remarks>Used to facilitate testing of a class with public properties.</remarks>
-    public class TestClassPublicProperties
-    {
-        /// <summary>
-        ///     Gets or sets a test property.
-        /// </summary>
-        [CommandLine.Argument('t', "test")]
-        public static string Test { get; set; }
-
-        /// <summary>
-        ///     Gets or sets a test property.
-        /// </summary>
-        [CommandLine.Operands]
-        public static string[] Operands { get; set; }
-    }
 }