Skip to content

Commit

Permalink
update unittest.yml (#115)
Browse files Browse the repository at this point in the history
Fix test timeout in Actions. Changed from 30 minutes to 90 minutes.

* Update unittest.yml

* add timeout to pytests
  • Loading branch information
Ziqi-Li committed Mar 7, 2022
1 parent eef4f70 commit 2a95535
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 3 deletions.
1 change: 1 addition & 0 deletions .ci/37.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,3 +15,4 @@ dependencies:
- coverage
- pytest
- pytest-cov
- pytest-timeout
1 change: 1 addition & 0 deletions .ci/38.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,3 +15,4 @@ dependencies:
- coverage
- pytest
- pytest-cov
- pytest-timeout
1 change: 1 addition & 0 deletions .ci/39-DEV.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -16,3 +16,4 @@ dependencies:
- coverage
- pytest
- pytest-cov
- pytest-timeout
1 change: 1 addition & 0 deletions .ci/39.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -15,3 +15,4 @@ dependencies:
- coverage
- pytest
- pytest-cov
- pytest-timeout
6 changes: 3 additions & 3 deletions .github/workflows/unittest.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
unittests:
name: ${{ matrix.os }}, ${{ matrix.environment-file }}
runs-on: ${{ matrix.os }}
timeout-minutes: 30
timeout-minutes: 90
strategy:
matrix:
os: ['macos-latest', 'ubuntu-latest', 'windows-latest']
Expand Down Expand Up @@ -45,12 +45,12 @@

- name: run tests - bash
shell: bash -l {0}
run: pytest -v mgwr --cov=mgwr --cov-report=xml
run: pytest -v mgwr --cov=mgwr --cov-report=xml --timeout=300
if: matrix.os != 'windows-latest'

- name: run tests - powershell
shell: powershell
run: pytest -v mgwr --cov=mgwr --cov-report=xml
run: pytest -v mgwr --cov=mgwr --cov-report=xml --timeout=300
if: matrix.os == 'windows-latest'

- name: codecov
Expand Down

0 comments on commit 2a95535

Please sign in to comment.