Skip to content

Commit

Permalink
Fix coveralls (#323)
Browse files Browse the repository at this point in the history
* Revert to AndreMiras/coveralls-python-action; couldn't get it to work with coverallsapp/github-action, final coverage was always wrong.
* Fix test for empymod v2.3
  • Loading branch information
prisae authored Mar 3, 2024
1 parent 1fd015e commit 072ba17
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 3 deletions.
10 changes: 8 additions & 2 deletions .github/workflows/linux.yml
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,10 @@ jobs:
pytest --cov=emg3d
- name: Coveralls
uses: coverallsapp/github-action@v2
# [pin v20201129]
uses: AndreMiras/coveralls-python-action@f5fd5c309b39d01599fb92c72d4f7409ea78aec9
# v Did not work when tried; final coverage was wrong v
# uses: coverallsapp/github-action@v2
with:
parallel: true
flag-name: ${{ matrix.case.python-version }} ${{ matrix.case.name }} (${{ matrix.case.os }})
Expand All @@ -133,7 +136,10 @@ jobs:
runs-on: ubuntu-latest
steps:
- name: Coveralls Finished
uses: coverallsapp/github-action@v2
# [pin v20201129]
uses: AndreMiras/coveralls-python-action@f5fd5c309b39d01599fb92c72d4f7409ea78aec9
# v Did not work when tried; final coverage was wrong v
# uses: coverallsapp/github-action@v2
with:
parallel-finished: true

Expand Down
2 changes: 1 addition & 1 deletion tests/test_time.py
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ def test_defaults(self, capsys):
assert Fourier.signal == 0 # Impulse respons
assert_allclose(times, Fourier.time, 0, 0)
assert Fourier.verb == 3 # Verbose by default
assert 'Key 201 CosSin (2012)' in out
assert 'key' in out.lower()
assert 'Req. freq' in out
assert 'Calc. freq' in out
assert Fourier.freq_compute.min() >= fmin
Expand Down

0 comments on commit 072ba17

Please sign in to comment.