From 6562532a8aa0e0eaee444c6270d555a5d6a734cf Mon Sep 17 00:00:00 2001 From: Ben Pfaff Date: Sat, 25 Sep 2010 15:37:02 -0700 Subject: [PATCH] stringi-map: Convert tests to use Autotest. --- tests/automake.mk | 3 +- tests/libpspp/stringi-map-test.c | 153 +++++++++++++++++++++++-------- tests/libpspp/stringi-map.at | 22 +++++ 3 files changed, 139 insertions(+), 39 deletions(-) create mode 100644 tests/libpspp/stringi-map.at diff --git a/tests/automake.mk b/tests/automake.mk index 2fc54fca..6fe59dca 100644 --- a/tests/automake.mk +++ b/tests/automake.mk @@ -146,7 +146,6 @@ dist_TESTS += tests/command/get-data-psql.sh endif nodist_TESTS = \ - tests/libpspp/stringi-map-test \ tests/libpspp/string-set-test \ tests/libpspp/stringi-set-test \ tests/libpspp/tower-test @@ -171,6 +170,7 @@ check_PROGRAMS += \ tests/libpspp/sparse-xarray-test \ tests/libpspp/str-test \ tests/libpspp/string-map-test \ + tests/libpspp/stringi-map-test \ tests/output/render-test tests_data_datasheet_test_SOURCES = \ @@ -431,6 +431,7 @@ TESTSUITE_AT = \ tests/libpspp/sparse-array.at \ tests/libpspp/str.at \ tests/libpspp/string-map.at \ + tests/libpspp/stringi-map.at \ tests/math/moments.at \ tests/output/render.at \ tests/output/charts.at \ diff --git a/tests/libpspp/stringi-map-test.c b/tests/libpspp/stringi-map-test.c index 626dfacb..a8dd9dd6 100644 --- a/tests/libpspp/stringi-map-test.c +++ b/tests/libpspp/stringi-map-test.c @@ -43,9 +43,6 @@ #include "libpspp/string-set.h" #include "libpspp/stringi-set.h" -/* Currently running test. */ -static const char *test_name; - /* Exit with a failure code. (Place a breakpoint on this function while debugging.) */ static void @@ -61,8 +58,7 @@ check_func (bool ok, int line) { if (!ok) { - printf ("Check failed in %s test at %s, line %d\n", - test_name, __FILE__, line); + fprintf (stderr, "%s:%d: check failed\n", __FILE__, line); check_die (); } } @@ -921,39 +917,120 @@ test_destroy_null (void) /* Main program. */ -/* Runs TEST_FUNCTION and prints a message about NAME. */ -static void -run_test (void (*test_function) (void), const char *name) -{ - test_name = name; - putchar ('.'); - fflush (stdout); - test_function (); -} +struct test + { + const char *name; + const char *description; + void (*function) (void); + }; + +static const struct test tests[] = + { + { + "insert-any-remove-any", + "insert any order, delete any order", + test_insert_any_remove_any + }, + { + "insert-any-remove-same", + "insert any order, delete same order", + test_insert_any_remove_same + }, + { + "insert-any-remove-reverse", + "insert any order, delete reverse order", + test_insert_any_remove_reverse + }, + { + "random-sequence", + "insert and delete in random sequence", + test_random_sequence + }, + { + "replace", + "insert and replace in random sequence", + test_replace + }, + { + "insert-ordered", + "insert in ascending order", + test_insert_ordered + }, + { + "clear", + "clear", + test_clear + }, + { + "clone", + "clone", + test_clone + }, + { + "swap", + "swap", + test_swap + }, + { + "node-swap-value", + "node_swap_value", + test_node_swap_value + }, + { + "insert-map", + "insert_map", + test_insert_map + }, + { + "replace-map", + "replace_map", + test_replace_map + }, + { + "get-keys-and-values", + "get keys and values", + test_get_keys_and_values + }, + { + "destroy-null", + "destroying null table", + test_destroy_null + }, + }; + +enum { N_TESTS = sizeof tests / sizeof *tests }; int -main (void) -{ - run_test (test_insert_any_remove_any, "insert any order, delete any order"); - run_test (test_insert_any_remove_same, - "insert any order, delete same order"); - run_test (test_insert_any_remove_reverse, - "insert any order, delete reverse order"); - run_test (test_random_sequence, "insert and delete in random sequence"); - run_test (test_replace, "insert and replace in random sequence"); - run_test (test_insert_ordered, "insert in ascending order"); - run_test (test_clear, "clear"); - run_test (test_clone, "clone"); - run_test (test_swap, "swap"); - run_test (test_node_swap_value, "node_swap_value"); - run_test (test_insert_map, "insert_map"); - run_test (test_replace_map, "replace_map"); - run_test (test_get_keys_and_values, "get keys and values"); - run_test (test_destroy_null, "destroying null table"); - - putchar ('\n'); - - free_strings (); - - return 0; +main (int argc, char *argv[]) +{ + int i; + + if (argc != 2) + { + fprintf (stderr, "exactly one argument required; use --help for help\n"); + return EXIT_FAILURE; + } + else if (!strcmp (argv[1], "--help")) + { + printf ("%s: test case-insensitive string map library\n" + "usage: %s TEST-NAME\n" + "where TEST-NAME is one of the following:\n", + argv[0], argv[0]); + for (i = 0; i < N_TESTS; i++) + printf (" %s\n %s\n", tests[i].name, tests[i].description); + return 0; + } + else + { + for (i = 0; i < N_TESTS; i++) + if (!strcmp (argv[1], tests[i].name)) + { + tests[i].function (); + free_strings (); + return 0; + } + + fprintf (stderr, "unknown test %s; use --help for help\n", argv[1]); + return EXIT_FAILURE; + } } diff --git a/tests/libpspp/stringi-map.at b/tests/libpspp/stringi-map.at new file mode 100644 index 00000000..c14f0e6f --- /dev/null +++ b/tests/libpspp/stringi-map.at @@ -0,0 +1,22 @@ +AT_BANNER([case-insensitive string map library]) + +m4_define([CHECK_STRINGI_MAP], + [AT_SETUP([case-insensitive string map -- $1]) + AT_KEYWORDS([stringi-map]) + AT_CHECK([stringi-map-test $1]) + AT_CLEANUP]) + +CHECK_STRINGI_MAP([insert-any-remove-any]) +CHECK_STRINGI_MAP([insert-any-remove-same]) +CHECK_STRINGI_MAP([insert-any-remove-reverse]) +CHECK_STRINGI_MAP([random-sequence]) +CHECK_STRINGI_MAP([replace]) +CHECK_STRINGI_MAP([insert-ordered]) +CHECK_STRINGI_MAP([clear]) +CHECK_STRINGI_MAP([clone]) +CHECK_STRINGI_MAP([swap]) +CHECK_STRINGI_MAP([node-swap-value]) +CHECK_STRINGI_MAP([insert-map]) +CHECK_STRINGI_MAP([replace-map]) +CHECK_STRINGI_MAP([get-keys-and-values]) +CHECK_STRINGI_MAP([destroy-null]) -- 2.30.2