Unit tests: Use AST_TEST_DEFINE in conditional code only.

If AST_TEST_DEFINE is not conditional to TEST_FRAMEWORK it produces dead
code.  This places all existing unit tests into a conditional block if
they weren't already.

ASTERISK-26211 #close

Change-Id: I8ef83ee11cbc991b07b7a37ecb41433e8c734686

diff --git a/apps/app_voicemail.c b/apps/app_voicemail.c
index 56ea1791ff638195519b8ffe14d633a795dbd05f..4f8302b03c2e9e169e441c40e047b7d6681e2ff1 100644 (file)
--- a/apps/app_voicemail.c
+++ b/apps/app_voicemail.c
@@ -11828,6 +11828,7 @@ static int append_mailbox(const char *context, const char *box, const char *data
        return 0;
 }
 
+#ifdef TEST_FRAMEWORK
 AST_TEST_DEFINE(test_voicemail_vmuser)
 {
        int res = 0;
@@ -12015,6 +12016,7 @@ AST_TEST_DEFINE(test_voicemail_vmuser)
        free_user(vmu);
        return res ? AST_TEST_FAIL : AST_TEST_PASS;
 }
+#endif
 
 static int vm_box_exists(struct ast_channel *chan, const char *data) 
 {

diff --git a/channels/sip/config_parser.c b/channels/sip/config_parser.c
index 45221e08d3cda7fe18d0d8074b54cc100eb8e663..17e5321f5293b9a390c0493cb5e97c7481e8714b 100644 (file)
--- a/channels/sip/config_parser.c
+++ b/channels/sip/config_parser.c
@@ -274,6 +274,7 @@ int sip_parse_register_line(struct sip_registry *reg, int default_expiry, const
        return 0;
 }
 
+#ifdef TEST_FRAMEWORK
 AST_TEST_DEFINE(sip_parse_register_line_test)
 {
        int res = AST_TEST_PASS;
@@ -643,6 +644,7 @@ alloc_fail:
        ast_test_status_update(test, "Out of memory. \n");
        return res;
 }
+#endif
 
 int sip_parse_host(char *line, int lineno, char **hostname, int *portnum, enum sip_transport *transport)
 {
@@ -708,6 +710,7 @@ int sip_parse_host(char *line, int lineno, char **hostname, int *portnum, enum s
        return 0;
 }
 
+#ifdef TEST_FRAMEWORK
 AST_TEST_DEFINE(sip_parse_host_line_test)
 {
        int res = AST_TEST_PASS;
@@ -787,6 +790,7 @@ AST_TEST_DEFINE(sip_parse_host_line_test)
        return res;
 
 }
+#endif
 
 /*! \brief Parse the comma-separated nat= option values */
 void sip_parse_nat_option(const char *value, struct ast_flags *mask, struct ast_flags *flags)
@@ -834,6 +838,7 @@ void sip_parse_nat_option(const char *value, struct ast_flags *mask, struct ast_
        }
 }
 
+#ifdef TEST_FRAMEWORK
 #define TEST_FORCE_RPORT      1 << 0
 #define TEST_COMEDIA          1 << 1
 #define TEST_AUTO_FORCE_RPORT 1 << 2
@@ -904,6 +909,8 @@ AST_TEST_DEFINE(sip_parse_nat_test)
 
        return res;
 }
