]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
pybind/mgr/dashboard: move pytest into requirements.txt 40535/head
authorKefu Chai <kchai@redhat.com>
Thu, 1 Apr 2021 03:05:19 +0000 (11:05 +0800)
committerKefu Chai <kchai@redhat.com>
Thu, 1 Apr 2021 03:05:21 +0000 (11:05 +0800)
commiteab195566d54122f826debd8efb7f36db78fa4e1
tree04d7d0ca1f236923108bec526334f4dea4115420
parent41fb5ff0b200886f923005f086b93c4740a62127
pybind/mgr/dashboard: move pytest into requirements.txt

before this change, pytest is included by both requirements-lint.txt
and requirements-test.txt. this fails the install-deps.sh script when
collecting the python package wheels:

ERROR: Double requirement given: pytest<4 (from -r requirements-test.txt (line 2)) (already in pytest (from -r requirements-lint.txt (line 12)), name='pytest')

also, since pytest is unconditionally imported in the source, for instance,
in pybind/mgr/dashboard/tests/test_ceph_service.py

it would be more straightforward just to include it in requirements.txt.

Signed-off-by: Kefu Chai <kchai@redhat.com>
src/pybind/mgr/dashboard/requirements-lint.txt
src/pybind/mgr/dashboard/requirements-test.txt
src/pybind/mgr/dashboard/requirements.txt