forked from mirrors/qmk_firmware
Move running pytest to travis_test (#7005)
This commit is contained in:
parent
6560dffc05
commit
e1de0d74a6
2 changed files with 27 additions and 19 deletions
|
@ -1,6 +1,6 @@
|
||||||
#!/bin/bash
|
#!/bin/bash
|
||||||
|
|
||||||
# if docker is installed - call make within the qmk docker image
|
# if docker is installed - patch calls to within the qmk docker image
|
||||||
if command -v docker >/dev/null; then
|
if command -v docker >/dev/null; then
|
||||||
function make() {
|
function make() {
|
||||||
docker run --rm -e MAKEFLAGS="$MAKEFLAGS" -w /qmk_firmware/ -v "$PWD":/qmk_firmware --user $(id -u):$(id -g) qmkfm/base_container make "$@"
|
docker run --rm -e MAKEFLAGS="$MAKEFLAGS" -w /qmk_firmware/ -v "$PWD":/qmk_firmware --user $(id -u):$(id -g) qmkfm/base_container make "$@"
|
||||||
|
@ -52,14 +52,6 @@ if [[ "$TRAVIS_COMMIT_MESSAGE" != *"[skip build]"* ]] ; then
|
||||||
fi
|
fi
|
||||||
done
|
done
|
||||||
fi
|
fi
|
||||||
# Check and run python tests if necessary
|
|
||||||
PFM=$(git diff --name-only -n 1 ${TRAVIS_COMMIT_RANGE} | grep -E -e '^(lib/python/)' -e '^(bin/qmk)' | wc -l)
|
|
||||||
if [ $PFM -gt 0 -o "$BRANCH" = "master" ]; then
|
|
||||||
echo
|
|
||||||
echo "Running python tests."
|
|
||||||
docker run --rm -w /qmk_firmware/ -v "$PWD":/qmk_firmware --user $(id -u):$(id -g) qmkfm/base_container 'bin/qmk pytest'
|
|
||||||
: $((exit_code = $exit_code + $?))
|
|
||||||
fi
|
|
||||||
fi
|
fi
|
||||||
exit $exit_code
|
exit $exit_code
|
||||||
fi
|
fi
|
||||||
|
|
|
@ -1,29 +1,45 @@
|
||||||
#!/bin/bash
|
#!/bin/bash
|
||||||
|
|
||||||
|
# if docker is installed - patch calls to within the qmk docker image
|
||||||
|
if command -v docker >/dev/null; then
|
||||||
|
function make() {
|
||||||
|
docker run --rm -e MAKEFLAGS="$MAKEFLAGS" -w /qmk_firmware/ -v "$PWD":/qmk_firmware --user $(id -u):$(id -g) qmkfm/base_container make "$@"
|
||||||
|
}
|
||||||
|
function qmk() {
|
||||||
|
docker run --rm -w /qmk_firmware/ -v "$PWD":/qmk_firmware --user $(id -u):$(id -g) qmkfm/base_container bin/qmk "$@"
|
||||||
|
}
|
||||||
|
fi
|
||||||
|
|
||||||
TRAVIS_COMMIT_MESSAGE="${TRAVIS_COMMIT_MESSAGE:-none}"
|
TRAVIS_COMMIT_MESSAGE="${TRAVIS_COMMIT_MESSAGE:-none}"
|
||||||
TRAVIS_COMMIT_RANGE="${TRAVIS_COMMIT_RANGE:-HEAD~1..HEAD}"
|
TRAVIS_COMMIT_RANGE="${TRAVIS_COMMIT_RANGE:-HEAD~1..HEAD}"
|
||||||
|
|
||||||
# test force push
|
# test force push
|
||||||
#TRAVIS_COMMIT_RANGE="c287f1bfc5c8...81f62atc4c1d"
|
#TRAVIS_COMMIT_RANGE="c287f1bfc5c8...81f62atc4c1d"
|
||||||
|
|
||||||
NUM_IMPACTING_CHANGES=$(git diff --name-only -n 1 ${TRAVIS_COMMIT_RANGE} | grep -Ecv '^(docs/)')
|
|
||||||
BRANCH=$(git rev-parse --abbrev-ref HEAD)
|
BRANCH=$(git rev-parse --abbrev-ref HEAD)
|
||||||
|
CHANGES=$(git diff --name-only -n 1 ${TRAVIS_COMMIT_RANGE})
|
||||||
|
|
||||||
|
NUM_CORE_CHANGES=$(echo "$CHANGES" | grep -Ecv -e '^(docs/)' -e '^(keyboards/)' -e '^(layouts/)')
|
||||||
|
NUM_PY_CHANGES=$(echo "$CHANGES" | grep -Ec -e '^(lib/python/)' -e '^(bin/qmk)')
|
||||||
|
|
||||||
if [[ "$TRAVIS_COMMIT_MESSAGE" == *"[skip test]"* ]]; then
|
if [[ "$TRAVIS_COMMIT_MESSAGE" == *"[skip test]"* ]]; then
|
||||||
echo "Skipping due to commit message"
|
echo "Skipping due to commit message"
|
||||||
exit 0
|
exit 0
|
||||||
fi
|
fi
|
||||||
|
|
||||||
if [ "$BRANCH" != "master" ] && [ "$NUM_IMPACTING_CHANGES" == "0" ]; then
|
exit_code=0
|
||||||
echo "Skipping due to changes not impacting tests"
|
|
||||||
exit 0
|
if [ "$BRANCH" == "master" ] || [ "$NUM_CORE_CHANGES" != "0" ]; then
|
||||||
|
echo "Running tests."
|
||||||
|
make test:all
|
||||||
|
: $((exit_code = $exit_code + $?))
|
||||||
|
|
||||||
fi
|
fi
|
||||||
|
|
||||||
# if docker is installed - call make within the qmk docker image
|
if [ "$BRANCH" == "master" ] || [ "$NUM_PY_CHANGES" != "0" ]; then
|
||||||
if command -v docker >/dev/null; then
|
echo "Running python tests."
|
||||||
function make() {
|
qmk pytest
|
||||||
docker run --rm -e MAKEFLAGS="$MAKEFLAGS" -w /qmk_firmware/ -v "$PWD":/qmk_firmware --user $(id -u):$(id -g) qmkfm/base_container make "$@"
|
: $((exit_code = $exit_code + $?))
|
||||||
}
|
|
||||||
fi
|
fi
|
||||||
|
|
||||||
make test:all
|
exit $exit_code
|
||||||
|
|
Loading…
Reference in a new issue