Merge pull request #402 from zii-dmg/use-value-index

User [Value] index when parsing

Author: nemec

src/CommandLine/Core/InstanceBuilder.cs

@@ -67,7 +67,7 @@ public static ParserResult<T> Build<T>(
 
                 var valueSpecPropsResult =
                     ValueMapper.MapValues(
-                        (from pt in specProps where pt.Specification.IsValue() select pt),
+                        (from pt in specProps where pt.Specification.IsValue() orderby ((ValueSpecification)pt.Specification).Index select pt),
                         valuesPartition,
                         (vals, type, isScalar) => TypeConverter.ChangeType(vals, type, isScalar, parsingCulture, ignoreValueCase));
 

tests/CommandLine.Tests/CommandLine.Tests.csproj

@@ -61,6 +61,7 @@
     <Compile Include="Fakes\Options_With_Default_Set_To_Sequence.cs" />
     <Compile Include="Fakes\Options_With_Guid.cs" />
     <Compile Include="Fakes\Options_With_SetName_That_Ends_With_Previous_SetName.cs" />
+    <Compile Include="Fakes\Options_With_Shuffled_Index_Values.cs" />
     <Compile Include="Fakes\Options_With_Uri_And_SimpleType.cs" />
     <Compile Include="Fakes\Options_With_Switches.cs" />
     <Compile Include="Fakes\Options_With_Two_Option_Required_Set_To_True_And_Two_Sets.cs" />

tests/CommandLine.Tests/Fakes/Options_With_Shuffled_Index_Values.cs

@@ -0,0 +1,19 @@
+// Copyright 2005-2015 Giacomo Stelluti Scala & Contributors. All rights reserved. See License.md in the project root for license information.
+
+using System;
+using System.Linq;
+
+namespace CommandLine.Tests.Fakes
+{
+    class Options_With_Shuffled_Index_Values
+    {
+        [Value(1)]
+        public string Arg1 { get; set; }
+
+        [Value(2)]
+        public string Arg2 { get; set; }
+
+        [Value(0)]
+        public string Arg0 { get; set; }
+    }
+}

tests/CommandLine.Tests/Unit/ParserTests.cs

@@ -803,5 +803,20 @@ public class NullDefaultCommandLineArguments
             [Option('u', "user", Default = null)]
             public string User { get; set; }
         }
+
+        [Fact]
+        public void Parse_options_with_shuffled_index_values()
+        {
+            var parser = Parser.Default;
+            parser.ParseArguments<Options_With_Shuffled_Index_Values>(
+                new[] { "zero", "one", "two" })
+                .WithNotParsed(errors => { throw new InvalidOperationException("Must be parsed."); })
+                .WithParsed(args =>
+                {
+                    Assert.Equal("zero", args.Arg0);
+                    Assert.Equal("one", args.Arg1);
+                    Assert.Equal("two", args.Arg2);
+                });
+        }
     }
 }