Bug 953767 - Arguments that are arrays should set the options to an a…

…rray

lib/rhc/commands.rb

@@ -299,6 +299,7 @@ def self.fill_arguments(cmd, options, args_metadata, options_metadata, args)
             raise ArgumentError, "Missing required argument '#{arg[:name]}'." unless arg[:optional]
             break if available.empty?
           else
+            value = Array(value) if arg[:arg_type] == :list
             slots[i] = value
             options.__hash__[option] = value if option
           end

spec/rhc/command_spec.rb

@@ -118,7 +118,7 @@ def test; 1; end
             alias_action :exe, :deprecated => true
             def execute(testarg); 1; end
 
-            argument :args, "Test arg list", ['--tests'], :arg_type => :list
+            argument :args, "Test arg list", ['--tests ARG'], :arg_type => :list
             summary "Test command execute-list"
             def execute_list(args); 1; end
 
@@ -185,6 +185,7 @@ def context_var
         it { expects_running('static-execute-list', '--trace').should call(:execute_list).on(instance).with([]) }
         it { expects_running('static-execute-list', '1', '2', '3').should call(:execute_list).on(instance).with(['1', '2', '3']) }
         it { expects_running('static-execute-list', '1', '2', '3').should call(:execute_list).on(instance).with(['1', '2', '3']) }
+        it('should make the option an array') { expects_running('static-execute-list', '--tests', '1').should call(:execute_list).on(instance).with(['1']) }
         it('should make the option available') { command_for('static-execute-list', '1', '2', '3').send(:options).tests.should == ['1','2','3'] }
       end