summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKai Blin <kai@samba.org>2007-12-24 13:04:56 -0600
committerStefan Metzmacher <metze@samba.org>2007-12-26 11:57:05 -0600
commit3c744ddd2c33a9a06013f357261b8ea86804e8e8 (patch)
tree431d069898a963280398c88b94f56498ac678aa8
parent95b1f554b2c57a9f975a0cc27ca51bec6c7594d6 (diff)
downloadsamba-3c744ddd2c33a9a06013f357261b8ea86804e8e8.tar.gz
samba-3c744ddd2c33a9a06013f357261b8ea86804e8e8.tar.bz2
samba-3c744ddd2c33a9a06013f357261b8ea86804e8e8.zip
r26588: Janitorial: Rename torture_*_add_*test to torture_*_add_*test_const.
Also rename the corresponding wrap_ functions. (This used to be commit e59c2eaf681f076d175b9779d1c27b5f74a57c96)
-rw-r--r--source4/lib/events/testsuite.c2
-rw-r--r--source4/lib/messaging/tests/irpc.c9
-rw-r--r--source4/lib/registry/tests/hive.c40
-rw-r--r--source4/lib/registry/tests/registry.c73
-rw-r--r--source4/lib/util/tests/strlist.c5
-rw-r--r--source4/libcli/security/tests/sddl.c6
-rw-r--r--source4/librpc/tests/binding_string.c13
-rw-r--r--source4/param/tests/share.c16
-rw-r--r--source4/torture/local/dbspeed.c6
-rw-r--r--source4/torture/rpc/session_key.c3
-rw-r--r--source4/torture/ui.c43
-rw-r--r--source4/torture/ui.h46
12 files changed, 138 insertions, 124 deletions
diff --git a/source4/lib/events/testsuite.c b/source4/lib/events/testsuite.c
index 46acf5938e..7f2729ceff 100644
--- a/source4/lib/events/testsuite.c
+++ b/source4/lib/events/testsuite.c
@@ -136,7 +136,7 @@ struct torture_suite *torture_local_event(TALLOC_CTX *mem_ctx)
int i;
for (i=0;list && list[i];i++) {
- torture_suite_add_simple_tcase(suite, list[i],
+ torture_suite_add_simple_tcase_const(suite, list[i],
test_event_context,
(const void *)list[i]);
}
diff --git a/source4/lib/messaging/tests/irpc.c b/source4/lib/messaging/tests/irpc.c
index ce3d1045e5..a2995fc983 100644
--- a/source4/lib/messaging/tests/irpc.c
+++ b/source4/lib/messaging/tests/irpc.c
@@ -260,11 +260,12 @@ struct torture_suite *torture_local_irpc(TALLOC_CTX *mem_ctx)
tcase->setup = irpc_setup;
for (i = 0; i < 5; i++) {
- torture_tcase_add_test(tcase, "addone", test_addone, (void *)values[i]);
+ torture_tcase_add_test_const(tcase, "addone", test_addone,
+ (void *)values[i]);
}
-
- torture_tcase_add_test(tcase, "echodata", test_echodata, NULL);
- torture_tcase_add_test(tcase, "speed", test_speed, NULL);
+
+ torture_tcase_add_test_const(tcase, "echodata", test_echodata, NULL);
+ torture_tcase_add_test_const(tcase, "speed", test_speed, NULL);
return suite;
}
diff --git a/source4/lib/registry/tests/hive.c b/source4/lib/registry/tests/hive.c
index 43ec9e4252..dc59d0f9ca 100644
--- a/source4/lib/registry/tests/hive.c
+++ b/source4/lib/registry/tests/hive.c
@@ -264,26 +264,26 @@ static bool test_list_values(struct torture_context *tctx,
static void tcase_add_tests(struct torture_tcase *tcase)
{
- torture_tcase_add_simple_test(tcase, "del_nonexistant_key",
- test_del_nonexistant_key);
- torture_tcase_add_simple_test(tcase, "add_subkey",
- test_add_subkey);
- torture_tcase_add_simple_test(tcase, "flush_key",
- test_flush_key);
- torture_tcase_add_simple_test(tcase, "get_info",
- test_keyinfo_root);
- torture_tcase_add_simple_test(tcase, "get_info_nums",
- test_keyinfo_nums);
- torture_tcase_add_simple_test(tcase, "set_value",
- test_set_value);
- torture_tcase_add_simple_test(tcase, "get_value",
- test_get_value);
- torture_tcase_add_simple_test(tcase, "list_values",
- test_list_values);
- torture_tcase_add_simple_test(tcase, "del_key",
- test_del_key);
- torture_tcase_add_simple_test(tcase, "del_value",
- test_del_value);
+ torture_tcase_add_simple_test_const(tcase, "del_nonexistant_key",
+ test_del_nonexistant_key);
+ torture_tcase_add_simple_test_const(tcase, "add_subkey",
+ test_add_subkey);
+ torture_tcase_add_simple_test_const(tcase, "flush_key",
+ test_flush_key);
+ torture_tcase_add_simple_test_const(tcase, "get_info",
+ test_keyinfo_root);
+ torture_tcase_add_simple_test_const(tcase, "get_info_nums",
+ test_keyinfo_nums);
+ torture_tcase_add_simple_test_const(tcase, "set_value",
+ test_set_value);
+ torture_tcase_add_simple_test_const(tcase, "get_value",
+ test_get_value);
+ torture_tcase_add_simple_test_const(tcase, "list_values",
+ test_list_values);
+ torture_tcase_add_simple_test_const(tcase, "del_key",
+ test_del_key);
+ torture_tcase_add_simple_test_const(tcase, "del_value",
+ test_del_value);
}
static bool hive_setup_dir(struct torture_context *tctx, void **data)
diff --git a/source4/lib/registry/tests/registry.c b/source4/lib/registry/tests/registry.c
index 6c520f54ed..7897909865 100644
--- a/source4/lib/registry/tests/registry.c
+++ b/source4/lib/registry/tests/registry.c
@@ -560,42 +560,43 @@ static bool setup_local_registry(struct torture_context *tctx, void **data)
static void tcase_add_tests(struct torture_tcase *tcase)
{
- torture_tcase_add_simple_test(tcase, "list_subkeys",
- test_list_subkeys);
- torture_tcase_add_simple_test(tcase, "get_predefined_key",
- test_get_predefined);
- torture_tcase_add_simple_test(tcase, "get_predefined_key",
- test_get_predefined_unknown);
- torture_tcase_add_simple_test(tcase, "create_key",
- test_create_subkey);
- torture_tcase_add_simple_test(tcase, "create_key",
- test_create_nested_subkey);
- torture_tcase_add_simple_test(tcase, "key_add_abs",
- test_key_add_abs);
- torture_tcase_add_simple_test(tcase, "key_add_abs_top",
- test_key_add_abs_top);
- torture_tcase_add_simple_test(tcase, "set_value",
- test_set_value);
- torture_tcase_add_simple_test(tcase, "get_value",
- test_get_value);
- torture_tcase_add_simple_test(tcase, "list_values",
- test_list_values);
- torture_tcase_add_simple_test(tcase, "del_key",
- test_del_key);
- torture_tcase_add_simple_test(tcase, "del_value",
- test_del_value);
- torture_tcase_add_simple_test(tcase, "flush_key",
- test_flush_key);
- torture_tcase_add_simple_test(tcase, "query_key",
- test_query_key);
- torture_tcase_add_simple_test(tcase, "query_key_nums",
- test_query_key_nums);
- torture_tcase_add_simple_test(tcase, "test_predef_key_by_name",
- test_predef_key_by_name);
- torture_tcase_add_simple_test(tcase, "security",
- test_security);
- torture_tcase_add_simple_test(tcase, "test_predef_key_by_name_invalid",
- test_predef_key_by_name_invalid);
+ torture_tcase_add_simple_test_const(tcase, "list_subkeys",
+ test_list_subkeys);
+ torture_tcase_add_simple_test_const(tcase, "get_predefined_key",
+ test_get_predefined);
+ torture_tcase_add_simple_test_const(tcase, "get_predefined_key",
+ test_get_predefined_unknown);
+ torture_tcase_add_simple_test_const(tcase, "create_key",
+ test_create_subkey);
+ torture_tcase_add_simple_test_const(tcase, "create_key",
+ test_create_nested_subkey);
+ torture_tcase_add_simple_test_const(tcase, "key_add_abs",
+ test_key_add_abs);
+ torture_tcase_add_simple_test_const(tcase, "key_add_abs_top",
+ test_key_add_abs_top);
+ torture_tcase_add_simple_test_const(tcase, "set_value",
+ test_set_value);
+ torture_tcase_add_simple_test_const(tcase, "get_value",
+ test_get_value);
+ torture_tcase_add_simple_test_const(tcase, "list_values",
+ test_list_values);
+ torture_tcase_add_simple_test_const(tcase, "del_key",
+ test_del_key);
+ torture_tcase_add_simple_test_const(tcase, "del_value",
+ test_del_value);
+ torture_tcase_add_simple_test_const(tcase, "flush_key",
+ test_flush_key);
+ torture_tcase_add_simple_test_const(tcase, "query_key",
+ test_query_key);
+ torture_tcase_add_simple_test_const(tcase, "query_key_nums",
+ test_query_key_nums);
+ torture_tcase_add_simple_test_const(tcase, "test_predef_key_by_name",
+ test_predef_key_by_name);
+ torture_tcase_add_simple_test_const(tcase, "security",
+ test_security);
+ torture_tcase_add_simple_test_const(tcase,
+ "test_predef_key_by_name_invalid",
+ test_predef_key_by_name_invalid);
}
struct torture_suite *torture_registry_registry(TALLOC_CTX *mem_ctx)
diff --git a/source4/lib/util/tests/strlist.c b/source4/lib/util/tests/strlist.c
index 3ecb982e24..9af26f9e71 100644
--- a/source4/lib/util/tests/strlist.c
+++ b/source4/lib/util/tests/strlist.c
@@ -93,9 +93,8 @@ struct torture_suite *torture_local_util_strlist(TALLOC_CTX *mem_ctx)
int i;
for (i = 0; test_lists_shell_strings[i]; i++) {
- torture_suite_add_simple_tcase(suite,
- "lists_shell", test_lists_shell,
- &test_lists_shell_strings[i]);
+ torture_suite_add_simple_tcase_const(suite, "lists_shell",
+ test_lists_shell, &test_lists_shell_strings[i]);
}
torture_suite_add_simple_test(suite, "list_copy", test_list_copy);
diff --git a/source4/libcli/security/tests/sddl.c b/source4/libcli/security/tests/sddl.c
index 3e6382309e..9e7705ea92 100644
--- a/source4/libcli/security/tests/sddl.c
+++ b/source4/libcli/security/tests/sddl.c
@@ -96,9 +96,9 @@ struct torture_suite *torture_local_sddl(TALLOC_CTX *mem_ctx)
int i;
for (i = 0; i < ARRAY_SIZE(examples); i++) {
- torture_suite_add_simple_tcase(suite,
- talloc_asprintf(suite, "%d", i),
- test_sddl, examples[i]);
+ torture_suite_add_simple_tcase_const(suite,
+ talloc_asprintf(suite, "%d", i),
+ test_sddl, examples[i]);
}
return suite;
diff --git a/source4/librpc/tests/binding_string.c b/source4/librpc/tests/binding_string.c
index 362848049b..5e6fb04515 100644
--- a/source4/librpc/tests/binding_string.c
+++ b/source4/librpc/tests/binding_string.c
@@ -154,17 +154,18 @@ static bool test_no_transport(struct torture_context *tctx)
struct torture_suite *torture_local_binding_string(TALLOC_CTX *mem_ctx)
{
int i;
- struct torture_suite *suite = torture_suite_create(mem_ctx,
- "BINDING");
+ struct torture_suite *suite = torture_suite_create(mem_ctx, "BINDING");
for (i = 0; i < ARRAY_SIZE(test_strings); i++) {
- torture_suite_add_simple_tcase(suite, test_strings[i],
- test_BindingString, test_strings[i]);
+ torture_suite_add_simple_tcase_const(suite, test_strings[i],
+ test_BindingString,
+ test_strings[i]);
}
- torture_suite_add_simple_test(suite, "no transport", test_no_transport);
+ torture_suite_add_simple_test(suite, "no transport",test_no_transport);
- torture_suite_add_simple_test(suite, "parsing results", test_parse_check_results);
+ torture_suite_add_simple_test(suite, "parsing results",
+ test_parse_check_results);
return suite;
}
diff --git a/source4/param/tests/share.c b/source4/param/tests/share.c
index 390da9dede..6d03b4e049 100644
--- a/source4/param/tests/share.c
+++ b/source4/param/tests/share.c
@@ -168,12 +168,16 @@ static bool test_double_create(struct torture_context *tctx,
static void tcase_add_share_tests(struct torture_tcase *tcase)
{
- torture_tcase_add_test(tcase, "list_empty", test_list_empty, NULL);
- torture_tcase_add_test(tcase, "share_create", test_create, NULL);
- torture_tcase_add_test(tcase, "share_remove", test_share_remove, NULL);
- torture_tcase_add_test(tcase, "share_remove_invalid", test_share_remove_invalid, NULL);
- torture_tcase_add_test(tcase, "share_create_invalid", test_create_invalid, NULL);
- torture_tcase_add_test(tcase, "share_double_create", test_double_create, NULL);
+ torture_tcase_add_test_const(tcase, "list_empty", test_list_empty,NULL);
+ torture_tcase_add_test_const(tcase, "share_create", test_create, NULL);
+ torture_tcase_add_test_const(tcase, "share_remove", test_share_remove,
+ NULL);
+ torture_tcase_add_test_const(tcase, "share_remove_invalid",
+ test_share_remove_invalid, NULL);
+ torture_tcase_add_test_const(tcase, "share_create_invalid",
+ test_create_invalid, NULL);
+ torture_tcase_add_test_const(tcase, "share_double_create",
+ test_double_create, NULL);
}
static bool setup_ldb(struct torture_context *tctx, void **data)
diff --git a/source4/torture/local/dbspeed.c b/source4/torture/local/dbspeed.c
index 47af7935fb..34083cd204 100644
--- a/source4/torture/local/dbspeed.c
+++ b/source4/torture/local/dbspeed.c
@@ -253,7 +253,9 @@ failed:
struct torture_suite *torture_local_dbspeed(TALLOC_CTX *mem_ctx)
{
struct torture_suite *s = torture_suite_create(mem_ctx, "DBSPEED");
- torture_suite_add_simple_tcase(s, "tdb_speed", test_tdb_speed, NULL);
- torture_suite_add_simple_tcase(s, "ldb_speed", test_ldb_speed, NULL);
+ torture_suite_add_simple_tcase_const(s, "tdb_speed", test_tdb_speed,
+ NULL);
+ torture_suite_add_simple_tcase_const(s, "ldb_speed", test_ldb_speed,
+ NULL);
return s;
}
diff --git a/source4/torture/rpc/session_key.c b/source4/torture/rpc/session_key.c
index 44c3ae2358..fcb828ddb2 100644
--- a/source4/torture/rpc/session_key.c
+++ b/source4/torture/rpc/session_key.c
@@ -201,7 +201,8 @@ static struct torture_tcase *add_test(struct torture_suite *suite, uint32_t bind
name = talloc_asprintf_append_buffer(name, " lm_key:%s", lm_key?"yes":"no");
settings->lm_key = lm_key;
- return torture_suite_add_simple_tcase(suite, name, test_secrets, settings);
+ return torture_suite_add_simple_tcase_const(suite, name, test_secrets,
+ settings);
}
static const bool bool_vals[] = { true, false };
diff --git a/source4/torture/ui.c b/source4/torture/ui.c
index 95d95937ed..ce72dd5b08 100644
--- a/source4/torture/ui.c
+++ b/source4/torture/ui.c
@@ -113,11 +113,11 @@ void torture_tcase_set_fixture(struct torture_tcase *tcase,
tcase->teardown = teardown;
}
-static bool wrap_test_with_testcase(struct torture_context *torture_ctx,
+static bool wrap_test_with_testcase_const(struct torture_context *torture_ctx,
struct torture_tcase *tcase,
struct torture_test *test)
{
- bool (*fn) (struct torture_context *,
+ bool (*fn) (struct torture_context *,
const void *tcase_data,
const void *test_data);
@@ -126,16 +126,17 @@ static bool wrap_test_with_testcase(struct torture_context *torture_ctx,
return fn(torture_ctx, tcase->data, test->data);
}
-struct torture_test *torture_tcase_add_test(struct torture_tcase *tcase,
- const char *name,
- bool (*run) (struct torture_context *, const void *tcase_data, const void *test_data),
- const void *data)
+struct torture_test *torture_tcase_add_test_const(struct torture_tcase *tcase,
+ const char *name,
+ bool (*run) (struct torture_context *, const void *tcase_data,
+ const void *test_data),
+ const void *data)
{
struct torture_test *test = talloc(tcase, struct torture_test);
test->name = talloc_strdup(test, name);
test->description = NULL;
- test->run = wrap_test_with_testcase;
+ test->run = wrap_test_with_testcase_const;
test->fn = run;
test->dangerous = false;
test->data = data;
@@ -400,9 +401,10 @@ const char *torture_setting_string(struct torture_context *test,
return ret;
}
-static bool wrap_test_with_simple_tcase(struct torture_context *torture_ctx,
- struct torture_tcase *tcase,
- struct torture_test *test)
+static bool wrap_test_with_simple_tcase_const (
+ struct torture_context *torture_ctx,
+ struct torture_tcase *tcase,
+ struct torture_test *test)
{
bool (*fn) (struct torture_context *, const void *tcase_data);
@@ -411,21 +413,21 @@ static bool wrap_test_with_simple_tcase(struct torture_context *torture_ctx,
return fn(torture_ctx, test->data);
}
-struct torture_tcase *torture_suite_add_simple_tcase(
+struct torture_tcase *torture_suite_add_simple_tcase_const(
struct torture_suite *suite, const char *name,
bool (*run) (struct torture_context *test, const void *),
const void *data)
{
struct torture_tcase *tcase;
- struct torture_test *test;
-
+ struct torture_test *test;
+
tcase = torture_suite_add_tcase(suite, name);
test = talloc(tcase, struct torture_test);
test->name = talloc_strdup(test, name);
test->description = NULL;
- test->run = wrap_test_with_simple_tcase;
+ test->run = wrap_test_with_simple_tcase_const;
test->fn = run;
test->data = data;
test->dangerous = false;
@@ -496,7 +498,7 @@ struct torture_suite *torture_find_suite(struct torture_suite *parent,
return NULL;
}
-static bool wrap_test_with_simple_test(struct torture_context *torture_ctx,
+static bool wrap_test_with_simple_test_const(struct torture_context *torture_ctx,
struct torture_tcase *tcase,
struct torture_test *test)
{
@@ -507,18 +509,19 @@ static bool wrap_test_with_simple_test(struct torture_context *torture_ctx,
return fn(torture_ctx, tcase->data);
}
-struct torture_test *torture_tcase_add_simple_test(
+struct torture_test *torture_tcase_add_simple_test_const(
struct torture_tcase *tcase,
const char *name,
- bool (*run) (struct torture_context *test, const void *tcase_data))
+ bool (*run) (struct torture_context *test,
+ const void *tcase_data))
{
- struct torture_test *test;
-
+ struct torture_test *test;
+
test = talloc(tcase, struct torture_test);
test->name = talloc_strdup(test, name);
test->description = NULL;
- test->run = wrap_test_with_simple_test;
+ test->run = wrap_test_with_simple_test_const;
test->fn = run;
test->data = NULL;
test->dangerous = false;
diff --git a/source4/torture/ui.h b/source4/torture/ui.h
index 5f0ec3dbff..503a4f4dec 100644
--- a/source4/torture/ui.h
+++ b/source4/torture/ui.h
@@ -140,67 +140,69 @@ struct torture_suite
};
/** Create a new torture suite */
-struct torture_suite *torture_suite_create(TALLOC_CTX *mem_ctx,
- const char *name);
+struct torture_suite *torture_suite_create(TALLOC_CTX *mem_ctx,
+ const char *name);
/** Change the setup and teardown functions for a testcase */
-void torture_tcase_set_fixture(struct torture_tcase *tcase,
+void torture_tcase_set_fixture(struct torture_tcase *tcase,
bool (*setup) (struct torture_context *, void **),
bool (*teardown) (struct torture_context *, void *));
/* Add another test to run for a particular testcase */
-struct torture_test *torture_tcase_add_test(struct torture_tcase *tcase,
- const char *name,
- bool (*run) (struct torture_context *test, const void *tcase_data,
- const void *test_data),
+struct torture_test *torture_tcase_add_test_const(struct torture_tcase *tcase,
+ const char *name,
+ bool (*run) (struct torture_context *test,
+ const void *tcase_data, const void *test_data),
const void *test_data);
/* Add a testcase to a testsuite */
-struct torture_tcase *torture_suite_add_tcase(struct torture_suite *suite,
+struct torture_tcase *torture_suite_add_tcase(struct torture_suite *suite,
const char *name);
-/* Convenience wrapper that adds a testcase against only one
+/* Convenience wrapper that adds a testcase against only one
* test will be run */
-struct torture_tcase *torture_suite_add_simple_tcase(
- struct torture_suite *suite,
+struct torture_tcase *torture_suite_add_simple_tcase_const(
+ struct torture_suite *suite,
const char *name,
- bool (*run) (struct torture_context *test, const void *test_data),
+ bool (*run) (struct torture_context *test,
+ const void *test_data),
const void *data);
-/* Convenience function that adds a test which only
+/* Convenience function that adds a test which only
* gets the test case data */
-struct torture_test *torture_tcase_add_simple_test(
+struct torture_test *torture_tcase_add_simple_test_const(
struct torture_tcase *tcase,
const char *name,
- bool (*run) (struct torture_context *test, const void *tcase_data));
+ bool (*run) (struct torture_context *test,
+ const void *tcase_data));
-/* Convenience wrapper that adds a test that doesn't need any
+/* Convenience wrapper that adds a test that doesn't need any
* testcase data */
struct torture_tcase *torture_suite_add_simple_test(
- struct torture_suite *suite,
+ struct torture_suite *suite,
const char *name,
bool (*run) (struct torture_context *test));
/* Add a child testsuite to an existing testsuite */
bool torture_suite_add_suite(struct torture_suite *suite,
- struct torture_suite *child);
+ struct torture_suite *child);
/* Run the specified testsuite recursively */
-bool torture_run_suite(struct torture_context *context,
+bool torture_run_suite(struct torture_context *context,
struct torture_suite *suite);
/* Run the specified testcase */
-bool torture_run_tcase(struct torture_context *context,
+bool torture_run_tcase(struct torture_context *context,
struct torture_tcase *tcase);
/* Run the specified test */
-bool torture_run_test(struct torture_context *context,
+bool torture_run_test(struct torture_context *context,
struct torture_tcase *tcase,
struct torture_test *test);
void torture_comment(struct torture_context *test, const char *comment, ...) PRINTF_ATTRIBUTE(2,3);
void torture_warning(struct torture_context *test, const char *comment, ...) PRINTF_ATTRIBUTE(2,3);
-void torture_result(struct torture_context *test,
+void torture_result(struct torture_context *test,
enum torture_result, const char *reason, ...) PRINTF_ATTRIBUTE(3,4);
#define torture_assert(torture_ctx,expr,cmt) \