Merge pull request #575 from danajp/fix-no-opt-boolean-arg-parsing

Fix --no-opt boolean parsing when followed by an argument

Author: rafaelfranca

lib/thor/parser/options.rb

@@ -189,7 +189,7 @@ def parse_boolean(switch)
           shift
           false
         else
-          true
+          !no_or_skip?(switch)
         end
       else
         @switches.key?(switch) || !no_or_skip?(switch)

spec/parser/options_spec.rb

@@ -349,6 +349,18 @@ def remaining
         expect(parse("--foo", "bar")).to eq("foo" => true)
         expect(@opt.remaining).to eq(%w(bar))
       end
+
+      it "doesn't eat the next part of the param with 'no-opt' variant" do
+        create :foo => :boolean
+        expect(parse("--no-foo", "bar")).to eq("foo" => false)
+        expect(@opt.remaining).to eq(%w(bar))
+      end
+
+      it "doesn't eat the next part of the param with 'skip-opt' variant" do
+        create :foo => :boolean
+        expect(parse("--skip-foo", "bar")).to eq("foo" => false)
+        expect(@opt.remaining).to eq(%w(bar))
+      end
     end
 
     describe "with :hash type" do