389-ds-base

#50127 Ticket 50126 - Incorrect usage of sudo in test
Closed by spichugi. Opened by firstyear.
firstyear/389-ds-base 50126-incorrect-sudo  into  master

Download 50127.patch

Bug Description:

Sudo is assumed to be present and working in passwordless
mode with basictest.

Fix Description:

We can not make this assumption, that sudo is installed (docker)
or in passwordless mode. As a result for this test to work, we should
run the suite as "sudo py.test" or "dirsrv". Potentially we may need
to update defaults.inf for people who want to run the tests as
their own user ID.

https://pagure.io/389-ds-base/issue/50126

Author: William Brown william@blackhats.net.au

Review by: ???

rebased onto 975a57724da5ffba0a486bbdbdf81c72c0b695e9

LGTM

If we can't guarantee something, we should check for it and skip the test if it's not there. See pytest.skipif fixture.

@vashirov I'm going to update this with "skipif uid != 0 || uid != dirsrv".

rebased onto dc7eb6a482d0aac076d1a5be31d5ca01116f0cb3

@mreynolds and @vashirov can you please check this again?

rebased onto ab7c0770d60b6c47b5f81f066daa6a73f46f8828

LGTM

rebased onto 50e290dced417b9537067acb5b4eee2321792b65

Pull-Request has been merged by firstyear

389-ds-base is moving from Pagure to Github. This means that new issues and pull requests
will be accepted only in 389-ds-base's github repository.

This pull request has been cloned to Github as issue and is available here:
- https://github.com/389ds/389-ds-base/issues/3186

If you want to continue to work on the PR, please navigate to the github issue,
download the patch from the attachments and file a new pull request.

Thank you for understanding. We apologize for all inconvenience.

Pull-Request has been closed by spichugi
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.