comp-chem/cp2k
1
0
Fork 0
mirror of https://github.com/cp2k/cp2k.git synced 2026-07-28 14:15:19 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

implement pre-commit checks in CI

Browse source 
fixes #83
This commit is contained in:
Tiziano Müller 2019-05-14 12:56:39 +02:00 • committed by Tiziano Müller
parent b8129c35f7
commit 3e740b3103
3 changed files with 28 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

18
tools/docker/scripts/test_python.sh
View file

@ -16,11 +16,11 @@ function run_tests {
echo "Skipping $i - it's Python3 only."
continue
fi
echo "Selftesting $i"
echo "Running $i"
cd "$(dirname "$i")"
SCRIPT=$(basename "$i")
if ! $PYTHON -B "./${SCRIPT}" ; then
echo "Selftest of $i failed"
echo "Test $i failed"
ERRORS=$((ERRORS+1))
fi
set -e #re-enable error trapping
@ -28,6 +28,15 @@ function run_tests {
done
}
function run_pre_commit {
set +e #disable error trapping
if ! pre-commit run --all-files --hook-stage manual check-ast ; then
echo "Syntax check failed for either Python 2 or 3"
ERRORS=$((ERRORS+1))
fi
set -e #re-enable error trapping
}
#===============================================================================
ERRORS=0
@ -46,8 +55,11 @@ run_tests python2.7 "${ALL_TEST_SCRIPTS}"
# python 3.x
run_tests python3 "${ALL_TEST_SCRIPTS}"
# run manual pre-commit checks
run_pre_commit
# print report
NUM_TEST_SCRIPTS=$(echo "${ALL_TEST_SCRIPTS}" | wc -w)
NUM_TEST_SCRIPTS=$(( $(wc -w <<< "${ALL_TEST_SCRIPTS}") + 1 ))
echo ""
echo "Summary: Run ${NUM_TEST_SCRIPTS} Python test scripts, found ${ERRORS} errors."
if [ $ERRORS == 0 ]; then