Add :cask to all Cask specs.

Library/Homebrew/cask/spec/cask/accessibility_spec.rb

@@ -1,8 +1,6 @@
-require "spec_helper"
-
 # TODO: this test should be named after the corresponding class, once
 #       that class is abstracted from installer.rb.
-describe "Accessibility Access" do
+describe "Accessibility Access", :cask do
   let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/with-accessibility-access.rb") }
   let(:fake_system_command) { class_double(Hbc::SystemCommand) }
   let(:installer) { Hbc::Installer.new(cask, command: fake_system_command) }

Library/Homebrew/cask/spec/cask/artifact/alt_target_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Artifact::App, :cask do
-
-describe Hbc::Artifact::App do
   describe "activate to alternate target" do
     let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/with-alt-target.rb") }
 

Library/Homebrew/cask/spec/cask/artifact/app_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Artifact::App, :cask do
-
-describe Hbc::Artifact::App do
   let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/local-caffeine.rb") }
   let(:command) { Hbc::SystemCommand }
   let(:force) { false }

Library/Homebrew/cask/spec/cask/artifact/binary_spec.rb

@@ -1,4 +1,4 @@
-describe Hbc::Artifact::Binary do
+describe Hbc::Artifact::Binary, :cask do
   let(:cask) {
     Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/with-binary.rb").tap do |cask|
       shutup do

Library/Homebrew/cask/spec/cask/artifact/generic_artifact_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Artifact::Artifact, :cask do
-
-describe Hbc::Artifact::Artifact do
   let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/with-generic-artifact.rb") }
 
   let(:install_phase) {

Library/Homebrew/cask/spec/cask/artifact/nested_container_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Artifact::NestedContainer, :cask do
-
-describe Hbc::Artifact::NestedContainer do
   describe "install" do
     it "extracts the specified paths as containers" do
       cask = Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/nested-app.rb").tap do |c|

Library/Homebrew/cask/spec/cask/artifact/pkg_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Artifact::Pkg, :cask do
-
-describe Hbc::Artifact::Pkg do
   let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/with-installable.rb") }
   let(:fake_system_command) { class_double(Hbc::SystemCommand) }
 

Library/Homebrew/cask/spec/cask/artifact/postflight_block_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Artifact::PostflightBlock, :cask do
-
-describe Hbc::Artifact::PostflightBlock do
   describe "install_phase" do
     it "calls the specified block after installing, passing a Cask mini-dsl" do
       called = false

Library/Homebrew/cask/spec/cask/artifact/preflight_block_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Artifact::PreflightBlock, :cask do
-
-describe Hbc::Artifact::PreflightBlock do
   describe "install_phase" do
     it "calls the specified block before installing, passing a Cask mini-dsl" do
       called = false

Library/Homebrew/cask/spec/cask/artifact/suite_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Artifact::Suite, :cask do
-
-describe Hbc::Artifact::Suite do
   let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/with-suite.rb") }
 
   let(:install_phase) { -> { Hbc::Artifact::Suite.new(cask).install_phase } }

Library/Homebrew/cask/spec/cask/artifact/two_apps_correct_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Artifact::App, :cask do
-
-describe Hbc::Artifact::App do
   describe "multiple apps" do
     let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/with-two-apps-correct.rb") }
 

Library/Homebrew/cask/spec/cask/artifact/two_apps_incorrect_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Artifact::App, :cask do
-
-describe Hbc::Artifact::App do
   # FIXME: Doesn't actually raise because the `app` stanza is not evaluated on load.
   # it "must raise" do
   #   lambda {

Library/Homebrew/cask/spec/cask/artifact/uninstall_no_zap_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Artifact::Zap, :cask do
-
-describe Hbc::Artifact::Zap do
   let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/with-installable.rb") }
 
   let(:zap_artifact) {

Library/Homebrew/cask/spec/cask/artifact/uninstall_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Artifact::Uninstall, :cask do
-
-describe Hbc::Artifact::Uninstall do
   let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/with-installable.rb") }
 
   let(:uninstall_artifact) {

Library/Homebrew/cask/spec/cask/artifact/zap_spec.rb

@@ -1,7 +1,5 @@
-require "spec_helper"
-
 # TODO: test that zap removes an alternate version of the same Cask
-describe Hbc::Artifact::Zap do
+describe Hbc::Artifact::Zap, :cask do
   let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/with-installable.rb") }
 
   let(:zap_artifact) {

Library/Homebrew/cask/spec/cask/audit_spec.rb

@@ -1,4 +1,4 @@
-describe Hbc::Audit do
+describe Hbc::Audit, :cask do
   def include_msg?(messages, msg)
     if msg.is_a?(Regexp)
       Array(messages).any? { |m| m =~ msg }

Library/Homebrew/cask/spec/cask/cask_spec.rb

@@ -1,4 +1,4 @@
-describe Hbc::Cask do
+describe Hbc::Cask, :cask do
   let(:cask) { described_class.new("versioned-cask") }
 
   context "when multiple versions are installed" do

Library/Homebrew/cask/spec/cask/cli/audit_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::CLI::Audit, :cask do
-
-describe Hbc::CLI::Audit do
   let(:auditor) { double }
   let(:cask) { double }
 

Library/Homebrew/cask/spec/cask/cli/cat_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::CLI::Cat, :cask do
-
-describe Hbc::CLI::Cat do
   describe "given a basic Cask" do
     let(:expected_output) {
       <<-EOS.undent

Library/Homebrew/cask/spec/cask/cli/cleanup_spec.rb

@@ -1,4 +1,4 @@
-describe Hbc::CLI::Cleanup do
+describe Hbc::CLI::Cleanup, :cask do
   let(:cache_location) { Pathname.new(Dir.mktmpdir).realpath }
   let(:cleanup_outdated) { false }
 

Library/Homebrew/cask/spec/cask/cli/create_spec.rb

@@ -1,5 +1,3 @@
-require "spec_helper"
-
 # monkeypatch for testing
 module Hbc
   class CLI
@@ -19,7 +17,7 @@ module Hbc
   end
 end
 
-describe Hbc::CLI::Create do
+describe Hbc::CLI::Create, :cask do
   before(:each) do
     Hbc::CLI::Create.reset!
   end

Library/Homebrew/cask/spec/cask/cli/edit_spec.rb

@@ -1,5 +1,3 @@
-require "spec_helper"
-
 # monkeypatch for testing
 module Hbc
   class CLI
@@ -19,7 +17,7 @@ module Hbc
   end
 end
 
-describe Hbc::CLI::Edit do
+describe Hbc::CLI::Edit, :cask do
   before(:each) do
     Hbc::CLI::Edit.reset!
   end

Library/Homebrew/cask/spec/cask/cli/fetch_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::CLI::Fetch, :cask do
-
-describe Hbc::CLI::Fetch do
   let(:local_transmission) {
     Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/local-transmission.rb")
   }

Library/Homebrew/cask/spec/cask/cli/home_spec.rb

@@ -1,5 +1,3 @@
-require "spec_helper"
-
 # monkeypatch for testing
 module Hbc
   class CLI
@@ -19,7 +17,7 @@ module Hbc
   end
 end
 
-describe Hbc::CLI::Home do
+describe Hbc::CLI::Home, :cask do
   before do
     Hbc::CLI::Home.reset!
   end

Library/Homebrew/cask/spec/cask/cli/info_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::CLI::Info, :cask do
-
-describe Hbc::CLI::Info do
   it "displays some nice info about the specified Cask" do
     expect {
       Hbc::CLI::Info.run("local-caffeine")

Library/Homebrew/cask/spec/cask/cli/install_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::CLI::Install, :cask do
-
-describe Hbc::CLI::Install do
   it "allows staging and activation of multiple Casks at once" do
     shutup do
       Hbc::CLI::Install.run("local-transmission", "local-caffeine")

Library/Homebrew/cask/spec/cask/cli/list_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::CLI::List, :cask do
-
-describe Hbc::CLI::List do
   it "lists the installed Casks in a pretty fashion" do
     casks = %w[local-caffeine local-transmission].map { |c| Hbc.load(c) }
 

Library/Homebrew/cask/spec/cask/cli/options_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::CLI, :cask do
-
-describe Hbc::CLI do
   it "supports setting the appdir" do
     Hbc::CLI.process_options %w[help --appdir=/some/path/foo]
 

Library/Homebrew/cask/spec/cask/cli/reinstall_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::CLI::Reinstall, :cask do
-
-describe Hbc::CLI::Reinstall do
   it "allows reinstalling a Cask" do
     shutup do
       Hbc::CLI::Install.run("local-transmission")

Library/Homebrew/cask/spec/cask/cli/search_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::CLI::Search, :cask do
-
-describe Hbc::CLI::Search do
   it "lists the available Casks that match the search term" do
     expect {
       Hbc::CLI::Search.run("local")

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

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::CLI::Uninstall, :cask do
-
-describe Hbc::CLI::Uninstall do
   it "shows an error when a bad Cask is provided" do
     expect {
       Hbc::CLI::Uninstall.run("notacask")

Library/Homebrew/cask/spec/cask/cli/version_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe "brew cask --version", :cask do
-
-describe "brew cask --version" do
   it "respects the --version argument" do
     expect {
       expect {

Library/Homebrew/cask/spec/cask/cli/zap_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::CLI::Zap, :cask do
-
-describe Hbc::CLI::Zap do
   it "shows an error when a bad Cask is provided" do
     expect {
       Hbc::CLI::Zap.run("notacask")

Library/Homebrew/cask/spec/cask/cli_spec.rb

@@ -1,4 +1,4 @@
-describe Hbc::CLI do
+describe Hbc::CLI, :cask do
   it "lists the taps for Casks that show up in two taps" do
     listing = Hbc::CLI.nice_listing(%w[
                                       caskroom/cask/adium

Library/Homebrew/cask/spec/cask/container/dmg_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Container::Dmg, :cask do
-
-describe Hbc::Container::Dmg do
   describe "#mount!" do
     it "does not store nil mounts for dmgs with extra data" do
       transmission = Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/local-transmission.rb")

Library/Homebrew/cask/spec/cask/container/naked_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Container::Naked, :cask do
-
-describe Hbc::Container::Naked do
   it "saves files with spaces in them from uris with encoded spaces" do
     cask = Hbc::Cask.new("spacey") do
       url "http://example.com/kevin%20spacey.pkg"

Library/Homebrew/cask/spec/cask/depends_on_spec.rb

@@ -1,8 +1,6 @@
-require "spec_helper"
-
 # TODO: this test should be named after the corresponding class, once
 #       that class is abstracted from installer.rb
-describe "Satisfy Dependencies and Requirements" do
+describe "Satisfy Dependencies and Requirements", :cask do
   subject {
     lambda do
       shutup do

Library/Homebrew/cask/spec/cask/download_strategy_spec.rb

@@ -1,4 +1,4 @@
-describe "download strategies" do
+describe "download strategies", :cask do
   let(:url) { "http://example.com/cask.dmg" }
   let(:url_options) { Hash.new }
   let(:cask) {

Library/Homebrew/cask/spec/cask/dsl/caveats_spec.rb

@@ -1,7 +1,6 @@
-require "spec_helper"
 require "test/support/helper/spec/shared_examples/hbc_dsl_base"
 
-describe Hbc::DSL::Caveats do
+describe Hbc::DSL::Caveats, :cask do
   let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/basic-cask.rb") }
   let(:dsl) { Hbc::DSL::Caveats.new(cask) }
 

Library/Homebrew/cask/spec/cask/dsl/postflight_spec.rb

@@ -1,8 +1,7 @@
-require "spec_helper"
 require "test/support/helper/spec/shared_examples/hbc_dsl_base"
 require "test/support/helper/spec/shared_examples/hbc_staged"
 
-describe Hbc::DSL::Postflight do
+describe Hbc::DSL::Postflight, :cask do
   let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/basic-cask.rb") }
   let(:dsl) { Hbc::DSL::Postflight.new(cask, Hbc::FakeSystemCommand) }
 

Library/Homebrew/cask/spec/cask/dsl/preflight_spec.rb

@@ -1,8 +1,7 @@
-require "spec_helper"
 require "test/support/helper/spec/shared_examples/hbc_dsl_base"
 require "test/support/helper/spec/shared_examples/hbc_staged"
 
-describe Hbc::DSL::Preflight do
+describe Hbc::DSL::Preflight, :cask do
   let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/basic-cask.rb") }
   let(:dsl) { Hbc::DSL::Preflight.new(cask, Hbc::FakeSystemCommand) }
 

Library/Homebrew/cask/spec/cask/dsl/uninstall_postflight_spec.rb

@@ -1,7 +1,6 @@
-require "spec_helper"
 require "test/support/helper/spec/shared_examples/hbc_dsl_base"
 
-describe Hbc::DSL::UninstallPostflight do
+describe Hbc::DSL::UninstallPostflight, :cask do
   let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/basic-cask.rb") }
   let(:dsl) { Hbc::DSL::UninstallPostflight.new(cask, Hbc::FakeSystemCommand) }
 

Library/Homebrew/cask/spec/cask/dsl/uninstall_preflight_spec.rb

@@ -1,8 +1,7 @@
-require "spec_helper"
 require "test/support/helper/spec/shared_examples/hbc_dsl_base"
 require "test/support/helper/spec/shared_examples/hbc_staged"
 
-describe Hbc::DSL::UninstallPreflight do
+describe Hbc::DSL::UninstallPreflight, :cask do
   let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/basic-cask.rb") }
   let(:dsl) { Hbc::DSL::UninstallPreflight.new(cask, Hbc::FakeSystemCommand) }
 

Library/Homebrew/cask/spec/cask/dsl_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::DSL, :cask do
-
-describe Hbc::DSL do
   let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/#{token}.rb") }
   let(:token) { "basic-cask" }
 
@@ -76,7 +74,7 @@ describe Hbc::DSL do
       end
     end
 
-    context "when it contains a deprecated DSL version" do
+    context "when it contains a deprecated DSL version", :needs_compat do
       let(:token) { "with-dsl-version" }
 
       it "may use deprecated DSL version hash syntax" do

Library/Homebrew/cask/spec/cask/installer_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Installer, :cask do
-
-describe Hbc::Installer do
   describe "install" do
     let(:empty_depends_on_stub) {
       double(formula: [], cask: [], macos: nil, arch: nil, x11: nil)

Library/Homebrew/cask/spec/cask/pkg_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::Pkg, :cask do
-
-describe Hbc::Pkg do
   describe "uninstall" do
     let(:fake_system_command) { Hbc::NeverSudoSystemCommand }
     let(:empty_response) { double(stdout: "") }

Library/Homebrew/cask/spec/cask/scopes_spec.rb

@@ -1,4 +1,4 @@
-describe Hbc::Scopes do
+describe Hbc::Scopes, :cask do
   describe "installed" do
     it "returns a list installed Casks by loading Casks for all the dirs that exist in the caskroom" do
       allow(Hbc).to receive(:load) { |token| "loaded-#{token}" }

Library/Homebrew/cask/spec/cask/staged_spec.rb

@@ -1,9 +1,7 @@
-require "spec_helper"
-
 # TODO: this test should be named after the corresponding class, once
 #       that class is abstracted from installer.rb.  It makes little sense
 #       to be invoking bundle_identifier off of the installer instance.
-describe "Operations on staged Casks" do
+describe "Operations on staged Casks", :cask do
   describe "bundle ID" do
     let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/local-transmission.rb") }
     let(:installer) { Hbc::Installer.new(cask) }

Library/Homebrew/cask/spec/cask/system_command_result_spec.rb

@@ -1,6 +1,6 @@
 require "hbc/system_command"
 
-describe Hbc::SystemCommand::Result do
+describe Hbc::SystemCommand::Result, :cask do
   describe "::_parse_plist" do
     subject { described_class._parse_plist(command, input) }
     let(:command) { Hbc::SystemCommand.new("/usr/bin/true", {}) }

Library/Homebrew/cask/spec/cask/system_command_spec.rb

@@ -1,4 +1,4 @@
-describe Hbc::SystemCommand do
+describe Hbc::SystemCommand, :cask do
   describe "when the exit code is 0" do
     describe "its result" do
       subject { described_class.run("/usr/bin/true") }

Library/Homebrew/cask/spec/cask/underscore_supporting_uri_spec.rb

@@ -1,4 +1,4 @@
-describe Hbc::UnderscoreSupportingURI do
+describe Hbc::UnderscoreSupportingURI, :cask do
   describe "parse" do
     it "works like normal on normal URLs" do
       uri = Hbc::UnderscoreSupportingURI.parse("http://example.com/TestCask.dmg")

Library/Homebrew/cask/spec/cask/url_checker_spec.rb

@@ -1,6 +1,4 @@
-require "spec_helper"
+describe Hbc::UrlChecker, :cask do
-
-describe Hbc::UrlChecker do
   describe "request processing" do
     let(:cask) { Hbc::CaskLoader.load_from_file(TEST_FIXTURE_DIR/"cask/Casks/basic-cask.rb") }
     let(:checker) { Hbc::UrlChecker.new(cask) }

Library/Homebrew/cask/spec/cask/verify_spec.rb

@@ -1,4 +1,4 @@
-describe Hbc::Verify do
+describe Hbc::Verify, :cask do
   let(:cask) { double("cask") }
 
   let(:verification_classes) {