mirror of
https://github.com/Homebrew/brew.git
synced 2025-07-14 16:09:03 +08:00
cask/url: deprecate url do
blocks.
We're removing these in https://github.com/Homebrew/homebrew-cask/pull/186501 so let's deprecate them too.
This commit is contained in:
parent
d1509cbef0
commit
5b46ec70e9
@ -117,6 +117,8 @@ module Cask
|
|||||||
@uri = uri
|
@uri = uri
|
||||||
@dsl = dsl
|
@dsl = dsl
|
||||||
@block = block
|
@block = block
|
||||||
|
|
||||||
|
odeprecated "cask `url do` blocks" if @block
|
||||||
end
|
end
|
||||||
|
|
||||||
sig { returns(T.any(T.any(URI::Generic, String), [T.any(URI::Generic, String), Hash])) }
|
sig { returns(T.any(T.any(URI::Generic, String), [T.any(URI::Generic, String), Hash])) }
|
||||||
|
@ -948,36 +948,6 @@ RSpec.describe Cask::Audit, :cask do
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
describe "url checks" do
|
|
||||||
let(:only) { %w[unnecessary_verified missing_verified no_match] }
|
|
||||||
|
|
||||||
context "with a block" do
|
|
||||||
let(:cask_token) { "booby-trap" }
|
|
||||||
|
|
||||||
context "when loading the cask" do
|
|
||||||
it "does not evaluate the block" do
|
|
||||||
expect { cask }.not_to raise_error
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
context "when doing an offline audit" do
|
|
||||||
let(:online) { false }
|
|
||||||
|
|
||||||
it "does not evaluate the block" do
|
|
||||||
expect(run).not_to error_with(/Boom/)
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
context "when doing and online audit" do
|
|
||||||
let(:online) { true }
|
|
||||||
|
|
||||||
it "evaluates the block" do
|
|
||||||
expect(run).to error_with(/Boom/)
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
|
||||||
|
|
||||||
describe "token conflicts" do
|
describe "token conflicts" do
|
||||||
let(:only) { ["token_conflicts"] }
|
let(:only) { ["token_conflicts"] }
|
||||||
let(:cask_token) { "with-binary" }
|
let(:cask_token) { "with-binary" }
|
||||||
@ -1171,36 +1141,6 @@ RSpec.describe Cask::Audit, :cask do
|
|||||||
|
|
||||||
it { is_expected.to error_with(/a homepage stanza is required/) }
|
it { is_expected.to error_with(/a homepage stanza is required/) }
|
||||||
end
|
end
|
||||||
|
|
||||||
context "when url is lazy" do
|
|
||||||
let(:strict) { true }
|
|
||||||
let(:cask_token) { "with-lazy" }
|
|
||||||
let(:cask) do
|
|
||||||
tmp_cask cask_token.to_s, <<~RUBY
|
|
||||||
cask '#{cask_token}' do
|
|
||||||
version '1.8.0_72,8.13.0.5'
|
|
||||||
sha256 '8dd95daa037ac02455435446ec7bc737b34567afe9156af7d20b2a83805c1d8a'
|
|
||||||
url do
|
|
||||||
['https://brew.sh/foo.zip', {referer: 'https://example.com', cookies: {'foo' => 'bar'}}]
|
|
||||||
end
|
|
||||||
name 'Audit'
|
|
||||||
desc 'Audit Description'
|
|
||||||
homepage 'https://brew.sh'
|
|
||||||
app 'Audit.app'
|
|
||||||
end
|
|
||||||
RUBY
|
|
||||||
end
|
|
||||||
|
|
||||||
it { is_expected.to pass }
|
|
||||||
|
|
||||||
it "receives a referer" do
|
|
||||||
expect(audit.cask.url.referer).to eq "https://example.com"
|
|
||||||
end
|
|
||||||
|
|
||||||
it "receives cookies" do
|
|
||||||
expect(audit.cask.url.cookies).to eq "foo" => "bar"
|
|
||||||
end
|
|
||||||
end
|
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
@ -1,8 +0,0 @@
|
|||||||
cask "booby-trap" do
|
|
||||||
version "0.0.7"
|
|
||||||
|
|
||||||
url do
|
|
||||||
# to be lazily evaluated
|
|
||||||
raise "Boom"
|
|
||||||
end
|
|
||||||
end
|
|
Loading…
x
Reference in New Issue
Block a user