~ruther/qmk_firmware

75e7018f72a716ca2809e337d524966c7c4137a8 — Joel Challis 5 years ago 8fe29f2
CI: Add workflow for CLI testing (#7357)

Create GitHub Actions cli test workflow and remove travis runs of 'qmk pytest'
3 files changed, 29 insertions(+), 8 deletions(-)

A .github/workflows/cli.yml
M lib/python/qmk/tests/test_cli_commands.py
M util/travis_test.sh
A .github/workflows/cli.yml => .github/workflows/cli.yml +28 -0
@@ 0,0 1,28 @@
name: CLI CI

on:
  push:
    branches:
    - master
    - future
  pull_request:
    paths:
    - 'lib/python/**'
    - 'bin/qmk'
    - 'requirements.txt'
    - '.github/workflows/cli.yml'

jobs:
  test:
    runs-on: ubuntu-latest

    container: qmkfm/base_container

    steps:
    - uses: actions/checkout@v1
      with:
        submodules: recursive
    - name: Install dependencies
      run: pip3 install -r requirements.txt
    - name: Run tests
      run: bin/qmk pytest

M lib/python/qmk/tests/test_cli_commands.py => lib/python/qmk/tests/test_cli_commands.py +1 -1
@@ 30,7 30,7 @@ def test_kle2json():


def test_doctor():
    result = check_subcommand('doctor')
    result = check_subcommand('doctor', '-n')
    assert result.returncode == 0
    assert 'QMK Doctor is checking your environment.' in result.stderr
    assert 'QMK is ready to go' in result.stderr

M util/travis_test.sh => util/travis_test.sh +0 -7
@@ 3,7 3,6 @@
source util/travis_utils.sh

NUM_CORE_CHANGES=$(echo "$QMK_CHANGES" | grep -Ecv -e '^(docs/)' -e '^(keyboards/)' -e '^(layouts/)' -e '^(util/)' -e '^(lib/python/)' -e '^(bin/qmk)' -e '^(requirements.txt)' -e '(.travis.yml)')
NUM_PY_CHANGES=$(echo "$QMK_CHANGES" | grep -Ec -e '^(lib/python/)' -e '^(bin/qmk)')

if [[ "$TRAVIS_COMMIT_MESSAGE" == *"[skip test]"* ]]; then
    echo "Skipping due to commit message"


@@ 19,10 18,4 @@ if [ "$LOCAL_BRANCH" == "master" ] || [ "$NUM_CORE_CHANGES" != "0" ]; then

fi

if [ "$LOCAL_BRANCH" == "master" ] || [ "$NUM_PY_CHANGES" != "0" ]; then
    echo "Running python tests."
    qmk pytest
    : $((exit_code = $exit_code + $?))
fi

exit $exit_code