The branch, master has been updated
       via  8ad127e s3:tests: fix the smbclient_s3 test for multiple users 
running it.
       via  0749f7a s3:tests: remove unused script/tests/test_functions.sh
       via  93ca1bc s3:tests: include subunit.sh instead of test_functions.sh 
in test_posix_s3.sh
       via  d03996a s3:tests: include subunit.sh instead of test_functions.sh 
in test_ntlm_auth_s3.sh
       via  6af6241 s3:tests: include subunit.sh instead of test_functions.sh 
in test_smbtorture_s3.sh
       via  4eb3da2 s3:tests: include subunit.sh instead of test_functions.sh 
in test_wbinfo_s3.sh
       via  ae9aa47 s3:tests: use the blackbox subunit output formatter for the 
testparm_s3 test.
       via  a1bec01 s3:tests: include subunit.sh instead of test_functions.sh 
in test_local_s3.sh
       via  745354e s3:tests: use the blackbox subunit output formatter for the 
smbclient_s3 test.
      from  9971061 s3: Pass logonHours through pdb_ads

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -----------------------------------------------------------------
commit 8ad127e187abb8f9e8076a872e08faaf4fbc1367
Author: Michael Adam <ob...@samba.org>
Date:   Sat Feb 19 11:49:43 2011 +0100

    s3:tests: fix the smbclient_s3 test for multiple users running it.
    
    The test created /tmp/log.smbclient, which lets the test fail if
    two users are running the test simultaneously.
    
    Now /tmp/log.test_smbclient_s3.$$ is used.
    
    Autobuild-User: Michael Adam <ob...@samba.org>
    Autobuild-Date: Sat Feb 19 13:06:57 CET 2011 on sn-devel-104

commit 0749f7a976fd1766bf076986f1b5e7eff323e5e0
Author: Michael Adam <ob...@samba.org>
Date:   Sat Feb 19 02:29:54 2011 +0100

    s3:tests: remove unused script/tests/test_functions.sh
    
    This is not used any more now with selftest.

commit 93ca1bc410b25492f02045f77bcd8d7d5bbfafe6
Author: Michael Adam <ob...@samba.org>
Date:   Sat Feb 19 01:14:54 2011 +0100

    s3:tests: include subunit.sh instead of test_functions.sh in 
test_posix_s3.sh
    
    It would actually not be necessary at all to include a script providing
    testit() and so on, since this script snippet is currently only sourced
    in tests.sh to plan the included test.

commit d03996afb50a22b24119cc4a6380c6923bcda997
Author: Michael Adam <ob...@samba.org>
Date:   Sat Feb 19 01:14:54 2011 +0100

    s3:tests: include subunit.sh instead of test_functions.sh in 
test_ntlm_auth_s3.sh
    
    It would actually not be necessary at all to include a script providing
    testit() and so on, since this script snippet is currently only sourced
    in tests.sh to plan the included test.

commit 6af6241040d376315f47716b4df7ca843b918d2d
Author: Michael Adam <ob...@samba.org>
Date:   Sat Feb 19 01:14:54 2011 +0100

    s3:tests: include subunit.sh instead of test_functions.sh in 
test_smbtorture_s3.sh
    
    It would actually not be necessary at all to include a script providing
    testit() and so on, since this script snippet is currently only sourced
    in tests.sh to plan the included test.

commit 4eb3da2b0ece1c4d33c679c39f2beff0f2a324da
Author: Michael Adam <ob...@samba.org>
Date:   Sat Feb 19 01:14:54 2011 +0100

    s3:tests: include subunit.sh instead of test_functions.sh in 
test_wbinfo_s3.sh
    
    It would actually not be necessary at all to include a script providing
    testit() and so on, since this script snippet is currently only sourced
    in tests.sh to plan the included test.

commit ae9aa47d6bcfa4ececdc1c74695c4c9a6c045ab6
Author: Michael Adam <ob...@samba.org>
Date:   Sat Feb 19 00:32:06 2011 +0100

    s3:tests: use the blackbox subunit output formatter for the testparm_s3 
test.
    
    With the old output, failures would go unnoticed by selftest.

