Skip to content

Commit

Permalink
Add pytest to Github tests
Browse files Browse the repository at this point in the history
  • Loading branch information
@Rudd-O
Rudd-O authored Feb 8, 2023
1 parent 48ba347 commit 11b5c4b
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion .github/workflows/test.yml
View file
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ jobs:
# Needed to allow Tox to run in the current env.
# Current env necessary because we are installing GTK+ and other
# libraries needed for the programs.
pip3 install tox-current-env

This comment has been minimized.

Sign in to view

Copy link
@Rudd-O

Rudd-O Feb 8, 2023

Author Owner

We have passing tests! @porridge

This comment has been minimized.

Sign in to view

Copy link
@porridge

porridge Feb 8, 2023

Contributor

Um, AFAICT pytest was installed even before, as a deb package, and there does not seem to be much of a difference to how the tests are executed compared to now 🤔

This comment has been minimized.

Sign in to view

Copy link
@Rudd-O

Rudd-O Feb 11, 2023

Author Owner

Well, it was installed before but it wasn't working (it just said no command named pytest).

This comment has been minimized.

Sign in to view

Copy link
@porridge

porridge Feb 11, 2023

Contributor

Not sure where you're seeing this. Here is what I'm seeing on the last commit in December:
Screenshot_20230211-083038_Chrome

This comment has been minimized.

Sign in to view

Copy link
@Rudd-O

Rudd-O Feb 11, 2023

Author Owner

It stopped working for some reason, and only adding pytest as a pip package would work to fix it. Maybe they just changed the container somehow subtly.
pip3 install tox-current-env pytest
- name: Run tests
run: |
# pylint $(git ls-files '*.py')
Expand Down

0 comments on commit 11b5c4b

Please sign in to comment.