Task #3553
closedMove tests out of Pulp's application code
100%
Description
The tests currently live at: https://github.com/pulp/pulp/tree/3.0-dev/pulpcore/pulpcore/app/tests
This is atypical. Usually tests live outside of the application code itself as a peer of it.
Since we do have two packages living in one repository, the existing tests would move from /pulpcore/pulpcore/app/tests
to /pulpcore/tests/
. Eventually we would add plugin API tests in /plugin/tests/
Updated by daviddavis over 6 years ago
- Groomed changed from No to Yes
- Sprint set to Sprint 35
Updated by dalley over 6 years ago
- Status changed from NEW to ASSIGNED
- Assignee set to dalley
Updated by dalley over 6 years ago
- Status changed from ASSIGNED to POST
Updated by dalley over 6 years ago
This exact scheme doesn't work because the tests/pulpcore package mirrors the pulpcore package and attempts to import the code from tests/pulpcore, which fails.
What we really want is just to move the tests from pulpcore/app/tests to pulpcore/tests. That is the proper structure since both pulpcore and pulpcore-plugin are their own separate python packages.
Added by dalley over 6 years ago
Updated by bmbouter over 6 years ago
dalley, Yes that sounds right. Would you want to update the ticket description for posterity?
Updated by dalley over 6 years ago
- Status changed from POST to MODIFIED
- % Done changed from 0 to 100
Applied in changeset pulp|f22615c15e601d2f9630cac0588e8952558ff5aa.
Updated by bmbouter about 5 years ago
- Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Move tests out of pulpcore/app
closes: #3553 https://pulp.plan.io/issues/3553