Merge pull request #500 from doudou/fix_subcommand_minus_minus_help

fix 'thor_script subcommand [args] --help' 🌈

Author: rafaelfranca

lib/thor.rb

@@ -235,8 +235,11 @@ def subcommand(subcommand, subcommand_class)
 
       define_method(subcommand) do |*args|
         args, opts = Thor::Arguments.split(args)
-        args.unshift("help") if opts.include? "--help" or opts.include? "-h"
-        invoke subcommand_class, args, opts, :invoked_via_subcommand => true, :class_options => options
+        invoke_args = [args, opts, { :invoked_via_subcommand => true, :class_options => options }]
+        if opts.delete('--help') or opts.delete("-h")
+          invoke_args.unshift 'help'
+        end
+        invoke subcommand_class, *invoke_args
       end
     end
     alias_method :subtask, :subcommand

spec/thor_spec.rb

@@ -441,6 +441,18 @@ def bar
     end
   end
 
+  describe "subcommands" do
+    it "triggers a subcommand help when passed --help" do
+      parent = Class.new(Thor)
+      child  = Class.new(Thor)
+      parent.desc 'child', 'child subcommand'
+      parent.subcommand 'child', child
+      parent.desc 'dummy', 'dummy'
+      expect(child).to receive(:help).with(anything, anything)
+      parent.start ['child', '--help']
+    end
+  end
+
   describe "when creating commands" do
     it "prints a warning if a public method is created without description or usage" do
       expect(capture(:stdout) do