s3: selftest: Remove test_vfs_widelinks.

All of the tests that were in there
are now tested in samba3.smbtorture_s3.LOCAL-CANONICALIZE-PATH
along with other paths.

Clean revert of f7fe3474298 not possible due to
changes in source3/selftest/tests.py

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

diff --git a/source3/modules/test_vfs_widelinks.c b/source3/modules/test_vfs_widelinks.c
deleted file mode 100644 (file)
index c6055c8..0000000
--- a/source3/modules/test_vfs_widelinks.c
+++ /dev/null
@@ -1,88 +0,0 @@
-/*
- *  Unix SMB/CIFS implementation.
- *
- *  Unit test for widelinks path validator.
- *
- *  Copyright (C) Jeremy Allison 2020
- *
- *  This program is free software; you can redistribute it and/or modify
- *  it under the terms of the GNU General Public License as published by
- *  the Free Software Foundation; either version 3 of the License, or
- *  (at your option) any later version.
- *
- *  This program is distributed in the hope that it will be useful,
- *  but WITHOUT ANY WARRANTY; without even the implied warranty of
- *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- *  GNU General Public License for more details.
- *
- *  You should have received a copy of the GNU General Public License
- *  along with this program; if not, see <http://www.gnu.org/licenses/>.
- */
-
-/* Needed for static build to complete... */
-#include "includes.h"
-#include "smbd/smbd.h"
-NTSTATUS vfs_widelinks_init(TALLOC_CTX *ctx);
-
-#include "vfs_widelinks.c"
-#include <cmocka.h>
-
-struct str_test_values {
-       const char *src_str;
-       const char *dst_str;
-} ;
-
-/* As many nasty edge cases as I can think of.. */
-
-static struct str_test_values examples[] = {
-       { "/", "/" },
-       { "/../../", "/" },
-       { "/foo/../", "/" },
-       { "/./././", "/" },
-       { "/./././.", "/" },
-       { "/.../././.", "/..." },
-       { "/./././.foo", "/.foo" },
-       { "/./././.foo.", "/.foo." },
-       { "/./././foo.", "/foo." },
-       { "/foo/bar/..", "/foo" },
-       { "/foo/bar/../baz/", "/foo/baz" },
-       { "////////////////", "/" },
-       { "/////////./././././.", "/" },
-       { "/./.././../.boo/../baz", "/baz" },
-       { "/a/component/path", "/a/component/path" },
-       { "/a/component/path/", "/a/component/path" },
-       { "/a/component/path/..", "/a/component" },
-       { "/a/component/../path/", "/a/path" },
-       { "///a/./././///component/../////path/", "/a/path" }
-};
-
-/*
- * Test our realpath resolution code.
- */
-static void test_resolve_realpath_name(void **state)
-{
-       unsigned i;
-       TALLOC_CTX *frame = talloc_stackframe();
-
-       for (i = 0; i < ARRAY_SIZE(examples); i++) {
-               char *test_dst = resolve_realpath_name(frame,
-                                       examples[i].src_str);
-               if (test_dst == NULL) {
-                       fail();
-               }
-               assert_string_equal(test_dst, examples[i].dst_str);
-               TALLOC_FREE(test_dst);
-       }
-       TALLOC_FREE(frame);
-}
-
-int main(int argc, char **argv)
-{
-       const struct CMUnitTest tests[] = {
-               cmocka_unit_test(test_resolve_realpath_name),
-       };
-
-       cmocka_set_message_output(CM_OUTPUT_SUBUNIT);
-
-       return cmocka_run_group_tests(tests, NULL, NULL);
-}

diff --git a/source3/modules/wscript_build b/source3/modules/wscript_build
index 1f2d2d3acb63c08297dcdae5d3f5cf9fb2b5d5b1..7f056f2b7f77f7e04d6038f7b8fdb327f7770ada 100644 (file)
--- a/source3/modules/wscript_build
+++ b/source3/modules/wscript_build
@@ -628,8 +628,3 @@ bld.SAMBA3_MODULE('vfs_widelinks',
                  init_function='',
                  internal_module=bld.SAMBA3_IS_STATIC_MODULE('vfs_widelinks'),
                  enabled=bld.SAMBA3_IS_ENABLED_MODULE('vfs_widelinks'))
-
-bld.SAMBA3_BINARY('test_vfs_widelinks',
-                  source='test_vfs_widelinks.c',
-                  deps='smbd_base cmocka',
-                  for_selftest=True)

diff --git a/source3/selftest/tests.py b/source3/selftest/tests.py
index a536a473cb5495bca6f21fe66bc368a09134248b..8210e981c3535d3e14476f5ccb07c857c4895142 100755 (executable)
--- a/source3/selftest/tests.py
+++ b/source3/selftest/tests.py
@@ -508,10 +508,6 @@ plantestsuite("samba3.test_nfs4_acl", "none",
               [os.path.join(bindir(), "test_nfs4_acls"),
                "$SMB_CONF_PATH"])
 
-plantestsuite("samba3.test_vfs_widelinks", "none",
-              [os.path.join(bindir(), "test_vfs_widelinks"),
-               "$SMB_CONF_PATH"])
-
 plantestsuite("samba3.test_vfs_full_audit", "none",
               [os.path.join(bindir(), "test_vfs_full_audit"),
                "$SMB_CONF_PATH"])