r26588: Janitorial: Rename torture_*_add_*test to torture_*_add_*test_const.

Also rename the corresponding wrap_ functions.
(This used to be commit e59c2eaf681f076d175b9779d1c27b5f74a57c96)

4 files changed, 53 insertions, 45 deletions

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) \