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

Tests: Fix integration sudo test #208

Closed
wants to merge 1 commit into from
Closed

Conversation

mirielka
Copy link
Contributor

@mirielka mirielka commented Nov 2, 2016

Tests with sudorules using only RunAsGroups attributes with empty RunAsUsers
attribute fail due to different expected value than is really returned. This is
caused by improper behaviour of sudo in versions before 1.8.18 (see [1]), to
which the tests were originally fitted. Changing the expected value to proper
one.

[1] - https://www.sudo.ws/pipermail/sudo-workers/2016-November/001025.html

https://fedorahosted.org/freeipa/ticket/6378

Tests with sudorules using only RunAsGroups attributes with empty RunAsUsers
attribute fail due to different expected value than is really returned. This is
caused by improper behaviour of sudo in versions before 1.8.18 (see [1]), to
which the tests were originally fitted. Changing the expected value to proper
one.

[1] - https://www.sudo.ws/pipermail/sudo-workers/2016-November/001025.html

https://fedorahosted.org/freeipa/ticket/6378
@lslebodn
Copy link
Contributor

lslebodn commented Nov 3, 2016

All versions of Fedora have sudo 1.8.18.
And thank you very much for nice/verbose explanation in commit message

ACK

@MartinBasti MartinBasti added the ack Pull Request approved, can be merged label Nov 3, 2016
@MartinBasti MartinBasti added the pushed Pull Request has already been pushed label Nov 7, 2016
@MartinBasti MartinBasti closed this Nov 7, 2016
@mirielka mirielka deleted the sudo branch November 9, 2016 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ack Pull Request approved, can be merged pushed Pull Request has already been pushed
Projects
None yet
3 participants