e2759fbdff
We added the `--all` flag (now renamed to `--eval-all`) for various commands for this behaviour so let's start deprecating this. Also, introduce a `HOMEBREW_EVAL_ALL` environment variable to use the existing, less secure, behaviour by default and avoid passing `--eval-all` everywhere.
89 lines
2.7 KiB
Ruby
89 lines
2.7 KiB
Ruby
# typed: false
|
|
# frozen_string_literal: true
|
|
|
|
require "description_cache_store"
|
|
|
|
describe DescriptionCacheStore do
|
|
subject(:cache_store) { described_class.new(database) }
|
|
|
|
let(:database) { double("database") }
|
|
let(:formula_name) { "test_name" }
|
|
let(:description) { "test_description" }
|
|
|
|
before { allow(Homebrew::EnvConfig).to receive(:eval_all?).and_return(true) }
|
|
|
|
describe "#update!" do
|
|
it "sets the formula description" do
|
|
expect(database).to receive(:set).with(formula_name, description)
|
|
cache_store.update!(formula_name, description)
|
|
end
|
|
end
|
|
|
|
describe "#delete!" do
|
|
it "deletes the formula description" do
|
|
expect(database).to receive(:delete).with(formula_name)
|
|
cache_store.delete!(formula_name)
|
|
end
|
|
end
|
|
|
|
describe "#update_from_report!" do
|
|
let(:report) { double(select_formula_or_cask: [], empty?: false) }
|
|
|
|
it "reads from the report" do
|
|
expect(database).to receive(:empty?).at_least(:once).and_return(false)
|
|
cache_store.update_from_report!(report)
|
|
end
|
|
end
|
|
|
|
describe "#update_from_formula_names!" do
|
|
it "sets the formulae descriptions" do
|
|
f = formula do
|
|
url "url-1"
|
|
desc "desc"
|
|
end
|
|
expect(Formulary).to receive(:factory).with(f.name).and_return(f)
|
|
expect(database).to receive(:empty?).and_return(false)
|
|
expect(database).to receive(:set).with(f.name, f.desc)
|
|
cache_store.update_from_formula_names!([f.name])
|
|
end
|
|
end
|
|
|
|
describe "#delete_from_formula_names!" do
|
|
it "deletes the formulae descriptions" do
|
|
expect(database).to receive(:empty?).and_return(false)
|
|
expect(database).to receive(:delete).with(formula_name)
|
|
cache_store.delete_from_formula_names!([formula_name])
|
|
end
|
|
end
|
|
|
|
describe CaskDescriptionCacheStore do
|
|
subject(:cache_store) { described_class.new(database) }
|
|
|
|
let(:database) { double("database") }
|
|
|
|
describe "#update_from_report!" do
|
|
let(:report) { double(select_formula_or_cask: [], empty?: false) }
|
|
|
|
it "reads from the report" do
|
|
expect(database).to receive(:empty?).at_least(:once).and_return(false)
|
|
cache_store.update_from_report!(report)
|
|
end
|
|
end
|
|
|
|
describe "#update_from_cask_tokens!" do
|
|
it "sets the cask descriptions" do
|
|
c = Cask::Cask.new("cask-names-desc") do
|
|
url "url-1"
|
|
name "Name 1"
|
|
name "Name 2"
|
|
desc "description"
|
|
end
|
|
expect(Cask::CaskLoader).to receive(:load).with("cask-names-desc", any_args).and_return(c)
|
|
expect(database).to receive(:empty?).and_return(false)
|
|
expect(database).to receive(:set).with(c.full_name, [c.name.join(", "), c.desc.presence])
|
|
cache_store.update_from_cask_tokens!([c.token])
|
|
end
|
|
end
|
|
end
|
|
end
|