Re: [PATCH 2/2] nfs-utils: add initial tests for statd that run via "make check"

[Steve Dickson <SteveD@redhat.com>]

On 12/16/2009 02:40 PM, Jeff Layton wrote:
> Leverage the support that automake already has for running tests via
> make check. Add a simple test that just checks that the statd mon and
> unmon calls actually work.
> 
> Adding more tests should be a simple matter of adding new scripts
> exit 0 on success and non-zero on fail, and adding those to the
> Makefile.am.
> 
> Signed-off-by: Jeff Layton <jlayton@redhat.com>
> ---
>  tests/Makefile.am                    |    2 +
>  tests/t0001-statd-basic-mon-unmon.sh |   55 ++++++++++++++++++++++++++++++++++
>  tests/test-lib.sh                    |   41 +++++++++++++++++++++++++
>  3 files changed, 98 insertions(+), 0 deletions(-)
>  create mode 100755 tests/t0001-statd-basic-mon-unmon.sh
>  create mode 100755 tests/test-lib.sh
> 
> diff --git a/tests/Makefile.am b/tests/Makefile.am
> index 997ac51..634fb7b 100644
> --- a/tests/Makefile.am
> +++ b/tests/Makefile.am
> @@ -3,3 +3,5 @@
>  SUBDIRS = statdtest
>  
>  MAINTAINERCLEANFILES = Makefile.in
> +
> +TESTS = t0001-statd-basic-mon-unmon.sh
> diff --git a/tests/t0001-statd-basic-mon-unmon.sh b/tests/t0001-statd-basic-mon-unmon.sh
> new file mode 100755
> index 0000000..d5dffc5
> --- /dev/null
> +++ b/tests/t0001-statd-basic-mon-unmon.sh
> @@ -0,0 +1,55 @@
> +#!/bin/bash
> +#
> +# statd_basic_mon_unmon -- test basic mon/unmon functionality with statd
> +#
> +# Copyright (C) 2009  Red Hat, Jeff Layton <jlayton@redhat.com>
> +#
> +# This program is free software; you can redistribute it and/or
> +# modify it under the terms of the GNU General Public License
> +# as published by the Free Software Foundation; either version 2
> +# of the License, or (at your option) any later version.
> +#
> +# This program is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> +# GNU General Public License for more details.
> +#
> +# You should have received a copy of the GNU General Public License
> +# along with this program; if not, write to the Free Software Foundation, Inc.,
> +# 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
> +#
> +
> +. ./test-lib.sh
> +
> +start_statd
> +if [ $? -ne 0 ]; then
> +	echo "FAIL: problem starting statd"
> +	exit 1
> +fi
> +
> +COOKIE=`echo $$ | md5sum | cut -d' ' -f1`
> +MON_NAME=`hostname`
> +
> +statdtest mon $MON_NAME $COOKIE
> +if [ $? -ne 0 ]; then
> +	echo "FAIL: mon failed"
> +	kill_statd
> +	exit 1
> +fi
> +
> +grep -q $COOKIE /var/lib/nfs/statd/sm/$MON_NAME
This is a distro specific path... The default is /var/lib/nfs/sm
It would be nice if somehow we could pull in the NFS_STATEDIR define
from support/include/config.h file... 

> +if [ $? -ne 0 ]; then
> +	echo "FAIL: $COOKIE not present in monitor file"
> +	kill_statd
> +	exit 1
> +fi
> +
> +statdtest unmon $MON_NAME
> +if [ $? -ne 0 ]; then
> +	echo "FAIL: unmon failed"
> +	kill_statd
> +	exit 1
> +fi
> +
> +kill_statd
> +
> diff --git a/tests/test-lib.sh b/tests/test-lib.sh
> new file mode 100755
> index 0000000..053e46d
> --- /dev/null
> +++ b/tests/test-lib.sh
> @@ -0,0 +1,41 @@
> +#!/bin/bash
> +#
> +# test-lib.sh -- library of functions for nfs-utils tests
> +#
> +# Copyright (C) 2009  Red Hat, Jeff Layton <jlayton@redhat.com>
> +#
> +# This program is free software; you can redistribute it and/or
> +# modify it under the terms of the GNU General Public License
> +# as published by the Free Software Foundation; either version 2
> +# of the License, or (at your option) any later version.
> +#
> +# This program is distributed in the hope that it will be useful,
> +# but WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
> +# GNU General Public License for more details.
> +#
> +# You should have received a copy of the GNU General Public License
> +# along with this program; if not, write to the Free Software Foundation, Inc.,
> +# 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
> +#
> +
> +export PATH=$PATH:$srcdir/statdtest
> +
> +lockd_registered() {
> +	rpcinfo -p | grep -q nlockmgr
> +	return $?
> +}
> +
> +start_statd() {
> +	rpcinfo -u 127.0.0.1 status 1 &> /dev/null
> +	if [ $? -eq 0 ]; then
> +		echo "***ERROR***: statd is already running and should "
> +		echo "             be down when starting this test"
> +		return 1
> +	fi
> +	$srcdir/../utils/statd/statd --no-notify
the $srcdir variable is not define so the script fails
right out of the box... So maybe something like this...
Initialize srcdir=`dirname $PWD`
then '$srcdir/utils/statd/statd --no-notify' will work.

> +}
> +
> +kill_statd() {
> +	kill `cat /var/run/rpc.statd.pid`
> +}

Overall I like the idea of having a test harness... but I have
to wonder if we start building the harness in shell scripts 
if we are not limiting things right out of the box... 

Maybe we should consider doing the harness in a more up to date
scripting languages like perl or python or even java... Granted 
I'm not as familiar (or comfortable) with those languages as with
shell scripting. Plus its not clear what type of dependencies that
would added to the core nfs-utils code, but we should at least
think about using a language that might give us more 
possibilities down the road... Who knowns, in the end, shell scripting
might be the best way to go... 

steved.