commit a1bec01eeb660e8ff00d77da185f81815be94c30
Author: Michael Adam <ob...@samba.org>
Date:   Sat Feb 19 01:14:54 2011 +0100

    s3:tests: include subunit.sh instead of test_functions.sh in 
test_local_s3.sh
    
    It would actually not be necessary at all to include a script providing
    testit() and so on, since this script snippet is currently only sourced
    in tests.sh to plan the included test.

commit 745354eab801ab08d161b7c5ee0142c82c1061b2
Author: Michael Adam <ob...@samba.org>
Date:   Sat Feb 19 00:32:06 2011 +0100

    s3:tests: use the blackbox subunit output formatter for the smbclient_s3 
test.
    
    With the old output, failures would go unnoticed by selftest.

-----------------------------------------------------------------------

Summary of changes:
 source3/script/tests/test_functions.sh     |  202 ----------------------------
 source3/script/tests/test_local_s3.sh      |    4 +-
 source3/script/tests/test_ntlm_auth_s3.sh  |    4 +-
 source3/script/tests/test_posix_s3.sh      |    4 +-
 source3/script/tests/test_smbclient_s3.sh  |   22 +++-
 source3/script/tests/test_smbtorture_s3.sh |    4 +-
 source3/script/tests/test_testparm_s3.sh   |    4 +-
 source3/script/tests/test_wbinfo_s3.sh     |    4 +-
 8 files changed, 30 insertions(+), 218 deletions(-)
 delete mode 100644 source3/script/tests/test_functions.sh


Changeset truncated at 500 lines:

