From 36051b6a71e60007e5200fc5e8b3b83eebade8f7 Mon Sep 17 00:00:00 2001
From: Mike McQuaid <mike@mikemcquaid.com>
Date: Mon, 15 Jul 2013 19:33:03 -0700
Subject: [PATCH] brew-test-bot: audit after installation.

Enables new post-installation Cellar audit failures.

Closes Homebrew/homebrew#21242.
---
 Library/Contributions/cmd/brew-test-bot.rb | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/Library/Contributions/cmd/brew-test-bot.rb b/Library/Contributions/cmd/brew-test-bot.rb
index e19272d304..4b5deb19c4 100755
--- a/Library/Contributions/cmd/brew-test-bot.rb
+++ b/Library/Contributions/cmd/brew-test-bot.rb
@@ -253,15 +253,17 @@ class Test
       return
     end
 
-    test "brew audit #{formula}"
     test "brew fetch #{dependencies}" unless dependencies.empty?
     test "brew fetch --force --build-bottle #{formula}"
     test "brew uninstall --force #{formula}" if formula_object.installed?
     test "brew install --verbose --build-bottle #{formula}"
-    return unless steps.last.passed?
-    bottle_step = test "brew bottle #{formula}", :puts_output_on_success => true
+    install_passed = steps.last.passed?
+    test "brew audit #{formula}"
+    return unless install_passed
+    test "brew bottle #{formula}", :puts_output_on_success => true
     bottle_revision = bottle_new_revision(formula_object)
     bottle_filename = bottle_filename(formula_object, bottle_revision)
+    bottle_step = steps.last
     if bottle_step.passed? and bottle_step.has_output?
       bottle_base = bottle_filename.gsub(bottle_suffix(bottle_revision), '')
       bottle_output = bottle_step.output.gsub /.*(bottle do.*end)/m, '\1'
-- 
GitLab