Add `Formatter` module.

2 files changed, 12 insertions, 9 deletions

diff --git a/Library/Homebrew/cask/lib/hbc/artifact/moved.rb b/Library/Homebrew/cask/lib/hbc/artifact/moved.rb
index ff6e5f273..22124005c 100644
--- a/Library/Homebrew/cask/lib/hbc/artifact/moved.rb
+++ b/Library/Homebrew/cask/lib/hbc/artifact/moved.rb
@@ -79,12 +79,10 @@ module Hbc
         load_specification artifact_spec
 
         if target.exist?
-          target_abv = " (#{target.abv})"
+          "#{printable_target} (#{target.abv})"
         else
-          error = "#{Tty.red}Missing #{self.class.artifact_english_name}:#{Tty.reset} "
+          Formatter.error(printable_target, label: "Missing #{self.class.artifact_english_name}")
         end
-
-        "#{error}#{printable_target}#{target_abv}"
       end
     end
   end
diff --git a/Library/Homebrew/cask/lib/hbc/artifact/symlinked.rb b/Library/Homebrew/cask/lib/hbc/artifact/symlinked.rb
index 3ab45cccc..46dd42322 100644
--- a/Library/Homebrew/cask/lib/hbc/artifact/symlinked.rb
+++ b/Library/Homebrew/cask/lib/hbc/artifact/symlinked.rb
@@ -57,12 +57,17 @@ module Hbc
       def summarize_artifact(artifact_spec)
         load_specification artifact_spec
 
-        return unless self.class.islink?(target)
-
-        link_description = "#{Tty.red}Broken Link#{Tty.reset}: " unless target.exist?
-        target_readlink_abv = " (#{target.readlink.abv})" if target.readlink.exist?
+        if self.class.islink?(target) && target.exist? && target.readlink.exist?
+          "#{printable_target} -> #{target.readlink} (#{target.readlink.abv})"
+        else
+          string = if self.class.islink?(target)
+                     "#{printable_target} -> #{target.readlink}"
+                   else
+                     printable_target
+                   end
 
-        "#{link_description}#{printable_target} -> #{target.readlink}#{target_readlink_abv}"
+          Formatter.error(string, label: "Broken Link")
+        end
       end
     end
   end