diff --git a/source3/script/tests/test_functions.sh 
b/source3/script/tests/test_functions.sh
deleted file mode 100644
index 4e80266..0000000
--- a/source3/script/tests/test_functions.sh
+++ /dev/null
@@ -1,202 +0,0 @@
-TEST_FUNCTIONS_SH="INCLUDED"
-
-samba3_stop_sig_term() {
-       RET=0
-       kill -USR1 `cat $PIDDIR/timelimit.nmbd.pid` >/dev/null 2>&1 || \
-               kill -ALRM `cat $PIDDIR/timelimit.nmbd.pid` || RET=$?
-
-       kill -USR1 `cat $PIDDIR/timelimit.winbindd.pid` >/dev/null 2>&1 || \
-               kill -ALRM `cat $PIDDIR/timelimit.winbindd.pid` || RET=$?
-
-       kill -USR1 `cat $PIDDIR/timelimit.smbd.pid` >/dev/null 2>&1 || \
-               kill -ALRM `cat $PIDDIR/timelimit.smbd.pid` || RET=$?
-
-       return $RET;
-}
-
-samba3_stop_sig_kill() {
-       kill -ALRM `cat $PIDDIR/timelimit.nmbd.pid` >/dev/null 2>&1
-       kill -ALRM `cat $PIDDIR/timelimit.winbindd.pid` >/dev/null 2>&1
-       kill -ALRM `cat $PIDDIR/timelimit.smbd.pid` >/dev/null 2>&1
-       return 0;
-}
-
-samba3_nmbd_test_log() {
-       if [ -n "$NMBD_TEST_LOG" ];then
-               if [ -r "$NMBD_TEST_LOG" ];then
-                       return 0;
-               fi
-       fi
-       return 1;
-}
-
-samba3_winbindd_test_log() {
-       if [ -n "$WINBINDD_TEST_LOG" ];then
-               if [ -r "$WINBINDD_TEST_LOG" ];then
-                       return 0;
-               fi
-       fi
-       return 1;
-}
-
-samba3_smbd_test_log() {
-       if [ -n "$SMBD_TEST_LOG" ];then
-               if [ -r "$SMBD_TEST_LOG" ];then
-                       return 0;
-               fi
-       fi
-       return 1;
-}
-
-samba3_check_only() {
-       if [ -n "$SERVER_TEST_FIFO" ];then
-               if [ -p "$SERVER_TEST_FIFO" ];then
-                       return 0;
-               fi
-               return 1;
-       fi
-       return 0;
-}
-
-testit() {
-       if [ -z "$PREFIX" ]; then
-           PREFIX=test_prefix
-           mkdir -p $PREFIX
-       fi
-       name=$1
-       shift 1
-       binary=$1
-       cmdline="$*"
-
-       SERVERS_ARE_UP="no"
-
-       shname=`echo $name | \
-       sed -e 
's%[^abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789\-]%_%g'`
-
-       UNIQUE_PID=`/bin/sh -c 'echo $$'`
-       TEST_LOG="$PREFIX/test_log.${UNIQUE_PID}"
-       TEST_PCAP="$PREFIX/test_${shname}_${UNIQUE_PID}.pcap"
-       trap "rm -f $TEST_LOG $TEST_PCAP" EXIT
-
-       samba3_nmbd_test_log && if [ -z "$nmbd_log_size" ]; then
-               nmbd_log_size=`wc -l < $NMBD_TEST_LOG`;
-       fi
-       samba3_winbindd_test_log && if [ -z "$winbindd_log_size" ]; then
-               winbindd_log_size=`wc -l < $WINBINDD_TEST_LOG`;
-       fi
-       samba3_smbd_test_log && if [ -z "$smbd_log_size" ]; then
-               smbd_log_size=`wc -l < $SMBD_TEST_LOG`;
-       fi
-
-       if [ x"$RUN_FROM_BUILD_FARM" = x"yes" ];then
-               echo "--==--==--==--==--==--==--==--==--==--==--"
-               echo "Running test $name (level 0 stdout)"
-               echo "--==--==--==--==--==--==--==--==--==--==--"
-               date
-               echo "Testing $name"
-       else
-               echo "Testing $name ($failed)"
-       fi
-
-       samba3_check_only && SERVERS_ARE_UP="yes"
-       if [ x"$SERVERS_ARE_UP" != x"yes" ];then
-               if [ x"$RUN_FROM_BUILD_FARM" = x"yes" ];then
-                       echo "SERVERS are down! Skipping: $cmdline"
-                       echo "=========================================="
-                       echo "TEST SKIPPED: $name (reason SERVERS are down)"
-                       echo "=========================================="
-               else
-                       echo "TEST SKIPPED: $name (reason SERVERS are down)"
-               fi
-               return 1
-       fi
-
-       if [ x"$MAKE_TEST_ENABLE_PCAP" = x"yes" ];then
-               SOCKET_WRAPPER_PCAP_FILE=$TEST_PCAP
-               export SOCKET_WRAPPER_PCAP_FILE
-       fi
-
-       MAKE_TEST_BINARY=$binary
-       export MAKE_TEST_BINARY
-       ( $cmdline > $TEST_LOG 2>&1 )
-       status=$?
-       MAKE_TEST_BINARY=
-       # show any additional output from smbd that has happened in this test
-       samba3_nmbd_test_log && {
-               new_log_size=`wc -l < $NMBD_TEST_LOG`;
-               test "$new_log_size" = "$nmbd_log_size" || {
-                       echo "NMBD OUTPUT:";
-                       incr_log_size=`expr $new_log_size - $nmbd_log_size`;
-                       tail -$incr_log_size $NMBD_TEST_LOG;
-                       nmbd_log_size=$new_log_size;
-               }
-       }
-       samba3_winbindd_test_log && {
-               new_log_size=`wc -l < $WINBINDD_TEST_LOG`;
-               test "$new_log_size" = "$winbindd_log_size" || {
-                       echo "WINBINDD OUTPUT:";
-                       incr_log_size=`expr $new_log_size - $winbindd_log_size`;
-                       tail -$incr_log_size $WINBINDD_TEST_LOG;
-                       winbindd_log_size=$new_log_size;
-               }
-       }
-       samba3_smbd_test_log && {
-               new_log_size=`wc -l < $SMBD_TEST_LOG`;
-               test "$new_log_size" = "$smbd_log_size" || {
-                       echo "SMBD OUTPUT:";
-                       incr_log_size=`expr $new_log_size - $smbd_log_size`;
-                       tail -$incr_log_size $SMBD_TEST_LOG;
-                       smbd_log_size=$new_log_size;
-               }
-       }
-
-       if [ x"$status" != x"0" ]; then
-               echo "TEST OUTPUT:"
-               cat $TEST_LOG;
-               rm -f $TEST_LOG;
-               if [ x"$MAKE_TEST_ENABLE_PCAP" = x"yes" ];then
-                       echo "TEST PCAP: $TEST_PCAP"
-               fi
-               if [ x"$RUN_FROM_BUILD_FARM" = x"yes" ];then
-                       echo "=========================================="
-                       echo "TEST FAILED: $name (status $status)"
-                       echo "=========================================="
-               else
-                       echo "TEST FAILED: $cmdline (status $status)"
-               fi
-               trap "" EXIT
-               return 1;
-       fi
-       rm -f $TEST_LOG;
-       if [ x"$MAKE_TEST_KEEP_PCAP" = x"yes" ];then
-               echo "TEST PCAP: $TEST_PCAP"
-       else
-               rm -f $TEST_PCAP;
-       fi
-       if [ x"$RUN_FROM_BUILD_FARM" = x"yes" ];then
-               echo "ALL OK: $cmdline"
-               echo "=========================================="
-               echo "TEST PASSED: $name"
-               echo "=========================================="
-       fi
-       trap "" EXIT
-       return 0;
-}
-
-testok() {
-       name=`basename $1`
-       failed=$2
-
-       if [ x"$failed" = x"0" ];then
-               :
-       else
-               echo "$failed TESTS FAILED or SKIPPED ($name)";
-       fi
-       exit $failed
-}
-
-if [ -z "$VALGRIND" ]; then
-    MALLOC_CHECK_=2
-    export MALLOC_CHECK_
-fi
-
diff --git a/source3/script/tests/test_local_s3.sh 
b/source3/script/tests/test_local_s3.sh
index 0c94871..de867c9 100755
--- a/source3/script/tests/test_local_s3.sh
+++ b/source3/script/tests/test_local_s3.sh
@@ -10,8 +10,8 @@ exit 1;
 fi
 
 test x"$TEST_FUNCTIONS_SH" != x"INCLUDED" && {
-incdir=`dirname $0`
-. $incdir/test_functions.sh
+incdir=`dirname $0`/../../../testprogs/blackbox
+. $incdir/subunit.sh
 }
 
 failed=0
