Made run_tests.sh use tox to run checks

Now run_tests.sh script will use tox to run PEP8 checks.
This will allow to use run_tests.sh in Jenkins build scripts and
that step is required to further improvements of OSTF tests.

This is a step required to unify our CI jobs configuration.

Change-Id: I51012b2d633557141baebd5669db256dd5e99efd
Partial-Bug: #1404892
(cherry picked from commit c0e9862e6a)

run_tests.sh

@@ -1,42 +1,77 @@
 #!/bin/bash
 
+#    Copyright 2015 Mirantis, Inc.
+#
+#    Licensed under the Apache License, Version 2.0 (the "License"); you may
+#    not use this file except in compliance with the License. You may obtain
+#    a copy of the License at
+#
+#         http://www.apache.org/licenses/LICENSE-2.0
+#
+#    Unless required by applicable law or agreed to in writing, software
+#    distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+#    License for the specific language governing permissions and limitations
+#    under the License.
+
 function usage {
   echo "Usage: $0 [OPTION]..."
   echo ""
-  echo " -p, --pep8 Just run pep8"
-  echo " -h, --help Print this usage message"
+  echo " -p, --pep8    Just run pep8"
+  echo " -P, --no-pep8 Do not run pep8"
+  echo " -h, --help    Print this usage message"
   echo ""
   exit
 }
 
+
 function process_option {
   case "$1" in
     -h|--help) usage;;
     -p|--pep8) just_pep8=1;;
+    -P|--no-pep8) no_just_pep8=1;;
   esac
 }
 
+
 just_pep8=0
+no_just_pep8=0
+
+
+function run_tests {
+  # This variable collects all failed tests. It'll be printed in
+  # the end of this function as a small statistic for user.
+  local errors=""
+
+
+  # Enable all tests if none was specified skipping all explicitly disabled tests.
+  if [[ $just_pep8 -eq 0 ]]; then
+    if [ $no_just_pep8 -ne 1 ];  then just_pep8=1;  fi
+  fi
+
+  # Run all enabled tests
+  if [ $just_pep8 -eq 1 ]; then
+    run_pep8 || errors+=" pep8_checks"
+  fi
+
+  # print failed tests
+  if [ -n "$errors" ]; then
+    echo Failed tests: $errors
+    exit 1
+  fi
+
+  exit
+}
+
+
+function run_pep8 {
+  echo "Running pep8 ..."
+  tox -epep8 -v
+}
+
 
 for arg in "$@"; do
     process_option $arg
 done
 
-function run_pep8 {
-  echo "Running pep8 ..."
-  # Opt-out files from pep8
-  ignore_scripts="*.pyc,*.pyo,*.sh,*.swp,*.rst"
-  ignore_files="*eventlet-patch:*pip-requires"
-  ignore_dirs=".venv,.tox,dist,doc,vendor,*egg"
-  GLOBIGNORE="$ignore_scripts:$ignore_files:$ignore_dirs"
-  ignore="$ignore_scripts,$ignore_dirs"
-  srcfiles="."
-  ${wrapper} pep8 --repeat $FLAGS --show-source \
-      --exclude=${ignore} ${srcfiles} | tee pep8.txt
-}
-
-if [ $just_pep8 -eq 1 ]; then
-run_pep8
-    exit
-fi
-
+run_tests