Skip to content
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

ci: add testing on latest #145

Merged
merged 2 commits into from
Aug 20, 2024
Merged

ci: add testing on latest #145

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1c15e1f
ci: add testing on latest
Saviq Aug 13, 2024
f6e2a7d
ci: run setup script on installed snaps
Saviq Aug 14, 2024
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
56 changes: 42 additions & 14 deletions .github/workflows/test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -21,12 +21,33 @@ jobs:
server:
- mir_kiosk
- ubuntu_frame
- confined_shell
- mir_test_tools
- mir_demo_server
- gnome_shell
channel: [stable]
include:
- server: ubuntu_frame
channel: 24/edge

- server: mir_test_tools
channel: 24/edge

- server: mir_demo_server
channel: null

- server: mir_demo_server
channel: null
ppa: mir-team/dev
Comment on lines +33 to +38
Copy link
Contributor

Choose a reason for hiding this comment

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

Why have mir_demo_server listed twice with and without a ppa? I assume the one without a ppa uses the stable version and the one with uses the most recent one?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Exactly, yes.


- server: confined_shell
channel: beta

- server: confined_shell
channel: edge

- server: gnome_shell
channel: null

- server: null
channel: null
mark: self
Comment on lines 49 to 51
Copy link
Contributor

Choose a reason for hiding this comment

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

What does this do?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This runs only the self tests, those marked with @pytest.mark.self. And measures coverage on them. So no server is involved - hence no channel.


runs-on: ubuntu-24.04
Expand All @@ -40,22 +61,29 @@ jobs:
with:
lfs: true

- id: pytest-args
name: Determine pytest arguments
- id: vars
name: Determine variables
run: |
ARGS=( --verbose --capture=no )
[ -n '${{ matrix.server }}' ] && ARGS+=( -k ${{ matrix.server }} )
[ -n '${{ matrix.mark }}' ] && ARGS+=( -m ${{ matrix.mark }} )
[ '${{ matrix.mark }}' == 'self' ] && ARGS+=( --cov --cov-branch --cov-report=xml )
echo args=${ARGS[@]} >> $GITHUB_OUTPUT
PYTEST_ARGS=( --verbose --capture=no )
[ -n '${{ matrix.server }}' ] && PYTEST_ARGS+=( -k ${{ matrix.server }} )
[ -n '${{ matrix.mark }}' ] && PYTEST_ARGS+=( -m ${{ matrix.mark }} )
[ '${{ matrix.mark }}' == 'self' ] && PYTEST_ARGS+=( --cov --cov-branch --cov-report=xml )
echo pytest_args=${PYTEST_ARGS[@]} >> $GITHUB_OUTPUT

ARTIFACT_KEY="${{ matrix.server || matrix.mark }}"
[ -n '${{ matrix.channel }}' ] && ARTIFACT_KEY+="-$( echo ${{ matrix.channel }} | tr / _ )"
[ -n '${{ matrix.ppa }}' ] && ARTIFACT_KEY+="-$( echo ${{ matrix.ppa }} | tr / _ )"
echo artifact_key=${ARTIFACT_KEY} >> $GITHUB_OUTPUT

- name: Set up dependencies
working-directory: mir-ci/mir_ci
run: |
[ -n '${{ matrix.ppa }}' ] && sudo add-apt-repository --yes ppa:${{ matrix.ppa }}
[ -n '${{ matrix.channel }}' ] && sudo snap install $( echo ${{ matrix.server }} | tr _ - ) --channel ${{ matrix.channel }}
sudo apt-get --yes install pkg-config libwayland-dev ffmpeg
pip install -e ..
[ '${{ matrix.mark }}' == 'self' ] && pip install pytest-cov
python -m pytest ${{ steps.pytest-args.outputs.args }} --deps
python -m pytest ${{ steps.vars.outputs.pytest_args }} --deps

- name: Run the tests
working-directory: mir-ci/mir_ci
Expand All @@ -68,13 +96,13 @@ jobs:

sudo systemd-run --uid 1001 -p Environment=XDG_SESSION_TYPE=wayland -p PAMName=login -p TTYPath=/dev/tty1 -- tail -f /dev/null

python -m pytest ${{ steps.pytest-args.outputs.args }} --junitxml=junit-${{ matrix.server || matrix.mark }}-${{ matrix.python }}.xml
python -m pytest ${{ steps.vars.outputs.pytest_args }} --junitxml=junit-${{ matrix.server || matrix.mark }}-${{ matrix.python }}.xml

- name: Upload test results
uses: actions/upload-artifact@v4
if: success() || failure()
with:
name: test-results-${{ matrix.server || matrix.mark }}
name: test-results-${{ steps.vars.outputs.artifact_key }}
path: |
mir-ci/mir_ci/junit-*.xml
/tmp/pytest-of-*/*-current/**/log.html
Expand All @@ -97,7 +125,7 @@ jobs:
name: Upload any core dumps
uses: actions/upload-artifact@v4
with:
name: core-dumps-${{ matrix.server || matrix.mark }}
name: core-dumps-${{ steps.vars.outputs.artifact_key }}
path: corefile-*
if-no-files-found: ignore

Expand Down