On Thu, Apr 28, 2022 at 12:53 PM <rsbecker@xxxxxxxxxxxxx> wrote: > /etc/sudoers is not standard although usual. This path should come from a knob somewhere. We can't run this test on our x86 system anyway - no access to root or sudo even though it is installed. correct and note that the test would succeed if the file doesn't exist because what we are really interested on, is to make sure that sudo won't mess with our path when invoking git, and if there is a chance it would (because that setting is enabled in a different file for example) then we will just skip these tests. Obviously the target I had in mind when I built this test was my own workstation and our public CI, but feel free to provide a fixup that would also make it work for your private runs if you are interested in also running this test. > Also, /etc/sudoers is typically secured 0600 so the grep will fail even if is_root passes It won't, because it is running with sudo ;). note that as stated in the commit message, this requires a fairly open sudo setup (like the one github provides with their actions). > - and I'm worried about the portability of is_root, which is mostly Linux. I actually made sure that is_root was posix shell compatible, but got a little carried away when refactoring it to accomodate for reuse; eitherway it is gone in v3. Carlo
