fix: Method get_args return an iterator of &Id

clap-rs · Oct 3, 2022 · 88f0c5f · 88f0c5f
1 parent 4d4126a
commit 88f0c5f
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/src/builder/arg_group.rs b/src/builder/arg_group.rs
@@ -199,8 +199,8 @@ impl ArgGroup {
     ///
     /// assert_eq!(group.get_args(), ["f", "c"]);
     /// ```
-    pub fn get_args(self) -> Vec<Id> {
-        self.args
+    pub fn get_args(&self) -> impl Iterator<Item = &Id> {
+        self.args.iter()
     }
 
     /// Allows more than one of the [`Arg`]s in this group to be used. (Default: `false`)
@@ -587,6 +587,10 @@ mod test {
         let args: Vec<Id> = vec!["a1".into(), "a4".into()];
         let grp = ArgGroup::new("program").args(&args);
 
-        assert_eq!(grp.get_args(), args);
+        let mut pos = 0;
+        for arg in grp.get_args() {
+            assert_eq!(*arg, args[pos]);
+            pos += 1;
+        }
     }
 }