Skip to content
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

Updates test to handle changes to Distribution CUD operations #1295

Merged
merged 1 commit into from Mar 12, 2019

Conversation

CodeHeeler
Copy link
Contributor

Create, update, and delete for Distributions were made asynchronous in:
pulp/pulpcore#6
in response to issue 3044:
https://pulp.plan.io/issues/3044
Because of these changes, tests that create distributions previously expected
this action to return a distribution but now receive a task so additional steps
were needed to retrieve the distribution from the created_resources of the task

Required PR: pulp/pulpcore#6

ref #3044
https://pulp.plan.io/issues/3044

@CodeHeeler CodeHeeler requested a review from a team March 6, 2019 21:06
@nixocio
Copy link

nixocio commented Mar 7, 2019

@CodeHeeler, test still failing due to the changes.

Create, update, and delete for Distributions were made asynchronous in:
pulp/pulpcore#6
in response to issue 3044:
https://pulp.plan.io/issues/3044
Because of these changes, tests that create distributions previously expected
this action to return a distribution but now receive a task so additional steps
were needed to retrieve the distribution from the created_resources of the task

ref pulp#3044
https://pulp.plan.io/issues/3044

Required PR: pulp/pulpcore#6
@goosemania
Copy link
Member

Merging since QE approved and changes are only in tests.

@goosemania goosemania merged commit dec1a65 into pulp:master Mar 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants