Testing: only use changed files to python3 test; Fix rucio#1505

hahahannes · Oct 16, 2018 · 641f558 · 641f558
1 parent af71aba
commit 641f558
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 111 deletions.
diff --git a/tools/travis/check_python_3.sh b/tools/travis/check_python_3.sh
@@ -20,16 +20,16 @@ echo '==============================='
 echo 'Running pylint                 '
 echo '==============================='
 
-for filename in $(find lib/rucio bin);
+for filename in $(cat changed_files.txt);
 do
     if grep -q "PY3K COMPATIBLE" $filename; then
         echo "Check if file" $filename "is compatible"
         pylint --py3k -d no-absolute-import $filename 
         if [ $? -ne 0 ]; then
             echo "PYLINT FAILED"
+            cat pylint.out
             exit 1
         fi
     fi
-done
+done 
 echo "PYLINT PASSED"
-tail -n 3 pylint.out
diff --git a/tools/travis/install_script.sh b/tools/travis/install_script.sh
@@ -39,7 +39,7 @@ elif [[ $SUITE == "all" ]]; then
     cp etc/docker/travis/Dockerfile Dockerfile
     docker build -t rucio/rucio .
 
-elif [[ SUITE == 'python3' ]]; then
+elif [[ $SUITE == 'python3' ]]; then 
     pip install -r tools/pip-requires-test
 
 fi
diff --git a/tools/travis/py3_compatible.txt b/tools/travis/py3_compatible.txt