Merge pull request #96 from cyberious/uniq_nodes

Add a uniq function to nodes array to ensure we deduplicate list
puppetlabs · Oct 16, 2017 · 555b4ed · 555b4ed
2 parents 103438c + 3a456bb
commit 555b4ed
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/lib/bolt/cli.rb b/lib/bolt/cli.rb
@@ -227,7 +227,7 @@ def print_help(mode)
 
     def parse_nodes(nodes)
       list = get_arg_input(nodes)
-      list.split(/[[:space:],]+/).reject(&:empty?)
+      list.split(/[[:space:],]+/).reject(&:empty?).uniq
     end
 
     def parse_params(params)

diff --git a/spec/bolt/cli_spec.rb b/spec/bolt/cli_spec.rb
@@ -105,6 +105,11 @@
       end
     end
 
+    it "accepts multiple nodes but is uniq" do
+      cli = Bolt::CLI.new(%w[command run --nodes foo,bar,foo])
+      expect(cli.parse).to include(nodes: %w[foo bar])
+    end
+
     it "generates an error message if no nodes given" do
       cli = Bolt::CLI.new(%w[command run --nodes])
       expect {