Skip to content

Commit f9b0fe2

Browse files
authored
Merge pull request #1716 from consideRatio/pr/codecov-fixes
maint: codecov step added, non-deprecated uploader, editable install
2 parents b3fa167 + 1bea278 commit f9b0fe2

File tree

2 files changed

+12
-8
lines changed

2 files changed

+12
-8
lines changed

.github/workflows/test.yml

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -171,7 +171,7 @@ jobs:
171171
- name: Setup Python package dependencies
172172
run: |
173173
pip install -r dev-requirements.txt -r helm-chart/images/binderhub/requirements.txt
174-
pip install .
174+
pip install -e .
175175
176176
- name: Install JupyterHub chart for main tests
177177
if: matrix.test == 'main'
@@ -360,9 +360,7 @@ jobs:
360360
daemonset/binderhub-test-pink
361361
362362
# GitHub action reference: https://github.com/codecov/codecov-action
363-
- name: Upload coverage stats
364-
uses: codecov/codecov-action@v3
365-
if: ${{ always() }}
363+
- uses: codecov/codecov-action@v3
366364

367365
test-local:
368366
runs-on: ubuntu-22.04
@@ -413,9 +411,14 @@ jobs:
413411
pip install --upgrade setuptools wheel
414412
415413
- name: Setup Python package dependencies
414+
# FIXME: We do "pip install ." before "pip install -e ." to ensure
415+
# binderhub/static/dist/bundle.js is built, and use "-e" to
416+
# ensure code coverage is captured.
417+
#
416418
run: |
417419
pip install -r dev-requirements.txt -r testing/local-binder-local-hub/requirements.txt
418420
pip install ".[pycurl]"
421+
pip install -e ".[pycurl]"
419422
420423
- name: Setup JupyterHub NPM dependencies
421424
run: npm install -g configurable-http-proxy
@@ -432,8 +435,11 @@ jobs:
432435
- name: Run remote tests
433436
run: |
434437
export BINDER_URL=http://localhost:8000/services/binder/
435-
pytest -m remote -v --color=yes
438+
pytest -m remote --cov=binderhub
436439
437440
- name: Show hub logs
438441
if: always()
439442
run: cat testing/local-binder-local-hub/jupyterhub.log
443+
444+
# GitHub action reference: https://github.com/codecov/codecov-action
445+
- uses: codecov/codecov-action@v3

dev-requirements.txt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,7 @@
1-
beautifulsoup4
1+
beautifulsoup4[html5lib]
22
build
33
chartpress>=2.1
44
click
5-
codecov
6-
html5lib
75
jsonschema
86
jupyter-repo2docker>=2021.08.0
97
jupyter_packaging>=0.10.4,<2

0 commit comments

Comments
 (0)