[Buildroot] [PATCH] support/testing: add sudo package test

Arnout Vandecappelle arnout at mind.be
Tue May 11 07:17:57 UTC 2021



On 10/05/2021 22:14, Peter Korsgaard wrote:
>>>>>> "Arnout" == Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be> writes:
> 
>  > Create a new user 'sudotest' to validate that sudo really works (i.e.
>  > properly has setuid).
> 
>  > Creating the user and adding it to sudoers is done at runtime, otherwise
>  > we'd need to add extra files to the config which complicates things a
>  > little bit.
> 
>  > Signed-off-by: Arnout Vandecappelle (Essensium/Mind) <arnout at mind.be>
>  > ---
>  >  support/testing/tests/package/test_sudo.py | 36 ++++++++++++++++++++++
>  >  1 file changed, 36 insertions(+)
>  >  create mode 100644 support/testing/tests/package/test_sudo.py
> 
>  > diff --git a/support/testing/tests/package/test_sudo.py b/support/testing/tests/package/test_sudo.py
>  > new file mode 100644
>  > index 0000000000..f9d0a6f301
>  > --- /dev/null
>  > +++ b/support/testing/tests/package/test_sudo.py
>  > @@ -0,0 +1,36 @@
>  > +import os
>  > +
>  > +import infra.basetest
>  > +
>  > +
>  > +class TestSudo(infra.basetest.BRTest):
>  > +    config = infra.basetest.BASIC_TOOLCHAIN_CONFIG + \
>  > +        """
>  > +        BR2_PACKAGE_SUDO=y
>  > +        BR2_TARGET_ROOTFS_CPIO=y
>  > +        # BR2_TARGET_ROOTFS_TAR is not set
>  > +        """
> 
> initrd, so that implies writable rootfs - So we don't really need those
> special adduser flags, but OK.

 I'm not sure what you mean. Do you mean the flags to use /tmp as a home
directory? There was a reason I added those for a reason I think, but I forgot
why :-(

 Regards,
 Arnout

> 
> Committed, thanks.
> 



More information about the buildroot mailing list