diff options
| author | Mike McQuaid | 2017-12-29 15:53:39 +0000 |
|---|---|---|
| committer | GitHub | 2017-12-29 15:53:39 +0000 |
| commit | c6eaa73db1336b3daa2f9b1cb9962d2b9e7c8922 (patch) | |
| tree | 7a71742bf30858583afab7a997646d62c53cc797 | |
| parent | c2e1b3cccff1056248bdc231c13ff3d2c9f572ff (diff) | |
| parent | 78fe082a2b85f097dae7b8333ba66a8fd31ca2d0 (diff) | |
| download | brew-c6eaa73db1336b3daa2f9b1cb9962d2b9e7c8922.tar.bz2 | |
Merge pull request #3604 from alyssais/cask_no_arguments
cask: exit successfully when given no arguments.
| -rw-r--r-- | Library/Homebrew/cask/lib/hbc/cli.rb | 1 | ||||
| -rw-r--r-- | Library/Homebrew/test/cask/cli_spec.rb | 8 |
2 files changed, 8 insertions, 1 deletions
diff --git a/Library/Homebrew/cask/lib/hbc/cli.rb b/Library/Homebrew/cask/lib/hbc/cli.rb index a553ad6da..5da2a137a 100644 --- a/Library/Homebrew/cask/lib/hbc/cli.rb +++ b/Library/Homebrew/cask/lib/hbc/cli.rb @@ -229,6 +229,7 @@ module Hbc purpose usage + return if @command.nil? return if @command == "help" && @args.empty? raise ArgumentError, "help does not take arguments." diff --git a/Library/Homebrew/test/cask/cli_spec.rb b/Library/Homebrew/test/cask/cli_spec.rb index 4a61e0088..f88af4188 100644 --- a/Library/Homebrew/test/cask/cli_spec.rb +++ b/Library/Homebrew/test/cask/cli_spec.rb @@ -16,10 +16,16 @@ describe Hbc::CLI, :cask do it "ignores the `--language` option, which is handled in `OS::Mac`" do cli = described_class.new("--language=en") expect(cli).to receive(:detect_command_and_arguments).with(no_args) - expect(cli).to receive(:exit).with(1) cli.run end + context "when given no arguments" do + it "exits successfully" do + expect(subject).not_to receive(:exit).with(be_nonzero) + subject.run + end + end + context "when no option is specified" do it "--binaries is true by default" do command = Hbc::CLI::Install.new("some-cask") |
