]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
s3:tests: Reformat test_net_rpc_share_allowedusers.sh
authorAndreas Schneider <asn@samba.org>
Fri, 22 Apr 2022 13:34:09 +0000 (15:34 +0200)
committerAndreas Schneider <asn@cryptomilk.org>
Wed, 8 Jun 2022 06:38:30 +0000 (06:38 +0000)
shfmt -f source3/script/| xargs shfmt -w -p -i 0 -fn

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
source3/script/tests/test_net_rpc_share_allowedusers.sh

index d22c758068125f6cf18b124d55c798f2a4265710..33658137a02838eea217fb9ba9d2aaab632d54aa 100755 (executable)
@@ -1,10 +1,10 @@
 #!/bin/sh
 
 if [ $# -lt 4 ]; then
-cat <<EOF
+       cat <<EOF
 Usage: test_net_rpc_share_allowedusers.sh  SERVER USERNAME PASSWORD PREFIX
 EOF
-exit 1;
+       exit 1
 fi
 
 SERVER="$1"
@@ -14,18 +14,18 @@ PREFIX="$4"
 shift 4
 ADDARGS="$*"
 
-incdir=`dirname $0`/../../../testprogs/blackbox
+incdir=$(dirname $0)/../../../testprogs/blackbox
 . $incdir/subunit.sh
 mkdir -p $PREFIX/private
 net=$BINDIR/net
 # Check for the SID for group "Everyone" as a basic test things are working.
-testit_grep "net_usersidlist" '^ S-1-1-0$' $VALGRIND $net usersidlist $ADDARGS || failed=`expr $failed + 1`
+testit_grep "net_usersidlist" '^ S-1-1-0$' $VALGRIND $net usersidlist $ADDARGS || failed=$(expr $failed + 1)
 # Check "print$" share is listed by default.
-testit_grep "net_rpc_share_allowedusers" '^print\$$' $net usersidlist | $VALGRIND $net rpc share allowedusers -S$SERVER -U$USERNAME%$PASSWORD $ADDARGS || failed=`expr $failed + 1`
+testit_grep "net_rpc_share_allowedusers" '^print\$$' $net usersidlist | $VALGRIND $net rpc share allowedusers -S$SERVER -U$USERNAME%$PASSWORD $ADDARGS || failed=$(expr $failed + 1)
 # Check "print$" share is listed if we ask for it.
-testit_grep "net_rpc_share_allowedusers" '^print\$$' $net usersidlist | $VALGRIND $net rpc share allowedusers -S$SERVER -U$USERNAME%$PASSWORD $ADDARGS - 'print$' || failed=`expr $failed + 1`
+testit_grep "net_rpc_share_allowedusers" '^print\$$' $net usersidlist | $VALGRIND $net rpc share allowedusers -S$SERVER -U$USERNAME%$PASSWORD $ADDARGS - 'print$' || failed=$(expr $failed + 1)
 # Check user "user1" is allowed to read share "tmp".
-testit_grep "net_rpc_share_allowedusers" '^ user1$' $net usersidlist | $VALGRIND $net rpc share allowedusers -S$SERVER -U$USERNAME%$PASSWORD $ADDARGS || failed=`expr $failed + 1`
+testit_grep "net_rpc_share_allowedusers" '^ user1$' $net usersidlist | $VALGRIND $net rpc share allowedusers -S$SERVER -U$USERNAME%$PASSWORD $ADDARGS || failed=$(expr $failed + 1)
 #
 # Subtle extra test for bug https://bugzilla.samba.org/show_bug.cgi?id=13992
 #
@@ -36,14 +36,13 @@ testit_grep "net_rpc_share_allowedusers" '^ user1$' $net usersidlist | $VALGRIND
 # user1 to read from it.
 #
 subunit_start_test "net_rpc_share_allowedusers"
-multi_userout=`$net usersidlist | $VALGRIND $net rpc share allowedusers -S$SERVER -U$USERNAME%$PASSWORD $ADDARGS`
-num_matches=`echo "$multi_userout" | grep -c '^ user1$'`
-if [ "$num_matches" -gt "1" ]
-then
+multi_userout=$($net usersidlist | $VALGRIND $net rpc share allowedusers -S$SERVER -U$USERNAME%$PASSWORD $ADDARGS)
+num_matches=$(echo "$multi_userout" | grep -c '^ user1$')
+if [ "$num_matches" -gt "1" ]; then
        subunit_pass_test "net_rpc_share_allowedusers"
 else
        echo "net_rpc_share_allowedusers only found $num_matches shares readable by user1. Should be greater than one.\n"
-       failed=`expr $failed + 1`
+       failed=$(expr $failed + 1)
        echo "$multi_userout" | subunit_fail_test "net_rpc_share_allowedusers"
 fi