Skip to content

Commit 504f531

Browse files
author
Sylvain MARIE
committed
Fixed nox file: for coverage measurements, installing package in dev mode is required
1 parent 44a925c commit 504f531

1 file changed

Lines changed: 8 additions & 5 deletions

File tree

noxfile.py

Lines changed: 8 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -103,19 +103,22 @@ def tests(session: PowerSession, coverage, pkg_specs):
103103
# Fail if the assumed python version is not the actual one
104104
session.run2("python ci_tools/check_python_version.py %s" % session.python)
105105

106-
# install self so that it is recognized by pytest
107-
session.run2("pip install . --no-deps")
108-
# session.install(".", "--no-deps")
109-
110106
# check that it can be imported even from a different folder
111107
# Important: do not surround the command into double quotes as in the shell !
112-
session.run('python', '-c', 'import os; os.chdir(\'./docs/\'); import %s' % pkg_name)
108+
# session.run('python', '-c', 'import os; os.chdir(\'./docs/\'); import %s' % pkg_name)
113109

114110
# finally run all tests
115111
if not coverage:
112+
# install self so that it is recognized by pytest
113+
session.run2("pip install . --no-deps")
114+
# session.install(".", "--no-deps")
115+
116116
# simple: pytest only
117117
session.run2("python -m pytest --cache-clear -v tests/")
118118
else:
119+
# install self in "develop" mode so that coverage can be measured
120+
session.run2("pip install -e . --no-deps")
121+
119122
# coverage + junit html reports + badge generation
120123
session.install_reqs(phase="coverage",
121124
phase_reqs=["coverage", "pytest-html", "genbadge[tests,coverage]"],

0 commit comments

Comments
 (0)