Skip to content

CI tests via GitHub actions and conda #504

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
murrayrm merged 10 commits into from
Jan 12, 2021
Merged

CI tests via GitHub actions and conda #504

Show file tree
Hide file tree
Changes from 3 commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
3f87c33
Create python-package-conda.yml
murrayrm Jan 10, 2021
21e47a6
remove flake8 lint; add dependencies
murrayrm Jan 10, 2021
d83e67d
skip X11 tests if no DISPLAY env variable
murrayrm Jan 10, 2021
627754f
expand test matrix to match Travis CI
murrayrm Jan 10, 2021
47251c6
simplify GitHub actions workflow
murrayrm Jan 11, 2021
6ff2e9b
pytest with slycot from source
murrayrm Jan 11, 2021
5bb9de4
update xvfb configuration; remove X11 marks
murrayrm Jan 12, 2021
36ecea1
update xvfb configuration; remove X11 marks
murrayrm Jan 12, 2021
b7b696f
add coveralls
murrayrm Jan 12, 2021
40aac67
Merge branch 'github-actions-conda_revised' into github-actions-conda
murrayrm Jan 12, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .coveragerc
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
[run]
source = control
omit = control/tests/*
relative_files = True

[report]
exclude_lines =
Expand Down
25 changes: 22 additions & 3 deletions .github/workflows/python-package-conda.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ name: Conda-based pytest
on: [push, pull_request]

jobs:
build-linux:
test-linux:
runs-on: ubuntu-latest

strategy:
Expand All @@ -19,6 +19,7 @@ jobs:

steps:
- uses: actions/checkout@v2

- name: Install dependencies
run: |
# Set up conda
Expand All @@ -31,18 +32,36 @@ jobs:

# Install test tools
conda install pip coverage pytest
conda install -c conda-forge pytest-cov
pip install coveralls

@bnavigator bnavigator Jan 12, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

coveralls is on conda-forge nowadays


# Install python-control dependencies
conda install numpy matplotlib scipy
if [[ '${{matrix.slycot}}' == 'conda' ]]; then
conda install -c conda-forge slycot
fi

- name: Test with pytest
env:
PYTHON_CONTROL_ARRAY_AND_MATRIX: ${{ matrix.array-and-matrix }}
run: |
source $CONDA/bin/activate test-environment
# Use xvfb-run instead of pytest-xvfb to get proper mpl backend
# Use coverage instead of pytest-cov to get .coverage file

@bnavigator bnavigator Jan 12, 2021

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pytest-cov produces one. See python-control/Slycot#140. Use the --cov parameter.

# See https://github.com/python-control/python-control/pull/504
xvfb-run --auto-servernum pytest control/tests
xvfb-run --auto-servernum coverage run -m pytest control/tests

- name: Coveralls parallel
# https://github.com/coverallsapp/github-action
uses: AndreMiras/coveralls-python-action@develop
with:
parallel: true

coveralls:
name: coveralls completion
needs: test-linux
runs-on: ubuntu-latest
steps:
- name: Coveralls Finished
uses: AndreMiras/coveralls-python-action@develop
with:
parallel-finished: true