]> git.ipfire.org Git - thirdparty/samba.git/commitdiff
selftest: Consolidate planning libsmbclient tests to source4
authorVolker Lendecke <vl@samba.org>
Sun, 23 Feb 2020 11:22:47 +0000 (12:22 +0100)
committerJeremy Allison <jra@samba.org>
Tue, 25 Feb 2020 17:44:45 +0000 (17:44 +0000)
We don't need to plan those tests both in source3 and source4's
tests.py. The source4 version seems more "advanced", it uses
smbtorture4_testsuites() to list all tests, making it less
error-prone, and it also does the tests for all SMB1 and SMB3

Signed-off-by: Volker Lendecke <vl@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
source3/selftest/tests.py

index 6060cec49c36327085548dc8c3548c33be873d2b..041bcf94e031b557594785700f3adcf43c6ec58c 100755 (executable)
@@ -539,13 +539,6 @@ unix = ["unix.info2", "unix.whoami"]
 
 nbt = ["nbt.dgram"]
 
-libsmbclient = ["libsmbclient.version", "libsmbclient.initialize",
-                "libsmbclient.configuration", "libsmbclient.setConfiguration",
-                "libsmbclient.options", "libsmbclient.opendir",
-                "libsmbclient.list_shares", "libsmbclient.readdirplus",
-               "libsmbclient.readdirplus_seek",
-               "libsmbclient.readdirplus2"]
-
 vfs = [
     "vfs.fruit",
     "vfs.acl_xattr",
@@ -556,7 +549,7 @@ vfs = [
     "vfs.unfruit",
 ]
 
-tests = base + raw + smb2 + rpc + unix + local + rap + nbt + libsmbclient + idmap + vfs
+tests = base + raw + smb2 + rpc + unix + local + rap + nbt + idmap + vfs
 
 for t in tests:
     if t == "base.delaywrite" or t == "base.deny1" or t == "base.deny2":
@@ -729,17 +722,6 @@ for t in tests:
     elif t == "rpc.samba3.netlogon" or t == "rpc.samba3.sessionkey":
         plansmbtorture4testsuite(t, "nt4_dc", '//$SERVER_IP/tmp -U$USERNAME%$PASSWORD --option=torture:wksname=samba3rpctest')
         plansmbtorture4testsuite(t, "ad_dc", '//$SERVER/tmp -U$USERNAME%$PASSWORD --option=torture:wksname=samba3rpctest')
-    elif t.startswith("libsmbclient"):
-        url = "smb://$USERNAME:$PASSWORD@$SERVER/tmp"
-        if t == "libsmbclient.list_shares":
-            url = "smb://$USERNAME:$PASSWORD@$SERVER"
-
-        plansmbtorture4testsuite(t, "nt4_dc", '//$SERVER_IP/tmp -U$USERNAME%%$PASSWORD '
-                                    '--option=torture:smburl=' + url +
-                                    ' --option=torture:replace_smbconf=%s' % os.path.join(srcdir(), "testdata/samba3/smb_new.conf"))
-        plansmbtorture4testsuite(t, "ad_dc", '//$SERVER/tmp -U$USERNAME%%$PASSWORD '
-                                    '--option=torture:smburl=' + url +
-                                    ' --option=torture:replace_smbconf=%s' % os.path.join(srcdir(), "testdata/samba3/smb_new.conf"))
     elif t == "smb2.streams":
         plansmbtorture4testsuite(t, "nt4_dc", '//$SERVER_IP/tmp -U$USERNAME%$PASSWORD')
         plansmbtorture4testsuite(t, "ad_dc", '//$SERVER/tmp -U$USERNAME%$PASSWORD')