cask: show accurate log messages for in-place upgrades

When a terminal has App Management or Full Disk Access permissions,
Homebrew upgrades cask apps in-place by replacing only the contents of
the .app directory, preserving its inode and macOS metadata (Dock
position, notification settings, granted permissions).

However, the log output still printed "Removing App" and "Moving App"
as if the entire bundle were being deleted and reinstalled.

With this change:
- In-place upgrades print a single "Upgrading App in-place" message
- Full replacement upgrades continue to print "Removing App" and
  "Moving App" as before

Author: sensebat

Library/Homebrew/cask/artifact/moved.rb

@@ -102,8 +102,6 @@ def move(adopt: false, auto_updates: false, force: false, verbose: false, predec
           end
         end
 
-        ohai "Moving #{self.class.english_name} '#{source.basename}' to '#{target}'"
-
         Utils.gain_permissions_mkpath(target.dirname, command:) unless target.dirname.exist?
 
         if target.directory? && Quarantine.app_management_permissions_granted?(app: target, command:)
@@ -116,8 +114,10 @@ def move(adopt: false, auto_updates: false, force: false, verbose: false, predec
           Quarantine.copy_xattrs(source, target, command:)
           FileUtils.rm_r(source)
         elsif target.dirname.writable?
+          ohai "Moving #{self.class.english_name} '#{source.basename}' to '#{target}'"
           FileUtils.move(source, target)
         else
+          ohai "Moving #{self.class.english_name} '#{source.basename}' to '#{target}'"
           # default sudo user isn't necessarily able to write to Homebrew's locations
           # e.g. with runas_default set in the sudoers (5) file.
           command.run!("/bin/cp", args: ["-pR", source, target], sudo: true)
@@ -179,20 +179,21 @@ def move_back(skip: false, force: false, adopt: false, command: nil, **options)
       end
 
       def delete(target, force: false, successor: nil, command: nil, **_)
-        ohai "Removing #{self.class.english_name} '#{target}'"
         raise CaskError, "Cannot remove undeletable #{self.class.english_name}." if undeletable?(target)
 
         return unless Utils.path_occupied?(target)
 
         if target.directory? && matching_artifact?(successor) && Quarantine.app_management_permissions_granted?(
           app: target, command:,
         )
+          ohai "Upgrading #{self.class.english_name} '#{target}' in-place"
           # If an app folder is deleted, macOS considers the app uninstalled and removes some data.
           # Remove only the contents to handle this case.
           target.children.each do |child|
             Utils.gain_permissions_remove(child, command:)
           end
         else
+          ohai "Removing #{self.class.english_name} '#{target}'"
           Utils.gain_permissions_remove(target, command:)
         end
       end