s3:tests: Pass down a normal domain user for test_smbget.sh
authorAndreas Schneider <asn@samba.org>
Thu, 7 Dec 2023 08:45:54 +0000 (09:45 +0100)
committerJule Anger <janger@samba.org>
Tue, 12 Dec 2023 09:02:18 +0000 (09:02 +0000)
It is better to test with a normal user than administrator.

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

Signed-off-by: Andreas Schneider <asn@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
(cherry picked from commit 337034e675aaeb366d360a791ec0d003426230af)

source3/script/tests/test_smbget.sh
source3/selftest/tests.py

index 5ab35a03e246e089a0923a75452677f0e1999cec..257291b18ff4d040d288a1b47aca1b62141574c2 100755 (executable)
@@ -16,9 +16,11 @@ DOMAIN=${3}
 REALM=${4}
 USERNAME=${5}
 PASSWORD=${6}
-WORKDIR=${7}
-SMBGET="$VALGRIND ${8}"
-shift 8
+DOMAIN_USER=${7}
+DOMAIN_USER_PASSWORD=${8}
+WORKDIR=${9}
+SMBGET="$VALGRIND ${10}"
+shift 10
 
 TMPDIR="$SELFTEST_TMPDIR"
 
@@ -89,7 +91,7 @@ test_singlefile_U_UPN()
 {
        clear_download_area
 
-       ${SMBGET} --verbose -U"${DC_USERNAME}@${REALM}%${DC_PASSWORD}" \
+       ${SMBGET} --verbose -U"${DOMAIN_USER}@${REALM}%${DOMAIN_USER_PASSWORD}" \
                "smb://${SERVER_IP}/smbget/testfile"
        ret=${?}
        if [ ${ret} -ne 0 ]; then
@@ -111,7 +113,7 @@ test_singlefile_U_domain()
 {
        clear_download_area
 
-       ${SMBGET} --verbose -U"${DOMAIN}/${DC_USERNAME}%${DC_PASSWORD}" \
+       ${SMBGET} --verbose -U"${DOMAIN}/${DOMAIN_USER}%${DOMAIN_USER_PASSWORD}" \
                "smb://${SERVER_IP}/smbget/testfile"
        ret=${?}
        if [ ${ret} -ne 0 ]; then
@@ -132,7 +134,7 @@ test_singlefile_U_domain()
 test_singlefile_smburl()
 {
        clear_download_area
-       $SMBGET --workgroup $DOMAIN smb://${USERNAME}:$PASSWORD@$SERVER_IP/smbget/testfile
+       $SMBGET --workgroup $DOMAIN smb://${DOMAIN_USER}:$DOMAIN_USER_PASSWORD@$SERVER_IP/smbget/testfile
        if [ $? -ne 0 ]; then
                echo 'ERROR: RC does not match, expected: 0'
                return 1
@@ -148,7 +150,7 @@ test_singlefile_smburl()
 test_singlefile_smburl2()
 {
        clear_download_area
-       $SMBGET "smb://$DOMAIN;${USERNAME}:$PASSWORD@$SERVER_IP/smbget/testfile"
+       $SMBGET "smb://$DOMAIN;${DOMAIN_USER}:$DOMAIN_USER_PASSWORD@$SERVER_IP/smbget/testfile"
        if [ $? -ne 0 ]; then
                echo 'ERROR: RC does not match, expected: 0'
                return 1
@@ -343,7 +345,7 @@ test_msdfs_link_domain()
 {
        clear_download_area
 
-       ${SMBGET} --verbose "-U${DOMAIN}/${DC_USERNAME}%${DC_PASSWORD}" \
+       ${SMBGET} --verbose "-U${DOMAIN}/${DOMAIN_USER}%${DOMAIN_USER_PASSWORD}" \
                "smb://${SERVER}/msdfs-share/deeppath/msdfs-src2/readable_file"
        ret=$?
        if [ ${ret} -ne 0 ]; then
@@ -358,7 +360,7 @@ test_msdfs_link_upn()
 {
        clear_download_area
 
-       ${SMBGET} --verbose "-U${DC_USERNAME}@${REALM}%${DC_PASSWORD}" \
+       ${SMBGET} --verbose "-U${DOMAIN_USER}@${REALM}%${DOMAIN_USER_PASSWORD}" \
                "smb://${SERVER}/msdfs-share/deeppath/msdfs-src2/readable_file"
        ret=$?
        if [ ${ret} -ne 0 ]; then
@@ -433,7 +435,7 @@ test_kerberos()
        KRB5CCNAME="FILE:${KRB5CCNAME_PATH}"
        export KRB5CCNAME
        kerberos_kinit "${samba_kinit}" \
-               "${DC_USERNAME}@${REALM}" "${DC_PASSWORD}"
+               "${DOMAIN_USER}@${REALM}" "${DOMAIN_USER_PASSWORD}"
 
        $SMBGET --verbose --use-krb5-ccache="${KRB5CCNAME}" \
                smb://$SERVER/smbget/testfile
index 5a784f1c5aab12828ed66c146ec7731229e7ef7d..973384f8c53b8b65423a18525a441e6613da1e68 100755 (executable)
@@ -931,6 +931,8 @@ plantestsuite("samba3.blackbox.smbget",
                   '$REALM',
                   'smbget_user',
                   '$PASSWORD',
+                  '$DOMAIN_USER',
+                  '$DOMAIN_USER_PASSWORD',
                   '$LOCAL_PATH/smbget',
                   smbget
               ])