fix(derive): Support 'group' attribute with Parser

clap-rs · Sep 30, 2022 · 5f145a2 · 5f145a2
1 parent 106d8f5
commit 5f145a2
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/clap_derive/src/lib.rs b/clap_derive/src/lib.rs
@@ -42,7 +42,7 @@ pub fn value_enum(input: TokenStream) -> TokenStream {
 /// receiving an instance of `clap::ArgMatches` from conducting parsing, and then
 /// implementing a conversion code to instantiate an instance of the user
 /// context struct.
-#[proc_macro_derive(Parser, attributes(clap, structopt, command, arg))]
+#[proc_macro_derive(Parser, attributes(clap, structopt, command, arg, group))]
 #[proc_macro_error]
 pub fn parser(input: TokenStream) -> TokenStream {
     let input: DeriveInput = parse_macro_input!(input);

diff --git a/tests/derive/groups.rs b/tests/derive/groups.rs
@@ -62,6 +62,7 @@ For more information try '--help'
 #[test]
 fn skip_group_avoids_duplicate_ids() {
     #[derive(Parser, Debug)]
+    #[group(skip)]
     struct Opt {
         #[command(flatten)]
         first: Compose<Empty, Empty>,
@@ -88,7 +89,7 @@ fn skip_group_avoids_duplicate_ids() {
     use clap::Args;
     assert_eq!(Empty::group_id(), None);
     assert_eq!(Compose::<Empty, Empty>::group_id(), None);
-    assert_eq!(Opt::group_id(), Some(clap::Id::from("Opt")));
+    assert_eq!(Opt::group_id(), None);
 }
 
 #[test]