Reorder and renumber tests appropriately

It appears that a patch did not apply properly when adding tests 12 and
13 and test 11 was duplicated.  These tests have been reordered and
renumbered such that they make sense.
........

Merged revisions 366882 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 366884 from http://svn.asterisk.org/svn/asterisk/branches/10

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

diff --git a/channels/sip/config_parser.c b/channels/sip/config_parser.c
index 4a03189..98b42d5 100644 (file)
--- a/channels/sip/config_parser.c
+++ b/channels/sip/config_parser.c
@@ -530,7 +530,48 @@ AST_TEST_DEFINE(sip_parse_register_line_test)
        ast_string_field_free_memory(reg);
        ast_free(reg);
 
-       /* ---Test reg12, add domain port --- */
+       /* ---Test reg 9, missing domain, expected to fail --- */
+       if (!(reg = ast_calloc_with_stringfields(1, struct sip_registry, 256))) {
+               goto alloc_fail;
+       } else if (!sip_parse_register_line(reg, default_expiry, reg9, 1)) {
+               ast_test_status_update(test,
+                               "Test 9, missing domain, expected to fail but did not.\n");
+               res = AST_TEST_FAIL;
+       }
+       ast_string_field_free_memory(reg);
+       ast_free(reg);
+
+       /* ---Test reg 10,  missing user, expected to fail --- */
+       if (!(reg = ast_calloc_with_stringfields(1, struct sip_registry, 256))) {
+               goto alloc_fail;
+       } else if (!sip_parse_register_line(reg, default_expiry, reg10, 1)) {
+               ast_test_status_update(test,
+                               "Test 10, missing user expected to fail but did not\n");
+               res = AST_TEST_FAIL;
+       }
+       ast_string_field_free_memory(reg);
+       ast_free(reg);
+
+       /* ---Test reg 11, no registry object, expected to fail--- */
+       if (!sip_parse_register_line(NULL, default_expiry, reg1, 1)) {
+               ast_test_status_update(test,
+                               "Test 11, no registry object, expected to fail but did not.\n");
+               res = AST_TEST_FAIL;
+       }
+
+       /* ---Test reg 12,  no registry line, expected to fail --- */
+       if (!(reg = ast_calloc_with_stringfields(1, struct sip_registry, 256))) {
+               goto alloc_fail;
+       } else if (!sip_parse_register_line(reg, default_expiry, NULL, 1)) {
+
+               ast_test_status_update(test,
+                               "Test 12, NULL register line expected to fail but did not.\n");
+               res = AST_TEST_FAIL;
+       }
+       ast_string_field_free_memory(reg);
+       ast_free(reg);
+
+       /* ---Test reg13, add domain port --- */
        if (!(reg = ast_calloc_with_stringfields(1, struct sip_registry, 256))) {
                goto alloc_fail;
        } else if (
@@ -553,11 +594,11 @@ AST_TEST_DEFINE(sip_parse_register_line_test)
                reg->callid_valid != FALSE          ||
                reg->ocseq != INITIAL_CSEQ) {
 
-               ast_test_status_update(test, "Test 12, add domain port failed.\n");
+               ast_test_status_update(test, "Test 13, add domain port failed.\n");
                res = AST_TEST_FAIL;
        }
 
-       /* ---Test reg13, domain port without secret --- */
+       /* ---Test reg14, domain port without secret --- */
        if (!(reg = ast_calloc_with_stringfields(1, struct sip_registry, 256))) {
                goto alloc_fail;
        } else if (
@@ -580,51 +621,10 @@ AST_TEST_DEFINE(sip_parse_register_line_test)
                reg->callid_valid != FALSE          ||
                reg->ocseq != INITIAL_CSEQ) {
 
-               ast_test_status_update(test, "Test 13, domain port without secret failed.\n");
+               ast_test_status_update(test, "Test 14, domain port without secret failed.\n");
                res = AST_TEST_FAIL;
 }
 
-       /* ---Test reg 9, missing domain, expected to fail --- */
-       if (!(reg = ast_calloc_with_stringfields(1, struct sip_registry, 256))) {
-               goto alloc_fail;
-       } else if (!sip_parse_register_line(reg, default_expiry, reg9, 1)) {
-               ast_test_status_update(test,
-                               "Test 9, missing domain, expected to fail but did not.\n");
-               res = AST_TEST_FAIL;
-       }
-       ast_string_field_free_memory(reg);
-       ast_free(reg);
-
-       /* ---Test reg 10,  missing user, expected to fail --- */
-       if (!(reg = ast_calloc_with_stringfields(1, struct sip_registry, 256))) {
-               goto alloc_fail;
-       } else if (!sip_parse_register_line(reg, default_expiry, reg10, 1)) {
-               ast_test_status_update(test,
-                               "Test 10, missing user expected to fail but did not\n");
-               res = AST_TEST_FAIL;
-       }
-       ast_string_field_free_memory(reg);
-       ast_free(reg);
-
-       /* ---Test reg 11, no registry object, expected to fail--- */
-       if (!sip_parse_register_line(NULL, default_expiry, reg1, 1)) {
-               ast_test_status_update(test,
-                               "Test 11, no registry object, expected to fail but did not.\n");
-               res = AST_TEST_FAIL;
-       }
-
-       /* ---Test reg 11,  no registry line, expected to fail --- */
-       if (!(reg = ast_calloc_with_stringfields(1, struct sip_registry, 256))) {
-               goto alloc_fail;
-       } else if (!sip_parse_register_line(reg, default_expiry, NULL, 1)) {
-
-               ast_test_status_update(test,
-                               "Test 11, NULL register line expected to fail but did not.\n");
-               res = AST_TEST_FAIL;
-       }
-       ast_string_field_free_memory(reg);
-       ast_free(reg);
-
 
        return res;