From b4a36bef64c7eb517f84be1d2c05eb0360fa4bc7 Mon Sep 17 00:00:00 2001
From: Rylan Polster <rslpolster@gmail.com>
Date: Sun, 21 Mar 2021 17:11:17 -0400
Subject: [PATCH] audit: fix --display-failures-only failure messaging

---
 Library/Homebrew/cask/audit.rb   | 11 +++++++----
 Library/Homebrew/cask/auditor.rb | 11 ++++++++---
 2 files changed, 15 insertions(+), 7 deletions(-)

diff --git a/Library/Homebrew/cask/audit.rb b/Library/Homebrew/cask/audit.rb
index a3255b2ceaf50..63dd5d234de42 100644
--- a/Library/Homebrew/cask/audit.rb
+++ b/Library/Homebrew/cask/audit.rb
@@ -126,9 +126,10 @@ def result
       end
     end
 
-    sig { params(include_passed: T::Boolean).returns(String) }
-    def summary(include_passed: false)
+    sig { params(include_passed: T::Boolean, include_warnings: T::Boolean).returns(String) }
+    def summary(include_passed: false, include_warnings: true)
       return if success? && !include_passed
+      return if warnings? && !errors? && !include_warnings
 
       summary = ["audit for #{cask}: #{result}"]
 
@@ -136,8 +137,10 @@ def summary(include_passed: false)
         summary << " #{Formatter.error("-")} #{error}"
       end
 
-      warnings.each do |warning|
-        summary << " #{Formatter.warning("-")} #{warning}"
+      if include_warnings
+        warnings.each do |warning|
+          summary << " #{Formatter.warning("-")} #{warning}"
+        end
       end
 
       summary.join("\n")
diff --git a/Library/Homebrew/cask/auditor.rb b/Library/Homebrew/cask/auditor.rb
index 178ee3e4b2c7f..508fb0a379736 100644
--- a/Library/Homebrew/cask/auditor.rb
+++ b/Library/Homebrew/cask/auditor.rb
@@ -75,7 +75,7 @@ def audit
       if !language && language_blocks
         language_blocks.each_key do |l|
           audit = audit_languages(l)
-          summary = audit.summary(include_passed: output_passed?)
+          summary = audit.summary(include_passed: output_passed?, include_warnings: output_warnings?)
           if summary.present? && output_summary?(audit)
             ohai "Auditing language: #{l.map { |lang| "'#{lang}'" }.to_sentence}" if output_summary?
             puts summary
@@ -85,7 +85,7 @@ def audit
         end
       else
         audit = audit_cask_instance(cask)
-        summary = audit.summary(include_passed: output_passed?)
+        summary = audit.summary(include_passed: output_passed?, include_warnings: output_warnings?)
         puts summary if summary.present? && output_summary?(audit)
         warnings += audit.warnings
         errors += audit.errors
@@ -97,7 +97,6 @@ def audit
     private
 
     def output_summary?(audit = nil)
-      return false if @display_failures_only.present?
       return true if @any_named_args.present?
       return true if @audit_strict.present?
       return false if audit.blank?
@@ -112,6 +111,12 @@ def output_passed?
       false
     end
 
+    def output_warnings?
+      return false if @display_failures_only.present?
+
+      true
+    end
+
     def audit_languages(languages)
       original_config = cask.config
       localized_config = original_config.merge(Config.new(explicit: { languages: languages }))