diff --git a/Library/Homebrew/cask/cmd.rb b/Library/Homebrew/cask/cmd.rb index 218a35e4f5..acb6de20e7 100644 --- a/Library/Homebrew/cask/cmd.rb +++ b/Library/Homebrew/cask/cmd.rb @@ -14,7 +14,6 @@ require "cask/cmd/audit" require "cask/cmd/fetch" require "cask/cmd/install" require "cask/cmd/reinstall" -require "cask/cmd/uninstall" module Cask # Implementation of the `brew cask` command-line interface. diff --git a/Library/Homebrew/cask/cmd/uninstall.rb b/Library/Homebrew/cask/cmd/uninstall.rb deleted file mode 100644 index 1700482a65..0000000000 --- a/Library/Homebrew/cask/cmd/uninstall.rb +++ /dev/null @@ -1,56 +0,0 @@ -# typed: false -# frozen_string_literal: true - -module Cask - class Cmd - # Cask implementation of the `brew uninstall` command. - # - # @api private - class Uninstall < AbstractCommand - extend T::Sig - - def self.parser - super do - switch "--force", - description: "Uninstall even if the is not installed, overwrite " \ - "existing files and ignore errors when removing files." - end - end - - sig { void } - def run - self.class.uninstall_casks( - *casks, - binaries: args.binaries?, - verbose: args.verbose?, - force: args.force?, - ) - end - - def self.uninstall_casks(*casks, binaries: nil, force: false, verbose: false) - require "cask/installer" - - options = { - binaries: binaries, - force: force, - verbose: verbose, - }.compact - - casks.each do |cask| - odebug "Uninstalling Cask #{cask}" - - raise CaskNotInstalledError, cask if !cask.installed? && !force - - Installer.new(cask, **options).uninstall - - next if (versions = cask.versions).empty? - - puts <<~EOS - #{cask} #{versions.to_sentence} #{(versions.count == 1) ? "is" : "are"} still installed. - Remove #{(versions.count == 1) ? "it" : "them all"} with `brew uninstall --cask --force #{cask}`. - EOS - end - end - end - end -end diff --git a/Library/Homebrew/cask/uninstall.rb b/Library/Homebrew/cask/uninstall.rb new file mode 100644 index 0000000000..71b3684a00 --- /dev/null +++ b/Library/Homebrew/cask/uninstall.rb @@ -0,0 +1,26 @@ +# typed: false +# frozen_string_literal: true + +module Cask + # @api private + class Uninstall + def self.uninstall_casks(*casks, binaries: nil, force: false, verbose: false) + require "cask/installer" + + casks.each do |cask| + odebug "Uninstalling Cask #{cask}" + + raise CaskNotInstalledError, cask if !cask.installed? && !force + + Installer.new(cask, binaries: binaries, force: force, verbose: verbose).uninstall + + next if (versions = cask.versions).empty? + + puts <<~EOS + #{cask} #{versions.to_sentence} #{(versions.count == 1) ? "is" : "are"} still installed. + Remove #{(versions.count == 1) ? "it" : "them all"} with `brew uninstall --cask --force #{cask}`. + EOS + end + end + end +end diff --git a/Library/Homebrew/cmd/uninstall.rb b/Library/Homebrew/cmd/uninstall.rb index 9465360839..e0db1db786 100644 --- a/Library/Homebrew/cmd/uninstall.rb +++ b/Library/Homebrew/cmd/uninstall.rb @@ -6,10 +6,10 @@ require "formula" require "diagnostic" require "migrator" require "cli/parser" -require "cask/cmd" require "cask/cask_loader" require "cask/exceptions" require "cask/installer" +require "cask/uninstall" require "uninstall" module Homebrew @@ -76,7 +76,7 @@ module Homebrew Cask::Installer.new(cask, verbose: args.verbose?, force: args.force?).zap end else - T.unsafe(Cask::Cmd::Uninstall).uninstall_casks( + Cask::Uninstall.uninstall_casks( *casks, verbose: args.verbose?, force: args.force?, diff --git a/Library/Homebrew/test/cask/cmd/uninstall_spec.rb b/Library/Homebrew/test/cask/uninstall_spec.rb similarity index 83% rename from Library/Homebrew/test/cask/cmd/uninstall_spec.rb rename to Library/Homebrew/test/cask/uninstall_spec.rb index 5620e112e5..b7f712c4fd 100644 --- a/Library/Homebrew/test/cask/cmd/uninstall_spec.rb +++ b/Library/Homebrew/test/cask/uninstall_spec.rb @@ -1,7 +1,9 @@ # typed: false # frozen_string_literal: true -describe Cask::Cmd::Uninstall, :cask do +require "cask/uninstall" + +describe Cask::Uninstall, :cask do it "displays the uninstallation progress" do caffeine = Cask::CaskLoader.load(cask_path("local-caffeine")) @@ -15,23 +17,22 @@ describe Cask::Cmd::Uninstall, :cask do EOS expect do - described_class.run("local-caffeine") + described_class.uninstall_casks(caffeine) end.to output(output).to_stdout end - it "shows an error when a bad Cask is provided" do - expect { described_class.run("notacask") } - .to raise_error(Cask::CaskUnavailableError, /is unavailable/) - end - it "shows an error when a Cask is provided that's not installed" do - expect { described_class.run("local-caffeine") } + caffeine = Cask::CaskLoader.load(cask_path("local-caffeine")) + + expect { described_class.uninstall_casks(caffeine) } .to raise_error(Cask::CaskNotInstalledError, /is not installed/) end it "tries anyway on a non-present Cask when --force is given" do + caffeine = Cask::CaskLoader.load(cask_path("local-caffeine")) + expect do - described_class.run("local-caffeine", "--force") + described_class.uninstall_casks(caffeine, force: true) end.not_to raise_error end @@ -45,7 +46,7 @@ describe Cask::Cmd::Uninstall, :cask do expect(caffeine).to be_installed expect(transmission).to be_installed - described_class.run("local-caffeine", "local-transmission") + described_class.uninstall_casks(caffeine, transmission) expect(caffeine).not_to be_installed expect(caffeine.config.appdir.join("Transmission.app")).not_to exist @@ -62,13 +63,13 @@ describe Cask::Cmd::Uninstall, :cask do cask.config.appdir.join("MyFancyApp.app").rmtree - expect { described_class.run("with-uninstall-script-app") } + expect { described_class.uninstall_casks(cask) } .to raise_error(Cask::CaskError, /uninstall script .* does not exist/) expect(cask).to be_installed expect do - described_class.run("with-uninstall-script-app", "--force") + described_class.uninstall_casks(cask, force: true) end.not_to raise_error expect(cask).not_to be_installed @@ -101,13 +102,13 @@ describe Cask::Cmd::Uninstall, :cask do end it "uninstalls one version at a time" do - described_class.run("versioned-cask") + described_class.uninstall_casks(Cask::Cask.new("versioned-cask")) expect(caskroom_path.join(first_installed_version)).to exist expect(caskroom_path.join(last_installed_version)).not_to exist expect(caskroom_path).to exist - described_class.run("versioned-cask") + described_class.uninstall_casks(Cask::Cask.new("versioned-cask")) expect(caskroom_path.join(first_installed_version)).not_to exist expect(caskroom_path).not_to exist @@ -116,7 +117,7 @@ describe Cask::Cmd::Uninstall, :cask do it "displays a message when versions remain installed" do expect do expect do - described_class.run("versioned-cask") + described_class.uninstall_casks(Cask::Cask.new("versioned-cask")) end.not_to output.to_stderr end.to output(/#{token} #{first_installed_version} is still installed./).to_stdout end @@ -148,7 +149,7 @@ describe Cask::Cmd::Uninstall, :cask do end it "can still uninstall them" do - described_class.run("ive-been-renamed") + described_class.uninstall_casks(Cask::Cask.new("ive-been-renamed")) expect(app).not_to exist expect(caskroom_path).not_to exist