diff --git a/source3/script/tests/test_ntlm_auth_s3.sh 
b/source3/script/tests/test_ntlm_auth_s3.sh
index c7ebc03..17aabef 100755
--- a/source3/script/tests/test_ntlm_auth_s3.sh
+++ b/source3/script/tests/test_ntlm_auth_s3.sh
@@ -1,8 +1,8 @@
 #!/bin/sh
 
 test x"$TEST_FUNCTIONS_SH" != x"INCLUDED" && {
-incdir=`dirname $0`
-. $incdir/test_functions.sh
+incdir=`dirname $0`/../../../testprogs/blackbox
+. $incdir/subunit.sh
 }
 
 failed=0
diff --git a/source3/script/tests/test_posix_s3.sh 
b/source3/script/tests/test_posix_s3.sh
index 8cdb1e2..904f9f1 100755
--- a/source3/script/tests/test_posix_s3.sh
+++ b/source3/script/tests/test_posix_s3.sh
@@ -17,8 +17,8 @@ shift 4
 ADDARGS="$*"
 
 test x"$TEST_FUNCTIONS_SH" != x"INCLUDED" && {
-incdir=`dirname $0`
-. $incdir/test_functions.sh
+incdir=`dirname $0`/../../../testprogs/blackbox
+. $incdir/subunit.sh
 }
 
 base="base.attr base.charset base.chkpath base.defer_open base.delaywrite 
base.delete"
diff --git a/source3/script/tests/test_smbclient_s3.sh 
b/source3/script/tests/test_smbclient_s3.sh
index a567274..08ed3b7 100755
--- a/source3/script/tests/test_smbclient_s3.sh
+++ b/source3/script/tests/test_smbclient_s3.sh
@@ -21,8 +21,8 @@ shift 6
 ADDARGS="$*"
 
 test x"$TEST_FUNCTIONS_SH" != x"INCLUDED" && {
-incdir=`dirname $0`
-. $incdir/test_functions.sh
+incdir=`dirname $0`/../../../testprogs/blackbox
+. $incdir/subunit.sh
 }
 
 failed=0
@@ -389,6 +389,16 @@ test_ccache_access()
     $WBINFO --logoff
 }
 
