# frozen_string_literal: true require "dev-cmd/audit" require "formulary" require "cmd/shared_examples/args_parse" describe "Homebrew.audit_args" do it_behaves_like "parseable arguments" end module Count def self.increment @count ||= 0 @count += 1 end end module Homebrew describe FormulaText do alias_matcher :have_data, :be_data alias_matcher :have_end, :be_end alias_matcher :have_trailing_newline, :be_trailing_newline let(:dir) { mktmpdir } def formula_text(name, body = nil, options = {}) path = dir/"#{name}.rb" path.write <<~RUBY class #{Formulary.class_s(name)} < Formula #{body} end #{options[:patch]} RUBY described_class.new(path) end specify "simple valid Formula" do ft = formula_text "valid", <<~RUBY url "https://www.brew.sh/valid-1.0.tar.gz" RUBY expect(ft).to have_trailing_newline expect(ft =~ /\burl\b/).to be_truthy expect(ft.line_number(/desc/)).to be nil expect(ft.line_number(/\burl\b/)).to eq(2) expect(ft).to include("Valid") end specify "#trailing_newline?" do ft = formula_text "newline" expect(ft).to have_trailing_newline end end describe FormulaAuditor do def formula_auditor(name, text, options = {}) path = Pathname.new "#{dir}/#{name}.rb" path.open("w") do |f| f.write text end described_class.new(Formulary.factory(path), options) end let(:dir) { mktmpdir } describe "#problems" do it "is empty by default" do fa = formula_auditor "foo", <<~RUBY class Foo < Formula url "https://brew.sh/foo-1.0.tgz" end RUBY expect(fa.problems).to be_empty end end describe "#audit_license" do let(:spdx_data) { JSON.parse Pathname(File.join(File.dirname(__FILE__), "../../data/spdx.json")).read } let(:custom_spdx_id) { "zzz" } let(:standard_mismatch_spdx_id) { "0BSD" } let(:license_array) { ["0BSD", "GPL-3.0"] } let(:license_array_mismatch) { ["0BSD", "MIT"] } let(:license_array_nonstandard) { ["0BSD", "zzz", "MIT"] } it "does not check if the formula is not a new formula" do fa = formula_auditor "foo", <<~RUBY, spdx_data: spdx_data, new_formula: false class Foo < Formula url "https://brew.sh/foo-1.0.tgz" end RUBY fa.audit_license expect(fa.problems).to be_empty end it "detects no license info" do fa = formula_auditor "foo", <<~RUBY, spdx_data: spdx_data, new_formula: true class Foo < Formula url "https://brew.sh/foo-1.0.tgz" end RUBY fa.audit_license expect(fa.problems.first).to match "No license specified for package." end it "detects if license is not a standard spdx-id" do fa = formula_auditor "foo", <<~RUBY, spdx_data: spdx_data, new_formula: true class Foo < Formula url "https://brew.sh/foo-1.0.tgz" license "#{custom_spdx_id}" end RUBY fa.audit_license expect(fa.problems.first).to match "Formula foo contains non-standard SPDX licenses: [\"zzz\"]." end it "detects if license array contains a non-standard spdx-id" do fa = formula_auditor "foo", <<~RUBY, spdx_data: spdx_data, new_formula: true class Foo < Formula url "https://brew.sh/foo-1.0.tgz" license #{license_array_nonstandard} end RUBY fa.audit_license expect(fa.problems.first).to match "Formula foo contains non-standard SPDX licenses: [\"zzz\"]." end it "verifies that a license info is a standard spdx id" do fa = formula_auditor "foo", <<~RUBY, spdx_data: spdx_data, new_formula: true class Foo < Formula url "https://brew.sh/foo-1.0.tgz" license "0BSD" end RUBY fa.audit_license expect(fa.problems).to be_empty end it "verifies that a license array contains only standard spdx id" do fa = formula_auditor "foo", <<~RUBY, spdx_data: spdx_data, new_formula: true class Foo < Formula url "https://brew.sh/foo-1.0.tgz" license #{license_array} end RUBY fa.audit_license expect(fa.problems).to be_empty end it "checks online and verifies that a standard license id is the same "\ "as what is indicated on its Github repo" do fa = formula_auditor "cask", <<~RUBY, spdx_data: spdx_data, online: true, core_tap: true, new_formula: true class Cask < Formula url "https://github.com/cask/cask/archive/v0.8.4.tar.gz" head "https://github.com/cask/cask.git" license "GPL-3.0" end RUBY fa.audit_license expect(fa.problems).to be_empty end it "checks online and detects that a formula-specified license is not "\ "the same as what is indicated on its Github repository" do fa = formula_auditor "cask", <<~RUBY, online: true, spdx_data: spdx_data, core_tap: true, new_formula: true class Cask < Formula url "https://github.com/cask/cask/archive/v0.8.4.tar.gz" head "https://github.com/cask/cask.git" license "#{standard_mismatch_spdx_id}" end RUBY fa.audit_license expect(fa.problems.first).to match "License mismatch - GitHub license is: [\"GPL-3.0\"], "\ "but Formulae license states: #{Array(standard_mismatch_spdx_id)}." end it "checks online and detects that an array of license does not contain "\ "what is indicated on its Github repository" do fa = formula_auditor "cask", <<~RUBY, online: true, spdx_data: spdx_data, core_tap: true, new_formula: true class Cask < Formula url "https://github.com/cask/cask/archive/v0.8.4.tar.gz" head "https://github.com/cask/cask.git" license #{license_array_mismatch} end RUBY fa.audit_license expect(fa.problems.first).to match "License mismatch - GitHub license is: [\"GPL-3.0\"], "\ "but Formulae license states: #{Array(license_array_mismatch)}." end it "checks online and verifies that an array of license contains "\ "what is indicated on its Github repository" do fa = formula_auditor "cask", <<~RUBY, online: true, spdx_data: spdx_data, core_tap: true, new_formula: true class Cask < Formula url "https://github.com/cask/cask/archive/v0.8.4.tar.gz" head "https://github.com/cask/cask.git" license #{license_array} end RUBY fa.audit_license expect(fa.problems).to be_empty end end describe "#audit_file" do specify "no issue" do fa = formula_auditor "foo", <<~RUBY class Foo < Formula url "https://brew.sh/foo-1.0.tgz" homepage "https://brew.sh" end RUBY fa.audit_file expect(fa.problems).to eq([]) end end describe "#audit_github_repository" do specify "#audit_github_repository when HOMEBREW_NO_GITHUB_API is set" do ENV["HOMEBREW_NO_GITHUB_API"] = "1" fa = formula_auditor "foo", <<~RUBY, strict: true, online: true class Foo < Formula homepage "https://github.com/example/example" url "https://brew.sh/foo-1.0.tgz" end RUBY fa.audit_github_repository expect(fa.problems).to eq([]) end end describe "#audit_github_repository_archived" do specify "#audit_github_repository_archived when HOMEBREW_NO_GITHUB_API is set" do fa = formula_auditor "foo", <<~RUBY, strict: true, online: true class Foo < Formula homepage "https://github.com/example/example" url "https://brew.sh/foo-1.0.tgz" end RUBY fa.audit_github_repository_archived expect(fa.problems).to eq([]) end end describe "#audit_gitlab_repository" do specify "#audit_gitlab_repository for stars, forks and creation date" do fa = formula_auditor "foo", <<~RUBY, strict: true, online: true class Foo < Formula homepage "https://gitlab.com/libtiff/libtiff" url "https://brew.sh/foo-1.0.tgz" end RUBY fa.audit_gitlab_repository expect(fa.problems).to eq([]) end end describe "#audit_gitlab_repository_archived" do specify "#audit gitlab repository for archived status" do fa = formula_auditor "foo", <<~RUBY, strict: true, online: true class Foo < Formula homepage "https://gitlab.com/libtiff/libtiff" url "https://brew.sh/foo-1.0.tgz" end RUBY fa.audit_gitlab_repository_archived expect(fa.problems).to eq([]) end end describe "#audit_bitbucket_repository" do specify "#audit_bitbucket_repository for stars, forks and creation date" do fa = formula_auditor "foo", <<~RUBY, strict: true, online: true class Foo < Formula homepage "https://bitbucket.com/libtiff/libtiff" url "https://brew.sh/foo-1.0.tgz" end RUBY fa.audit_bitbucket_repository expect(fa.problems).to eq([]) end end describe "#audit_deps" do describe "a dependency on a macOS-provided keg-only formula" do describe "which is allowlisted" do subject { fa } let(:fa) do formula_auditor "foo", <<~RUBY, new_formula: true class Foo < Formula url "https://brew.sh/foo-1.0.tgz" homepage "https://brew.sh" depends_on "openssl" end RUBY end let(:f_openssl) do formula do url "https://brew.sh/openssl-1.0.tgz" homepage "https://brew.sh" keg_only :provided_by_macos end end before do allow(fa.formula.deps.first) .to receive(:to_formula).and_return(f_openssl) fa.audit_deps end its(:problems) { are_expected.to be_empty } end describe "which is not allowlisted", :needs_macos do subject { fa } let(:fa) do formula_auditor "foo", <<~RUBY, new_formula: true, core_tap: true class Foo < Formula url "https://brew.sh/foo-1.0.tgz" homepage "https://brew.sh" depends_on "bc" end RUBY end let(:f_bc) do formula do url "https://brew.sh/bc-1.0.tgz" homepage "https://brew.sh" keg_only :provided_by_macos end end before do allow(fa.formula.deps.first) .to receive(:to_formula).and_return(f_bc) fa.audit_deps end its(:new_formula_problems) { are_expected.to match([/is provided by macOS/]) } end end end describe "#audit_revision_and_version_scheme" do subject { fa = described_class.new(Formulary.factory(formula_path), git: true) fa.audit_revision_and_version_scheme fa.problems.first } let(:origin_tap_path) { Tap::TAP_DIRECTORY/"homebrew/homebrew-foo" } let(:foo_version) { Count.increment } let(:formula_subpath) { "Formula/foo#{foo_version}.rb" } let(:origin_formula_path) { origin_tap_path/formula_subpath } let(:tap_path) { Tap::TAP_DIRECTORY/"homebrew/homebrew-bar" } let(:formula_path) { tap_path/formula_subpath } before do origin_formula_path.write <<~RUBY class Foo#{foo_version} < Formula url "https://brew.sh/foo-1.0.tar.gz" sha256 "31cccfc6630528db1c8e3a06f6decf2a370060b982841cfab2b8677400a5092e" revision 2 version_scheme 1 end RUBY origin_tap_path.mkpath origin_tap_path.cd do system "git", "init" system "git", "add", "--all" system "git", "commit", "-m", "init" end tap_path.mkpath tap_path.cd do system "git", "clone", origin_tap_path, "." end end def formula_gsub(before, after = "") text = formula_path.read text.gsub! before, after formula_path.unlink formula_path.write text end def formula_gsub_origin_commit(before, after = "") text = origin_formula_path.read text.gsub!(before, after) origin_formula_path.unlink origin_formula_path.write text origin_tap_path.cd do system "git", "commit", "-am", "commit" end tap_path.cd do system "git", "fetch" system "git", "reset", "--hard", "origin/master" end end context "checksums" do context "should not change with the same version" do before do formula_gsub( 'sha256 "31cccfc6630528db1c8e3a06f6decf2a370060b982841cfab2b8677400a5092e"', 'sha256 "3622d2a53236ed9ca62de0616a7e80fd477a9a3f862ba09d503da188f53ca523"', ) end it { is_expected.to match("stable sha256 changed without the version also changing") } end context "can change with the different version" do before do formula_gsub_origin_commit( 'sha256 "31cccfc6630528db1c8e3a06f6decf2a370060b982841cfab2b8677400a5092e"', 'sha256 "3622d2a53236ed9ca62de0616a7e80fd477a9a3f862ba09d503da188f53ca523"', ) formula_gsub "foo-1.0.tar.gz", "foo-1.1.tar.gz" formula_gsub_origin_commit( 'sha256 "3622d2a53236ed9ca62de0616a7e80fd477a9a3f862ba09d503da188f53ca523"', 'sha256 "e048c5e6144f5932d8672c2fade81d9073d5b3ca1517b84df006de3d25414fc1"', ) end it { is_expected.to be_nil } end end context "revisions" do context "should not be removed when first committed above 0" do it { is_expected.to be_nil } end context "should not decrease with the same version" do before { formula_gsub_origin_commit "revision 2", "revision 1" } it { is_expected.to match("revision should not decrease (from 2 to 1)") } end context "should not be removed with the same version" do before { formula_gsub_origin_commit "revision 2" } it { is_expected.to match("revision should not decrease (from 2 to 0)") } end context "should not decrease with the same, uncommitted version" do before { formula_gsub "revision 2", "revision 1" } it { is_expected.to match("revision should not decrease (from 2 to 1)") } end context "should be removed with a newer version" do before { formula_gsub_origin_commit "foo-1.0.tar.gz", "foo-1.1.tar.gz" } it { is_expected.to match("'revision 2' should be removed") } end context "should not warn on an newer version revision removal" do before do formula_gsub_origin_commit "revision 2", "" formula_gsub_origin_commit "foo-1.0.tar.gz", "foo-1.1.tar.gz" end it { is_expected.to be_nil } end context "should only increment by 1 with an uncommitted version" do before do formula_gsub "foo-1.0.tar.gz", "foo-1.1.tar.gz" formula_gsub "revision 2", "revision 4" end it { is_expected.to match("revisions should only increment by 1") } end context "should not warn on past increment by more than 1" do before do formula_gsub_origin_commit "revision 2", "# no revision" formula_gsub_origin_commit "foo-1.0.tar.gz", "foo-1.1.tar.gz" formula_gsub_origin_commit "# no revision", "revision 3" end it { is_expected.to be_nil } end end context "version_schemes" do context "should not decrease with the same version" do before { formula_gsub_origin_commit "version_scheme 1" } it { is_expected.to match("version_scheme should not decrease (from 1 to 0)") } end context "should not decrease with a new version" do before do formula_gsub_origin_commit "foo-1.0.tar.gz", "foo-1.1.tar.gz" formula_gsub_origin_commit "version_scheme 1", "" formula_gsub_origin_commit "revision 2", "" end it { is_expected.to match("version_scheme should not decrease (from 1 to 0)") } end context "should only increment by 1" do before do formula_gsub_origin_commit "version_scheme 1", "# no version_scheme" formula_gsub_origin_commit "foo-1.0.tar.gz", "foo-1.1.tar.gz" formula_gsub_origin_commit "revision 2", "" formula_gsub_origin_commit "# no version_scheme", "version_scheme 3" end it { is_expected.to match("version_schemes should only increment by 1") } end end context "versions" do context "uncommitted should not decrease" do before { formula_gsub "foo-1.0.tar.gz", "foo-0.9.tar.gz" } it { is_expected.to match("stable version should not decrease (from 1.0 to 0.9)") } end context "committed can decrease" do before do formula_gsub_origin_commit "revision 2" formula_gsub_origin_commit "foo-1.0.tar.gz", "foo-0.9.tar.gz" end it { is_expected.to be_nil } end context "can decrease with version_scheme increased" do before do formula_gsub "revision 2" formula_gsub "foo-1.0.tar.gz", "foo-0.9.tar.gz" formula_gsub "version_scheme 1", "version_scheme 2" end it { is_expected.to be_nil } end end end describe "#audit_versioned_keg_only" do specify "it warns when a versioned formula is not `keg_only`" do fa = formula_auditor "foo@1.1", <<~RUBY, core_tap: true class FooAT11 < Formula url "https://brew.sh/foo-1.1.tgz" end RUBY fa.audit_versioned_keg_only expect(fa.problems.first) .to match("Versioned formulae in homebrew/core should use `keg_only :versioned_formula`") end specify "it warns when a versioned formula has an incorrect `keg_only` reason" do fa = formula_auditor "foo@1.1", <<~RUBY, core_tap: true class FooAT11 < Formula url "https://brew.sh/foo-1.1.tgz" keg_only :provided_by_macos end RUBY fa.audit_versioned_keg_only expect(fa.problems.first) .to match("Versioned formulae in homebrew/core should use `keg_only :versioned_formula`") end specify "it does not warn when a versioned formula has `keg_only :versioned_formula`" do fa = formula_auditor "foo@1.1", <<~RUBY, core_tap: true class FooAT11 < Formula url "https://brew.sh/foo-1.1.tgz" keg_only :versioned_formula end RUBY fa.audit_versioned_keg_only expect(fa.problems).to eq([]) end end include_examples "formulae exist", described_class::VERSIONED_KEG_ONLY_ALLOWLIST include_examples "formulae exist", described_class::VERSIONED_HEAD_SPEC_ALLOWLIST include_examples "formulae exist", described_class::PROVIDED_BY_MACOS_DEPENDS_ON_ALLOWLIST include_examples "formulae exist", described_class::THROTTLED_FORMULAE.keys include_examples "formulae exist", described_class::UNSTABLE_ALLOWLIST.keys include_examples "formulae exist", described_class::GNOME_DEVEL_ALLOWLIST.keys end end