base tests: make sure to fail CI if tests need formatting

Author: matthiaskrgr

ci/base-tests.sh

@@ -38,6 +38,7 @@ cargo +nightly fmt --all -- --check
 
 #avoid loop spam
 set +x
+
 # make sure tests are formatted
 
 # some lints are sensitive to formatting, exclude some files
@@ -47,11 +48,13 @@ rustup default nightly
 for file in `find tests -not -path "tests/ui/methods.rs" -not -path "tests/ui/format.rs" -not -path "tests/ui/formatting.rs" -not -path "tests/ui/empty_line_after_outer_attribute.rs" -not -path "tests/ui/double_parens.rs" -not -path "tests/ui/doc.rs" -not -path "tests/ui/unused_unit.rs" | grep "\.rs$"` ; do
 rustfmt ${file} --check  || echo "${file} needs reformatting!" ; needs_formatting=true
 done
+
+set -x
+
 # switch back to master
 rustup default master
 
-if [ "${needs_reformatting}" = true ] ; then
+if [[ ${needs_reformatting} == true ]] ; then
     echo "Tests need reformatting!"
     exit 2
 fi
-set -x