Skip to content

Commit

Permalink
Revert "[MAINT] use pytest-reporter-html1 to create test reports (nil…
Browse files Browse the repository at this point in the history
…earn#4444)" (nilearn#4485)

This reverts commit 85e7784.
  • Loading branch information
Remi-Gau authored Jul 1, 2024
1 parent 85e7784 commit fd3a3bf
Show file tree
Hide file tree
Showing 6 changed files with 5 additions and 98 deletions.
6 changes: 0 additions & 6 deletions .github/workflows/test_with_tox.yml
Original file line number Diff line number Diff line change
Expand Up @@ -54,12 +54,6 @@ jobs:
run: tox c
- name: Run test suite
run: tox run -e ${{ matrix.env }} -- nilearn
- name: Upload test report
if: success() || failure()
uses: actions/upload-artifact@v4
with:
name: ${{ matrix.os }}_${{ matrix.py }}_${{ matrix.description }}_report.html
path: report.html
- name: Upload coverage to CodeCov
uses: codecov/codecov-action@v4
with:
Expand Down
1 change: 0 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,6 @@ venv/

# testing
.pytest_cache
report.html

# CI benchmark
maint_tools/*_runs_timing.*
Expand Down
5 changes: 0 additions & 5 deletions .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -112,11 +112,6 @@ repos:
hooks:
- id: prettier
types_or: [css, html]
exclude: |
(?x)^(
maint_tools/templates/index.html
)$

# Check that Python code complies with PEP8 guidelines
# flake8 uses pydocstyle to check docstrings: https://flake8.pycqa.org/en/latest/
Expand Down
80 changes: 0 additions & 80 deletions maint_tools/templates/index.html

This file was deleted.

5 changes: 2 additions & 3 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -91,8 +91,7 @@ style = [
test = [
"coverage",
"pytest>=6.0.0",
"pytest-cov",
"pytest-reporter-html1>=0.9.0"
"pytest-cov"
]

[project.urls]
Expand Down Expand Up @@ -149,7 +148,7 @@ src_paths = [
]

[tool.pytest.ini_options]
addopts = "-ra --strict-config --strict-markers --doctest-modules --showlocals -s -vv --durations=0 --template=maint_tools/templates/index.html"
addopts = "-ra --strict-config --strict-markers --doctest-modules --showlocals -s -vv --durations=0"
doctest_optionflags = "NORMALIZE_WHITESPACE ELLIPSIS"
junit_family = "xunit2"
minversion = "6.0"
Expand Down
6 changes: 3 additions & 3 deletions tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,7 @@ description = run tests on latest version of all dependencies (plotting not incl
passenv = {[global_var]passenv}
extras = test
commands =
pytest --cov=nilearn --cov-report=xml --report=report.html {posargs:}
pytest --cov=nilearn --cov-report=xml {posargs:}

[testenv:test_plotting]
description = run tests on latest version of all dependencies
Expand All @@ -112,9 +112,9 @@ extras = test
deps =
{[plotting]deps}
commands =
pytest doc/_additional_doctests.txt --report=report_doc.html
pytest doc/_additional_doctests.txt
; TODO find a way to rely on globbing instead of listing a specific folder
pytest --doctest-glob='*.rst' doc/manipulating_images/ --report=report_doc.html
pytest --doctest-glob='*.rst' doc/manipulating_images/

[testenv:test_pre]
description = run test_latest and test_doc on pre-release version of all dependencies
Expand Down

0 comments on commit fd3a3bf

Please sign in to comment.