Update parser tests
- Remove named_args linux specs now that that logic has been moved to the parser
This commit is contained in:
parent
b7e28ef48f
commit
35500130c4
@ -135,7 +135,7 @@ module Homebrew
|
|||||||
end
|
end
|
||||||
|
|
||||||
def respond_to_missing?(method_name, *)
|
def respond_to_missing?(method_name, *)
|
||||||
!frozen? || @table.key?(method_name)
|
@table.key?(method_name)
|
||||||
end
|
end
|
||||||
|
|
||||||
def method_missing(method_name, *args)
|
def method_missing(method_name, *args)
|
||||||
|
|||||||
@ -118,12 +118,6 @@ describe Homebrew::CLI::NamedArgs do
|
|||||||
expect { described_class.new("foo").to_formulae_and_casks }.to raise_error(FormulaOrCaskUnavailableError)
|
expect { described_class.new("foo").to_formulae_and_casks }.to raise_error(FormulaOrCaskUnavailableError)
|
||||||
end
|
end
|
||||||
|
|
||||||
it "raises an error when formula is absent and cask is available on linux", :needs_linux do
|
|
||||||
stub_cask_loader foo_cask
|
|
||||||
|
|
||||||
expect { described_class.new("foo").to_formulae_and_casks }.to raise_error(FormulaUnavailableError)
|
|
||||||
end
|
|
||||||
|
|
||||||
it "returns formula when formula is present and cask is unreadable", :needs_macos do
|
it "returns formula when formula is present and cask is unreadable", :needs_macos do
|
||||||
stub_formula_loader foo
|
stub_formula_loader foo
|
||||||
setup_unredable_cask "foo"
|
setup_unredable_cask "foo"
|
||||||
@ -311,12 +305,6 @@ describe Homebrew::CLI::NamedArgs do
|
|||||||
|
|
||||||
expect(described_class.new("foo", "baz").to_paths(only: :cask)).to eq [cask_path, Cask::CaskLoader.path("baz")]
|
expect(described_class.new("foo", "baz").to_paths(only: :cask)).to eq [cask_path, Cask::CaskLoader.path("baz")]
|
||||||
end
|
end
|
||||||
|
|
||||||
it "returns only formulae by default on linux", :needs_linux do
|
|
||||||
expect(Formulary).to receive(:path).with("foo").and_return(formula_path)
|
|
||||||
|
|
||||||
expect(described_class.new("foo", "baz").to_paths).to eq [formula_path, Formulary.path("baz")]
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
|
|
||||||
describe "#to_taps" do
|
describe "#to_taps" do
|
||||||
|
|||||||
@ -570,12 +570,24 @@ describe Homebrew::CLI::Parser do
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
it "throws an error by default" do
|
it "throws an error when defined" do
|
||||||
expect { parser.parse(["--cask"]) }.to raise_error UsageError, /Casks are not supported on Linux/
|
expect { parser.parse(["--cask"]) }.to raise_error UsageError, /Casks are not supported on Linux/
|
||||||
end
|
end
|
||||||
|
end
|
||||||
|
|
||||||
it "only warns developers", :needs_macos do
|
describe "--formula on linux", :needs_linux do
|
||||||
expect { parser.parse(["--cask"]) }.not_to raise_error
|
it "doesn't set --formula when not defined" do
|
||||||
|
parser = described_class.new
|
||||||
|
args = parser.parse([])
|
||||||
|
expect(args.respond_to?(:formula?)).to be(false)
|
||||||
|
end
|
||||||
|
|
||||||
|
it "sets --formula to true when defined" do
|
||||||
|
parser = described_class.new do
|
||||||
|
switch "--formula"
|
||||||
|
end
|
||||||
|
args = parser.parse([])
|
||||||
|
expect(args.formula?).to be(true)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user