Skip to content

Fix the tests#441

Draft
adamruzicka wants to merge 1 commit intotheforeman:masterfrom
adamruzicka:fix-test
Draft

Fix the tests#441
adamruzicka wants to merge 1 commit intotheforeman:masterfrom
adamruzicka:fix-test

Conversation

@adamruzicka
Copy link
Contributor

This most likely broke when theforeman/foreman#10701 went in.

result += untaxed_environments
end
result
ForemanPuppet::Environment.authorized(:view_environments).pluck(:name)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This currently requires theforeman/foreman#10898 . An alternative would be this suggestion

Suggested change
ForemanPuppet::Environment.authorized(:view_environments).pluck(:name)
ForemanPuppet::Environment.authorized(:view_environments).distinct.pluck(:name)

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.

1 participant