Merge pull request #2678 from reitermarkus/fix-cli-default-options

Fix cli default options.

Library/Homebrew/cask/lib/hbc/cli/options.rb

@@ -24,10 +24,12 @@ module Hbc
 
           if [true, false].include?(default_value)
             define_method(:"#{method}?") do
+              return default_value unless instance_variable_defined?(:"@#{method}")
               instance_variable_get(:"@#{method}") == true
             end
           else
             define_method(:"#{method}") do
+              return default_value unless instance_variable_defined?(:"@#{method}")
               instance_variable_get(:"@#{method}")
             end
           end

Library/Homebrew/test/cask/cli_spec.rb

@@ -13,6 +13,13 @@ describe Hbc::CLI, :cask do
                           ])
   end
 
+  context "when no option is specified" do
+    it "--binaries is true by default" do
+      command = Hbc::CLI::Install.new("some-cask")
+      expect(command.binaries?).to be true
+    end
+  end
+
   context "::run" do
     let(:noop_command) { double("CLI::Noop") }