Prevent unexpected network calls in tests

Library/Homebrew/cli/named_args.rb

@@ -256,11 +256,15 @@ def to_paths(only: parent&.only_formula_or_cask, recurse_tap: false)
  paths = []
 
  if formula_path.exist? ||
- (!CoreTap.instance.installed? && Homebrew::API::Formula.all_formulae.key?(path.basename.to_s))
+ (!Homebrew::EnvConfig.no_install_from_api? &&
+ !CoreTap.instance.installed? &&
+ Homebrew::API::Formula.all_formulae.key?(path.basename.to_s))
  paths << formula_path
  end
  if cask_path.exist? ||
- (!CoreCaskTap.instance.installed? && Homebrew::API::Cask.all_casks.key?(path.basename.to_s))
+ (!Homebrew::EnvConfig.no_install_from_api? &&
+ !CoreCaskTap.instance.installed? &&
+ Homebrew::API::Cask.all_casks.key?(path.basename.to_s))
  paths << cask_path
  end
 

Library/Homebrew/test/cask/cask_spec.rb

@@ -46,13 +46,15 @@
  expect(c.token).to eq("caffeine")
  end
 
- it "returns an instance of the Cask from a URL" do
+ it "returns an instance of the Cask from a URL", :needs_utils_curl do
+ expect(Homebrew::API).not_to receive(:fetch_json_api_file)
  c = Cask::CaskLoader.load("file://#{tap_path}/Casks/local-caffeine.rb")
  expect(c).to be_a(described_class)
  expect(c.token).to eq("local-caffeine")
  end
 
- it "raises an error when failing to download a Cask from a URL" do
+ it "raises an error when failing to download a Cask from a URL", :needs_utils_curl do
+ expect(Homebrew::API).not_to receive(:fetch_json_api_file)
  expect do
  Cask::CaskLoader.load("file://#{tap_path}/Casks/notacask.rb")
  end.to raise_error(Cask::CaskUnavailableError)

Library/Homebrew/test/cask/info_spec.rb

@@ -3,6 +3,11 @@
 require "utils"
 
 RSpec.describe Cask::Info, :cask do
+ before do
+ # Prevent unnecessary network requests in `Utils::Analytics.cask_output`
+ ENV["HOMEBREW_NO_ANALYTICS"] = "1"
+ end
+
  it "displays some nice info about the specified Cask" do
  expect do
  described_class.info(Cask::CaskLoader.load("local-transmission"))

Library/Homebrew/test/formulary_spec.rb

@@ -119,7 +119,8 @@ class Wrong#{described_class.class_s(formula_name)} < Formula
  expect(described_class.factory(formula_path)).to be_a(Formula)
  end
 
- it "returns a Formula when given a URL" do
+ it "returns a Formula when given a URL", :needs_utils_curl do
+ expect(Homebrew::API).not_to receive(:fetch_json_api_file)
  formula = described_class.factory("file://#{formula_path}")
  expect(formula).to be_a(Formula)
  end
@@ -383,6 +384,10 @@ def formula_json_contents(extra_items = {})
  before do
  ENV.delete("HOMEBREW_NO_INSTALL_FROM_API")
 
+ # avoid unnecessary network calls
+ allow(Homebrew::API::Formula).to receive_messages(all_aliases: {}, all_renames: {})
+ allow(CoreTap.instance).to receive(:tap_migrations).and_return({})
+
  # don't try to load/fetch gcc/glibc
  allow(DevelopmentTools).to receive_messages(needs_libc_formula?: false, needs_compiler_formula?: false)
  end

Library/Homebrew/test/spec_helper.rb

@@ -165,6 +165,15 @@
  skip "Requires network connection." unless ENV["HOMEBREW_TEST_ONLINE"]
  end
 
+ config.before do |example|
+ next if example.metadata.key?(:needs_network)
+ next if example.metadata.key?(:needs_utils_curl)
+
+ allow(Utils::Curl).to receive(:curl_executable).and_raise(<<~ERROR)
+ Unexpected call to Utils::Curl.curl_executable without setting :needs_network or :needs_utils_curl.
+ ERROR
+ end
+
  config.before(:each, :needs_svn) do
  svn_shim = HOMEBREW_SHIMS_PATH/"shared/svn"
  skip "Subversion is not installed." unless quiet_system svn_shim, "--version"

Library/Homebrew/test/tap_spec.rb

@@ -192,7 +192,7 @@ def setup_completion(link:)
  end
 
  describe "#remote" do
- it "returns the remote URL" do
+ it "returns the remote URL", :needs_network do
  setup_git_repo
 
  expect(homebrew_foo_tap.remote).to eq("https://github.com/Homebrew/homebrew-foo")