test_substituition: Fix failed test reporting to actually report failure.

You cannot put the "Testing <blah> pass/fail" on a single line before
actually performing the test.  Now any additional failure information is
logged before the test pass/fail announcement.

* Added an additional CDR(answer,u) test.

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@398018 65c4cc65-6c06-0410-ace0-fbb531ad65f3

diff --git a/tests/test_substitution.c b/tests/test_substitution.c
index 8faf3d3058fc71296a491fbd0baad796b4e10561..fae6f2997b89ba2a8e873cda98123581156d55c3 100644 (file)
--- a/tests/test_substitution.c
+++ b/tests/test_substitution.c
@@ -53,7 +53,6 @@ static enum ast_test_result_state test_chan_integer(struct ast_test *test,
        char workspace[4096];
        struct ast_str *str = ast_str_create(16);
 
-       ast_test_status_update(test, "Testing '%s' . . . . . %s\n", expression, okay ? "passed" : "FAILED");
        for (i = 0; i < 256; i++) {
                *ifield = i;
                ast_str_substitute_variables(&str, 0, c, expression);
@@ -63,6 +62,7 @@ static enum ast_test_result_state test_chan_integer(struct ast_test *test,
                        okay = 0;
                }
        }
+       ast_test_status_update(test, "Tested '%s' . . . . . %s\n", expression, okay ? "passed" : "FAILED");
 
        ast_free(str);
 
@@ -82,13 +82,14 @@ static enum ast_test_result_state test_chan_string(struct ast_test *test,
                ast_copy_string(cfield, values[i], cfieldsize);
                ast_str_substitute_variables(&str, 0, c, expression);
                pbx_substitute_variables_helper(c, expression, workspace, sizeof(workspace));
-               ast_test_status_update(test, "Testing '%s' . . . . . %s\n",
-                               expression, okay ? "passed" : "FAILED");
+
                if (strcmp(cfield, ast_str_buffer(str)) != 0 || strcmp(cfield, workspace) != 0) {
                        ast_test_status_update(test, "%s != %s != %s\n", cfield, ast_str_buffer(str), workspace);
                        okay = 0;
                }
        }
+       ast_test_status_update(test, "Tested '%s' . . . . . %s\n",
+               expression, okay ? "passed" : "FAILED");
 
        ast_free(str);
 
@@ -109,14 +110,15 @@ static enum ast_test_result_state test_chan_variable(struct ast_test *test,
                pbx_builtin_setvar_helper(c, varname, values[i]);
                ast_str_substitute_variables(&str, 0, c, ast_str_buffer(var));
                pbx_substitute_variables_helper(c, ast_str_buffer(var), workspace, sizeof(workspace));
-               ast_test_status_update(test, "Testing '%s' . . . . . %s\n",
-                               ast_str_buffer(var), okay ? "passed" : "FAILED");
+
                if (strcmp(values[i], ast_str_buffer(str)) != 0 || strcmp(values[i], workspace) != 0) {
                        ast_test_status_update(test, "%s != %s != %s\n",
                                        values[i], ast_str_buffer(str), workspace);
                        okay = 0;
                }
        }
+       ast_test_status_update(test, "Tested '%s' . . . . . %s\n",
+               ast_str_buffer(var), okay ? "passed" : "FAILED");
 
        ast_free(str);
        ast_free(var);
@@ -133,13 +135,14 @@ static enum ast_test_result_state test_chan_function(struct ast_test *test,
 
        ast_str_substitute_variables(&str, 0, c, expression);
        pbx_substitute_variables_helper(c, expression, workspace, sizeof(workspace));
-       ast_test_status_update(test, "Testing '%s' . . . . . %s\n",
-                       expression, okay ? "passed" : "FAILED");
+
        if (strcmp(workspace, ast_str_buffer(str)) != 0) {
-               ast_test_status_update(test, "test_chan_function, expr: '%s' ... %s != %s\n",
+               ast_test_status_update(test, "expr: '%s' ... %s != %s\n",
                                expression, ast_str_buffer(str), workspace);
                okay = 0;
        }
+       ast_test_status_update(test, "Tested '%s' . . . . . %s\n",
+               expression, okay ? "passed" : "FAILED");
 
        ast_free(str);
 
@@ -159,15 +162,13 @@ static enum ast_test_result_state test_2way_function(struct ast_test *test,
        ast_str_substitute_variables(&str, 0, c, ast_str_buffer(expression));
 
        okay = !strcmp(ast_str_buffer(str), "foobarbaz");
-
-       ast_test_status_update(test, "Testing '%s%s' and '%s%s' . . . . . %s\n",
-                       encode1, encode2, decode1, decode2,
-                       okay ? "passed" : "FAILED");
-
        if (!okay) {
-               ast_test_status_update(test, "  '%s' != 'foobarbaz'\n",
+               ast_test_status_update(test, "'%s' != 'foobarbaz'\n",
                                ast_str_buffer(str));
        }
+       ast_test_status_update(test, "Tested '%s%s' and '%s%s' . . . . . %s\n",
+               encode1, encode2, decode1, decode2,
+               okay ? "passed" : "FAILED");
 
        ast_free(str);
 
@@ -181,16 +182,15 @@ static enum ast_test_result_state test_expected_result(struct ast_test *test,
        int okay;
 
        ast_str_substitute_variables(&str, 0, c, expression);
-       okay = !strcmp(ast_str_buffer(str), result);
-
-       ast_test_status_update(test, "Testing '%s' ('%s') == '%s' . . . . . %s\n",
-                       ast_str_buffer(str), expression, result,
-                       okay ? "passed" : "FAILED");
 
+       okay = !strcmp(ast_str_buffer(str), result);
        if (!okay) {
-               ast_test_status_update(test, "test_expected_result: '%s' != '%s'\n",
+               ast_test_status_update(test, "'%s' != '%s'\n",
                                ast_str_buffer(str), result);
        }
+       ast_test_status_update(test, "Tested '%s' ('%s') == '%s' . . . . . %s\n",
+               ast_str_buffer(str), expression, result,
+               okay ? "passed" : "FAILED");
 
        ast_free(str);
 
@@ -264,6 +264,7 @@ AST_TEST_DEFINE(test_substitution)
        TEST(test_expected_result(test, c, "${LISTFILTER(list1,&,cd)}", "ab&ef"));
        TEST(test_expected_result(test, c, "${SHELL(printf '%d' 123)},${SHELL(printf '%d' 456)}", "123,456"));
        TEST(test_expected_result(test, c, "${foo},${CDR(answer)},${SHELL(printf '%d' 456)}", "123,,456"));
+       TEST(test_expected_result(test, c, "${foo},${CDR(answer,u)},${SHELL(printf '%d' 456)}", "123,0.000000,456"));
        TEST(test_expected_result(test, c, "${foo},${this_does_not_exist},${THIS_DOES_NOT_EXIST(either)}", "123,,"));
 #undef TEST