+LOGDIR=/tmp/log.test_smbclient_s3.$$
+
+testit "rm -rf $LOGDIR" \
+    rm -rf $LOGDIR || \
+    exit 1
+
+testit "mkdir -p $LOGDIR" \
+    mkdir -p $LOGDIR || \
+    exit 1
+
 
 testit "smbclient -L $SERVER_IP" $SMBCLIENT $CONFIGURATION -L $SERVER_IP -N -p 
139 || failed=`expr $failed + 1`
 testit "smbclient -L $SERVER -I $SERVER_IP" $SMBCLIENT $CONFIGURATION -L 
$SERVER -I $SERVER_IP -N -p 139 || failed=`expr $failed + 1`
@@ -398,7 +408,7 @@ testit "noninteractive smbclient does not prompt" \
     failed=`expr $failed + 1`
 
 testit "noninteractive smbclient -l does not prompt" \
-   test_noninteractive_no_prompt -l /tmp || \
+   test_noninteractive_no_prompt -l $LOGDIR || \
     failed=`expr $failed + 1`
 
 testit "interactive smbclient prompts on stdout" \
@@ -406,7 +416,7 @@ testit "interactive smbclient prompts on stdout" \
     failed=`expr $failed + 1`
 
 testit "interactive smbclient -l prompts on stdout" \
-   test_interactive_prompt_stdout -l /tmp || \
+   test_interactive_prompt_stdout -l $LOGDIR || \
     failed=`expr $failed + 1`
 
 testit "creating a bad symlink and deleting it" \
@@ -433,4 +443,8 @@ testit "ccache access works for smbclient" \
     test_ccache_access || \
     failed=`expr $failed + 1`
 
+testit "rm -rf $LOGDIR" \
+    rm -rf $LOGDIR || \
+    failed=`expr $failed + 1`
+
 testok $0 $failed
diff --git a/source3/script/tests/test_smbtorture_s3.sh 
b/source3/script/tests/test_smbtorture_s3.sh
index 77ffc0c..c9466e9 100755
--- a/source3/script/tests/test_smbtorture_s3.sh
+++ b/source3/script/tests/test_smbtorture_s3.sh
@@ -17,8 +17,8 @@ shift 4
 ADDARGS="$*"
 
 test x"$TEST_FUNCTIONS_SH" != x"INCLUDED" && {
-incdir=`dirname $0`
-. $incdir/test_functions.sh
+incdir=`dirname $0`/../../../testprogs/blackbox
+. $incdir/subunit.sh
 }
 
 SMB_CONF_PATH="$CONFFILE"
diff --git a/source3/script/tests/test_testparm_s3.sh 
b/source3/script/tests/test_testparm_s3.sh
index b9e4b81..0fd1a57 100755
--- a/source3/script/tests/test_testparm_s3.sh
+++ b/source3/script/tests/test_testparm_s3.sh
@@ -9,8 +9,8 @@ TEMP_CONFFILE=${LOCAL_PATH}/smb.conf.tmp
 TESTPARM="$VALGRIND ${TESTPARM:-$BINDIR/testparm} --suppress-prompt 
--skip-logic-checks"
 
 test x"$TEST_FUNCTIONS_SH" != x"INCLUDED" && {
-incdir=`dirname $0`
-. $incdir/test_functions.sh
+incdir=`dirname $0`/../../../testprogs/blackbox
+. $incdir/subunit.sh
 }
 
 failed=0
diff --git a/source3/script/tests/test_wbinfo_s3.sh 
b/source3/script/tests/test_wbinfo_s3.sh
index d501311..a63bcba 100755
--- a/source3/script/tests/test_wbinfo_s3.sh
+++ b/source3/script/tests/test_wbinfo_s3.sh
@@ -15,8 +15,8 @@ shift 4
 ADDARGS="$*"
 
 test x"$TEST_FUNCTIONS_SH" != x"INCLUDED" && {
-incdir=`dirname $0`
-. $incdir/test_functions.sh
+incdir=`dirname $0`/../../../testprogs/blackbox
+. $incdir/subunit.sh
 }
 
 OLDIFS=$IFS;


-- 
Samba Shared Repository

Reply via email to