svn commit: r275657 - head/usr.sbin/pw/tests

[NGie Cooper]

Hi Brad!

On Tue, Dec 9, 2014 at 2:11 PM, Brad Davis <brd at freebsd.org> wrote:
> Author: brd (doc committer)
> Date: Tue Dec  9 22:11:56 2014
> New Revision: 275657
> URL: https://svnweb.freebsd.org/changeset/base/275657
>
> Log:
>   Add some tests for user modification. [1]
>   Fix a missing test in the Makefile from my previous commit.
>
>   PR:           195834 [1]
>   Submitted by: Robert O'Neil <oneil.rs at gmail.com>
>   Approved by:  will
>
> Added:
>   head/usr.sbin/pw/tests/pw_usermod.sh   (contents, props changed)
> Modified:
>   head/usr.sbin/pw/tests/Makefile
>
> Modified: head/usr.sbin/pw/tests/Makefile
> ==============================================================================
> --- head/usr.sbin/pw/tests/Makefile     Tue Dec  9 21:43:03 2014        (r275656)
> +++ head/usr.sbin/pw/tests/Makefile     Tue Dec  9 22:11:56 2014        (r275657)
> @@ -5,11 +5,13 @@ TESTSRC=      ${.CURDIR}/../../../contrib/net
>
>  TESTSDIR=      ${TESTSBASE}/usr.sbin/pw
>
> -ATF_TESTS_SH=  pw_etcdir pw_lock pw_groupmod pw_useradd pw_userdel
> +ATF_TESTS_SH=  pw_etcdir pw_lock pw_groupdel pw_groupmod pw_useradd pw_userdel pw_usermod

Please put this one item per-line so it's easier to backport and
doesn't exceed 80 columns.

> +TEST_METADATA.pw_groupdel+=    required_user="root"
>  TEST_METADATA.pw_groupmod+=    required_user="root"
>  TEST_METADATA.pw_useradd+=     required_user="root"
>  TEST_METADATA.pw_userdel+=     required_user="root"
> +TEST_METADATA.pw_usermod+=     required_user="root"

This should probably just be a for-loop, e.g.

.for tp in ${ATF_TESTS_SH}
TEST_METADATA.${tp}+= required_user="root"
.endif