selftest: fix cleanup of test_printing_var_exp.sh

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14668

Signed-off-by: Ralph Boehme <slow@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>

Autobuild-User(master): Jeremy Allison <jra@samba.org>
Autobuild-Date(master): Tue Mar 16 22:08:10 UTC 2021 on sn-devel-184

source3/script/tests/printing/printing_var_exp_lpr_cmd.sh

@@ -1,7 +1,9 @@
 #!/bin/bash
 
-rm -f /tmp/printing_var_exp.log
+logfile=/tmp/$USER_printing_var_exp.log
+
+rm -f "$logfile"
 
 for i in $(seq 1 $#) ; do
-    eval echo "arg $i: \$$i" >> /tmp/printing_var_exp.log
+    eval echo "arg $i: \$$i" >> "$logfile"
 done

source3/script/tests/test_printing_var_exp.sh

@@ -22,15 +22,35 @@ incdir=`dirname $0`/../../../testprogs/blackbox
 smbclient="$BINDIR/smbclient"
 
 test_var_expansion() {
+	logfile=/tmp/$USER_printing_var_exp.log
+
 	$smbclient -U $DOMAIN/$USERNAME%$PASSWORD \
 		   //$SERVER_IP/print_var_exp \
-		   -c "print $SRCDIR/testdata/printing/example.ps" || return 1
+		   -c "print $SRCDIR/testdata/printing/example.ps"
+	if [ $? -ne 0 ] ; then
+	    rm -f "$logfile"
+	    return 1
+	fi
+	cat "$logfile"
 
-	cat /tmp/printing_var_exp.log
+	grep "Windows user: $USERNAME" "$logfile"
+	if [ $? -ne 0 ] ; then
+	    rm -f "$logfile"
+	    return 1
+	fi
+	grep "UNIX user: $USERNAME" "$logfile"
+	if [ $? -ne 0 ] ; then
+	    rm -f "$logfile"
+	    return 1
+	fi
+	grep "Domain: $DOMAIN" "$logfile"
+	if [ $? -ne 0 ] ; then
+	    rm -f "$logfile"
+	    return 1
+	fi
 
-	grep "Windows user: $USERNAME" /tmp/printing_var_exp.log || return 1
-	grep "UNIX user: $USERNAME" /tmp/printing_var_exp.log || return 1
-	grep "Domain: $DOMAIN" /tmp/printing_var_exp.log || return 1
+	rm -f "$logfile"
+	return 0
 }
 
 testit "Test variable expansion for '%U', '%u' and '%D'" \