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

Jump to bottom

🔧 Tweak pytest coverage #485

Merged
merged 9 commits into from Nov 5, 2022
Merged

🔧 Tweak pytest coverage #485

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
f4b28f5
🔧 Tweak pytest cov
tiangolo Nov 5, 2022
6cc446c
🍻 Try excluding 3.6 from coverage and see if that changes things
tiangolo Nov 5, 2022
a9998ba
🍻 Try to enable --forked again and see if that changes things
tiangolo Nov 5, 2022
0491398
🔧 Try running coverage directly instead of pytest-cov plus pytest-xdi…
tiangolo Nov 5, 2022
8d2c82a
🔧 Update CI install to use source, so coverage subprocesses get the r…
tiangolo Nov 5, 2022
90fc176
🐛 Fix CI config
tiangolo Nov 5, 2022
01e8a57
🔧 Try to bring back pytest-xdist now that I found the cause for no co…
tiangolo Nov 5, 2022
db45b81
🔧 Try to simplify pytest-cov configs
tiangolo Nov 5, 2022
74f7cf0
🔧 Simplify pytest-cov config
tiangolo Nov 5, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
5 changes: 5 additions & 0 deletions .github/workflows/test.yml
View file
Open in desktop
Expand Up @@ -25,6 +25,11 @@ jobs:
- name: Install Flit
run: pip install flit
- name: Install Dependencies
if: ${{ matrix.python-version != '3.6' }}
run: python -m flit install --symlink
- name: Install Dependencies
if: ${{ matrix.python-version == '3.6' }}
# This doesn't install the editable install, so coverage doesn't get subprocesses
run: python -m pip install ".[test]"
- name: Install Click 7
if: matrix.click-7
Expand Down
4 changes: 2 additions & 2 deletions scripts/test.sh
View file
Open in desktop
Expand Up @@ -8,5 +8,5 @@ export TERMINAL_WIDTH=3000
# Force disable terminal for tests inside of pytest, takes precedence over GITHUB_ACTIONS env var
export _TYPER_FORCE_DISABLE_TERMINAL=1
bash ./scripts/test-files.sh
# Use xdist-pytest --forked to ensure modified sys.path to import relative modules in examples keeps working
pytest --cov=typer --cov=tests --cov=docs_src --cov-report=term-missing -o console_output_style=progress --numprocesses=auto ${@}
# It seems xdist-pytest ensures modified sys.path to import relative modules in examples keeps working
pytest --cov-config=.coveragerc --cov --cov-report=term-missing -o console_output_style=progress --numprocesses=auto ${@}