From 85be4b643c882e54ca8717638038641cbfe2c9b9 Mon Sep 17 00:00:00 2001
From: Matthew Grasmick <matthew.grasmick@acquia.com>
Date: Mon, 18 Jul 2016 20:21:58 -0400
Subject: [PATCH] Making security test failure more verbose.

---
 template/build/core/phing/tasks/tests.xml | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/template/build/core/phing/tasks/tests.xml b/template/build/core/phing/tasks/tests.xml
index bc32cc4df..cbbc0e85d 100644
--- a/template/build/core/phing/tasks/tests.xml
+++ b/template/build/core/phing/tasks/tests.xml
@@ -132,13 +132,21 @@
   </target>
 
   <!-- Ensure Drupal doesn't have any modules requiring security updates. -->
-  <!-- Intentionally removed checkreturn="true" so that this does not cause failure. -->
   <target name="tests:security-updates"
           description="Check local Drupal installation for security updates.">
     <exec dir="${docroot}"
           command="! ${drush.cmd} -n ups --check-disabled --security-only 2>/dev/null | grep 'SECURITY UPDATE'"
           checkreturn="true"
           logoutput="true"
+          returnProperty="securityPass"
           passthru="true"/>
+
+    <!-- Output message on failure. -->
+    <if>
+      <not><equals arg1="${securityPass}" arg2="0"/></not>
+      <then>
+        <echo>One or more of your dependency has an outstanding security update. Please apply update(s) immediately. Failing build.</echo>
+      </then>
+    </if>
   </target>
 </project>