Skip to content

Commit

Permalink
Do not run dashboard workflow test by default
Browse files Browse the repository at this point in the history
  • Loading branch information
stefsmeets committed May 10, 2023
1 parent 9fd8d21 commit 4863af9
Show file tree
Hide file tree
Showing 4 changed files with 13 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -49,7 +49,7 @@ jobs:
python-version: ${{ matrix.python-version }}

- name: Run unit tests
run: pytest -v -m "not dashboard"
run: pytest -v

- name: Verify that we can build the package
run: python3 setup.py sdist bdist_wheel
Expand Down
1 change: 0 additions & 1 deletion pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -92,5 +92,4 @@ line-length = 120

[tool.ruff.isort]
known-first-party = ["dianna"]
force-single-line = true
no-lines-before = ["future","standard-library","third-party","first-party","local-folder"]
4 changes: 4 additions & 0 deletions setup.cfg
Original file line number Diff line number Diff line change
Expand Up @@ -116,3 +116,7 @@ console_scripts =

[options.packages.find]
include = dianna, dianna.*

[tool:pytest]
markers =
dashboard: Test dashboard user workflow, requires playwright with browser installed (`playwright install chromium`)
8 changes: 8 additions & 0 deletions tests/conftest.py
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
import pytest


def pytest_runtest_setup(item):
"""Configure pytest per item."""
# Do not test dashboard workflow by default
if 'dashboard' in (mark.name for mark in item.iter_markers()):
pytest.skip('Use `-m dashboard` to test dashboard workflow.')

0 comments on commit 4863af9

Please sign in to comment.