-
Notifications
You must be signed in to change notification settings - Fork 123
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move tests to new kickstart fixture #1832
Conversation
Attached issue: https://pulp.plan.io/issues/7040 |
41e0302
to
7e93daa
Compare
7baddec
to
bf1a047
Compare
RPM_PACKAGECATEGORY_CONTENT_NAME: 1, | ||
RPM_PACKAGEENVIRONMENT_CONTENT_NAME: 1, | ||
RPM_PACKAGEGROUP_CONTENT_NAME: 1, | ||
RPM_PACKAGELANGPACKS_CONTENT_NAME: 1, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I decided to not make these integers constants since they don't need to be used elsewhere. Happy to reconsider if anyone disagrees.
Tests are failing as I need to add code to cleanup content. Setting to draft. |
2a9fb7f
to
fc7466c
Compare
I found the cause of these test failures. Turns out orphan cleanup is not removing kickstart content. I filed a bug: https://pulp.plan.io/issues/7440 |
fixes #7040 [nocoverage]
18d9ee5
to
7b58262
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
fixes #7040