fix: support the Description field of VersionOptionFromMemberAttribute (

#490)
natemcmaster · Jan 30, 2022 · 3352423 · 3352423
1 parent e81e13b
commit 3352423
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/src/CommandLineUtils/Attributes/VersionOptionFromMemberAttribute.cs b/src/CommandLineUtils/Attributes/VersionOptionFromMemberAttribute.cs
@@ -64,7 +64,9 @@ internal CommandOption Configure(CommandLineApplication app, Type type, Func<obj
  };
  }
 
- return app.VersionOption(Template, shortFormGetter, longFormGetter);
+ var option = app.VersionOption(Template, shortFormGetter, longFormGetter);
+ Configure(option);
+ return option;
  }
  }
 }
diff --git a/test/CommandLineUtils.Tests/VersionOptionFromMemberAttributeTests.cs b/test/CommandLineUtils.Tests/VersionOptionFromMemberAttributeTests.cs
@@ -2,6 +2,7 @@
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
 using System;
+using System.Linq;
 using Xunit;
 
 namespace McMaster.Extensions.CommandLineUtils.Tests
@@ -67,5 +68,19 @@ public void ThrowWhenMemberDoesNotExist()
  });
  Assert.Equal(Strings.NoPropertyOrMethodFound("Version", typeof(MissingMethod)), ex.Message);
  }
+
+ [VersionOptionFromMember(Description = "42")]
+ private class Description
+ {
+ }
+
+ [Fact]
+ public void GetsDescriptionFromOption()
+ {
+ var app = new CommandLineApplication<Description>();
+ app.Conventions.UseVersionOptionFromMemberAttribute();
+
+ Assert.Equal("42", app.Options.First().Description);
+ }
  }
 }