Merge pull request #2521 from MikeMcQuaid/tests-reduce-noise

tests: reduce some noise.

Author: MikeMcQuaid

Library/Homebrew/dev-cmd/tests.rb

@@ -33,7 +33,12 @@ def tests
       ENV["HOMEBREW_DEVELOPER"] = "1"
       ENV["HOMEBREW_NO_COMPAT"] = "1" if ARGV.include? "--no-compat"
       ENV["HOMEBREW_TEST_GENERIC_OS"] = "1" if ARGV.include? "--generic"
-      ENV["HOMEBREW_NO_GITHUB_API"] = "1" unless ARGV.include? "--online"
+
+      if ARGV.include? "--online"
+        ENV["HOMEBREW_TEST_ONLINE"] = "1"
+      else
+        ENV["HOMEBREW_NO_GITHUB_API"] = "1"
+      end
 
       if ARGV.include? "--official-cmd-taps"
         ENV["HOMEBREW_TEST_OFFICIAL_CMD_TAPS"] = "1"

Library/Homebrew/test/cask/cli/style_spec.rb

@@ -81,7 +81,7 @@
     end
 
     context "version" do
-      it "matches `HOMEBREW_RUBOCOP_VERSION`" do
+      it "matches `HOMEBREW_RUBOCOP_VERSION`", :needs_network do
         stdout, status = Open3.capture2("gem", "dependency", "rubocop-cask", "--version", HOMEBREW_RUBOCOP_CASK_VERSION, "--pipe", "--remote")
 
         expect(status).to be_a_success

Library/Homebrew/test/cask/cli/uninstall_spec.rb

@@ -13,7 +13,9 @@
 
   it "tries anyway on a non-present Cask when --force is given" do
     expect {
-      Hbc::CLI::Uninstall.run("local-caffeine", "--force")
+      shutup do
+        Hbc::CLI::Uninstall.run("local-caffeine", "--force")
+      end
     }.not_to raise_error
   end
 

Library/Homebrew/test/cmd/bundle_spec.rb

@@ -1,6 +1,6 @@
 describe "brew bundle", :integration_test, :needs_test_cmd_taps do
   describe "check" do
-    it "checks if a Brewfile's dependencies are satisfied" do
+    it "checks if a Brewfile's dependencies are satisfied", :needs_network do
       setup_remote_tap "homebrew/bundle"
 
       HOMEBREW_REPOSITORY.cd do

Library/Homebrew/test/dev-cmd/pull_spec.rb

@@ -6,9 +6,7 @@
       .and be_a_failure
   end
 
-  it "fetches a patch from a GitHub commit or pull request and applies it" do
-    skip "Requires network connection." if ENV["HOMEBREW_NO_GITHUB_API"]
-
+  it "fetches a patch from a GitHub commit or pull request and applies it", :needs_network do
     CoreTap.instance.path.cd do
       shutup do
         system "git", "init"

Library/Homebrew/test/missing_formula_spec.rb

@@ -139,7 +139,7 @@
   end
 
   context "::deleted_reason" do
-    subject { described_class.deleted_reason(formula) }
+    subject { described_class.deleted_reason(formula, silent: true) }
 
     before do
       Tap.clear_cache

Library/Homebrew/test/spec_helper.rb

@@ -61,6 +61,10 @@
     skip "Python not installed." unless which("python")
   end
 
+  config.before(:each, :needs_network) do
+    skip "Requires network connection." unless ENV["HOMEBREW_TEST_ONLINE"]
+  end
+
   config.around(:each) do |example|
     begin
       TEST_DIRECTORIES.each(&:mkpath)