+#endif
+
 /*! \brief SIP test registration */
 void sip_config_parser_register_tests(void)
 {

diff --git a/channels/sip/reqresp_parser.c b/channels/sip/reqresp_parser.c
index 9b435d00c9fe49cb7cdb02a593cdbe93bc907e2e..30a718ad186afd4eb09e6b7446b2164949e1484b 100644 (file)
--- a/channels/sip/reqresp_parser.c
+++ b/channels/sip/reqresp_parser.c
@@ -220,7 +220,7 @@ int parse_uri_full(char *uri, const char *scheme, char **user, char **pass,
        return error;
 }
 
-
+#ifdef TEST_FRAMEWORK
 AST_TEST_DEFINE(sip_parse_uri_full_test)
 {
        int res = AST_TEST_PASS;
@@ -429,7 +429,7 @@ AST_TEST_DEFINE(sip_parse_uri_full_test)
 
        return res;
 }
-
+#endif
 
 int parse_uri(char *uri, const char *scheme, char **user, char **pass,
              char **hostport, char **transport) {
@@ -445,6 +445,7 @@ int parse_uri(char *uri, const char *scheme, char **user, char **pass,
        return ret;
 }
 
+#ifdef TEST_FRAMEWORK
 AST_TEST_DEFINE(sip_parse_uri_test)
 {
        int res = AST_TEST_PASS;
@@ -590,6 +591,7 @@ AST_TEST_DEFINE(sip_parse_uri_test)
 
        return res;
 }
+#endif
 
 /*! \brief  Get caller id name from SIP headers, copy into output buffer
  *
@@ -720,6 +722,7 @@ const char *get_calleridname(const char *input, char *output, size_t outputsize)
        return input;
 }
 
+#ifdef TEST_FRAMEWORK
 AST_TEST_DEFINE(get_calleridname_test)
 {
        int res = AST_TEST_PASS;
@@ -802,6 +805,7 @@ AST_TEST_DEFINE(get_calleridname_test)
 
        return res;
 }
+#endif
 
 int get_name_and_number(const char *hdr, char **name, char **number)
 {
@@ -843,6 +847,7 @@ int get_name_and_number(const char *hdr, char **name, char **number)
        return 0;
 }
 
+#ifdef TEST_FRAMEWORK
 AST_TEST_DEFINE(get_name_and_number_test)
 {
        int res = AST_TEST_PASS;
@@ -947,6 +952,7 @@ AST_TEST_DEFINE(get_name_and_number_test)
 
        return res;
 }
+#endif
 
 int get_in_brackets_const(const char *src,const char **start,int *length)
 {
@@ -1079,6 +1085,7 @@ char *get_in_brackets(char *tmp)
        return out;
 }
 
+#ifdef TEST_FRAMEWORK
 AST_TEST_DEFINE(get_in_brackets_test)
 {
        int res = AST_TEST_PASS;
@@ -1155,7 +1162,7 @@ AST_TEST_DEFINE(get_in_brackets_test)
 
        return res;
 }
-
+#endif
 
 int parse_name_andor_addr(char *uri, const char *scheme, char **name,
                          char **user, char **pass, char **hostport,
@@ -1201,6 +1208,7 @@ int parse_name_andor_addr(char *uri, const char *scheme, char **name,
        return parse_uri_full(uri, scheme, user, pass, hostport, params, headers, residue2);
 }
 
+#ifdef TEST_FRAMEWORK
 AST_TEST_DEFINE(parse_name_andor_addr_test)
 {
        int res = AST_TEST_PASS;
@@ -1330,6 +1338,7 @@ AST_TEST_DEFINE(parse_name_andor_addr_test)
 
        return res;
 }
+#endif
 
 int get_comma(char *in, char **out)
 {
@@ -1426,6 +1435,7 @@ int parse_contact_header(char *contactheader, struct contactliststruct *contactl
        return last;
 }
 
+#ifdef TEST_FRAMEWORK
 AST_TEST_DEFINE(parse_contact_header_test)
 {
        int res = AST_TEST_PASS;
@@ -1571,6 +1581,7 @@ AST_TEST_DEFINE(parse_contact_header_test)
 
        return res;
 }
+#endif
 
 /*!
  * \brief Parse supported header in incoming packet
@@ -1658,6 +1669,7 @@ unsigned int parse_sip_options(const char *options, char *unsupported, size_t un
        return profile;
 }
 
+#ifdef TEST_FRAMEWORK
 AST_TEST_DEFINE(sip_parse_options_test)
 {
        int res = AST_TEST_PASS;
@@ -1795,6 +1807,7 @@ AST_TEST_DEFINE(sip_parse_options_test)
 
        return res;
 }
+#endif
 
 /*! \brief helper routine for sip_uri_cmp to compare URI parameters
  *
@@ -2149,6 +2162,7 @@ int sip_uri_cmp(const char *input1, const char *input2)
 #define URI_CMP_MATCH 0
 #define URI_CMP_NOMATCH 1
 
+#ifdef TEST_FRAMEWORK
 AST_TEST_DEFINE(sip_uri_cmp_test)
 {
        static const struct {
@@ -2265,6 +2279,7 @@ AST_TEST_DEFINE(sip_uri_cmp_test)
 
        return test_res;
 }
+#endif
 
 void free_via(struct sip_via *v)
 {
@@ -2351,6 +2366,7 @@ struct sip_via *parse_via(const char *header)
        return v;
 }
 
+#ifdef TEST_FRAMEWORK
 AST_TEST_DEFINE(parse_via_test)
 {
        int res = AST_TEST_PASS;
@@ -2528,6 +2544,7 @@ AST_TEST_DEFINE(parse_via_test)
        }
        return res;
 }
+#endif
 
 void sip_request_parser_register_tests(void)
 {
@@ -2542,6 +2559,7 @@ void sip_request_parser_register_tests(void)
        AST_TEST_REGISTER(sip_uri_cmp_test);
        AST_TEST_REGISTER(parse_via_test);
 }
+
 void sip_request_parser_unregister_tests(void)
 {
        AST_TEST_UNREGISTER(sip_parse_uri_test);

diff --git a/funcs/func_curl.c b/funcs/func_curl.c
index 1795a8128bc74c6cb8418c3e8fd77734b9598f2c..991da309342e476028907e018d442ff92ae51811 100644 (file)
--- a/funcs/func_curl.c
+++ b/funcs/func_curl.c
@@ -794,6 +794,7 @@ static struct ast_custom_function acf_curlopt = {
        .write = acf_curlopt_write,
 };
 
+#ifdef TEST_FRAMEWORK
 AST_TEST_DEFINE(vulnerable_url)
 {
        const char *bad_urls [] = {
@@ -841,6 +842,7 @@ AST_TEST_DEFINE(vulnerable_url)
 
        return res;
 }
+#endif
 
 static int unload_module(void)
 {