Skip to content

Commit 45773c2

Browse files
committed
Instrument workflows to investigate skipped tests
This makes jobs from both test workflows give more information relevant to examining which tests are skipped (and if any tests xfail, those too) in what environments: - Values of os.name and git.util.is_win. - The name of each test that runs, with its status. The latter doesn't increase the output length as much as might be expected, because due to the way the output is handled, the pytest-sugar pretty output format without -v looked like: test/test_actor.py ✓ 0% test/test_actor.py ✓✓ 0% ▏ test/test_actor.py ✓✓✓ 1% ▏ test/test_actor.py ✓✓✓✓ 1% ▏ When instead it was intended to fit on a single line. Still, the current output with -v has extra newlines, increasing length and worsening readability, so it should be improved on if possible.
1 parent abd445f commit 45773c2

File tree

2 files changed

+10
-2
lines changed

2 files changed

+10
-2
lines changed

.github/workflows/cygwin-test.yml

+5-1
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,11 @@ jobs:
5858
run: |
5959
/usr/bin/python -m pip install ".[test]"
6060
61+
- name: Check 'is_win'
62+
run: |
63+
/usr/bin/python -c 'import os, git; print(f"os.name={os.name}, is_win={git.compat.is_win}")'
64+
6165
- name: Test with pytest
6266
run: |
6367
set +x
64-
/usr/bin/python -m pytest
68+
/usr/bin/python -m pytest -v

.github/workflows/pythonpackage.yml

+5-1
Original file line numberDiff line numberDiff line change
@@ -73,9 +73,13 @@ jobs:
7373
# so we have to ignore errors until that changes.
7474
continue-on-error: true
7575

76+
- name: Check 'is_win'
77+
run: |
78+
python -c 'import os, git; print(f"os.name={os.name}, is_win={git.compat.is_win}")'
79+
7680
- name: Test with pytest
7781
run: |
78-
pytest
82+
pytest -v
7983
continue-on-error: false
8084

8185
- name: Documentation

0 commit comments

Comments
 (0)