From f726705ae5e2171c141ca33ec1b713a4f3b22d1f Mon Sep 17 00:00:00 2001
From: Ana Krivokapic <akrivoka@redhat.com>
Date: Mon, 27 Jan 2014 13:54:01 -0500
Subject: [PATCH] Run PEP8 check by default when tests are run

Change-Id: I9dacc1f4df807de3a24b4affdea22b4dca651949
Closes-Bug: #1273372
---
 run_tests.sh | 9 ++++++++-
 1 file changed, 8 insertions(+), 1 deletion(-)

diff --git a/run_tests.sh b/run_tests.sh
index cfd48453c..407a43508 100755
--- a/run_tests.sh
+++ b/run_tests.sh
@@ -312,7 +312,14 @@ function run_tests_all {
   # Remove the leftover coverage files from the -p flag earlier.
   rm -f .coverage.*
 
-  if [ $(($TUSKAR_UI_RESULT)) -eq 0 ]; then
+  PEP8_RESULT=0
+  if [ $only_selenium -eq 0 ]; then
+    run_pep8
+    PEP8_RESULT=$?
+  fi
+
+  TEST_RESULT=$(($TUSKAR_UI_RESULT || $PEP8_RESULT))
+  if [ $TEST_RESULT -eq 0 ]; then
     echo "Tests completed successfully."
   else
     echo "Tests failed."