Selaa lähdekoodia

Unit tests: Split functions into separate tests

bel2125 7 vuotta sitten
vanhempi
commit
c5582b0107
3 muutettua tiedostoa jossa 61 lisäystä ja 21 poistoa
  1. 2 2
      appveyor.yml
  2. 10 2
      test/CMakeLists.txt
  3. 49 17
      test/private.c

+ 2 - 2
appveyor.yml

@@ -69,7 +69,7 @@ environment:
       configuration: Release
       configuration: Release
       platform: x64
       platform: x64
     # Debug builds
     # Debug builds
-    - id: Full-x86
+    - id: Full-x86-Debug
       compiler: msvc-19-seh
       compiler: msvc-19-seh
       build_shared: NO
       build_shared: NO
       no_files: NO
       no_files: NO
@@ -80,7 +80,7 @@ environment:
       no_caching: NO
       no_caching: NO
       configuration: Debug
       configuration: Debug
       platform: x86
       platform: x86
-    - id: Full-x64
+    - id: Full-x64-Debug
       compiler: msvc-19-seh
       compiler: msvc-19-seh
       build_shared: NO
       build_shared: NO
       no_files: NO
       no_files: NO

+ 10 - 2
test/CMakeLists.txt

@@ -150,8 +150,16 @@ endmacro(civetweb_add_test)
 
 
 # Tests of private functions
 # Tests of private functions
 civetweb_add_test(Private "HTTP Message")
 civetweb_add_test(Private "HTTP Message")
-civetweb_add_test(Private "URL Parsing")
-civetweb_add_test(Private "Internal Parsing")
+civetweb_add_test(Private "HTTP Keep Alive")
+civetweb_add_test(Private "URL Parsing 1")
+civetweb_add_test(Private "URL Parsing 2")
+civetweb_add_test(Private "URL Parsing 3")
+civetweb_add_test(Private "Internal Parsing 1")
+civetweb_add_test(Private "Internal Parsing 2")
+civetweb_add_test(Private "Internal Parsing 3")
+civetweb_add_test(Private "Internal Parsing 4")
+civetweb_add_test(Private "Internal Parsing 5")
+civetweb_add_test(Private "Internal Parsing 6")
 civetweb_add_test(Private "Encode Decode")
 civetweb_add_test(Private "Encode Decode")
 civetweb_add_test(Private "Mask Data")
 civetweb_add_test(Private "Mask Data")
 civetweb_add_test(Private "Date Parsing")
 civetweb_add_test(Private "Date Parsing")

+ 49 - 17
test/private.c

@@ -871,33 +871,65 @@ make_private_suite(void)
 	Suite *const suite = suite_create("Private");
 	Suite *const suite = suite_create("Private");
 
 
 	TCase *const tcase_http_message = tcase_create("HTTP Message");
 	TCase *const tcase_http_message = tcase_create("HTTP Message");
-	TCase *const tcase_url_parsing = tcase_create("URL Parsing");
-	TCase *const tcase_internal_parse = tcase_create("Internal Parsing");
+	TCase *const tcase_http_keep_alive = tcase_create("HTTP Keep Alive");
+	TCase *const tcase_url_parsing_1 = tcase_create("URL Parsing 1");
+	TCase *const tcase_url_parsing_2 = tcase_create("URL Parsing 2");
+	TCase *const tcase_url_parsing_3 = tcase_create("URL Parsing 3");
+	TCase *const tcase_internal_parse_1 = tcase_create("Internal Parsing 1");
+	TCase *const tcase_internal_parse_2 = tcase_create("Internal Parsing 2");
+	TCase *const tcase_internal_parse_3 = tcase_create("Internal Parsing 3");
+	TCase *const tcase_internal_parse_4 = tcase_create("Internal Parsing 4");
+	TCase *const tcase_internal_parse_5 = tcase_create("Internal Parsing 5");
+	TCase *const tcase_internal_parse_6 = tcase_create("Internal Parsing 6");
 	TCase *const tcase_encode_decode = tcase_create("Encode Decode");
 	TCase *const tcase_encode_decode = tcase_create("Encode Decode");
 	TCase *const tcase_mask_data = tcase_create("Mask Data");
 	TCase *const tcase_mask_data = tcase_create("Mask Data");
 	TCase *const tcase_parse_date_string = tcase_create("Date Parsing");
 	TCase *const tcase_parse_date_string = tcase_create("Date Parsing");
 	TCase *const tcase_sha1 = tcase_create("SHA1");
 	TCase *const tcase_sha1 = tcase_create("SHA1");
 
 
 	tcase_add_test(tcase_http_message, test_parse_http_message);
 	tcase_add_test(tcase_http_message, test_parse_http_message);
-	tcase_add_test(tcase_http_message, test_should_keep_alive);
 	tcase_set_timeout(tcase_http_message, civetweb_min_test_timeout);
 	tcase_set_timeout(tcase_http_message, civetweb_min_test_timeout);
 	suite_add_tcase(suite, tcase_http_message);
 	suite_add_tcase(suite, tcase_http_message);
 
 
-	tcase_add_test(tcase_url_parsing, test_match_prefix);
-	tcase_add_test(tcase_url_parsing,
+	tcase_add_test(tcase_http_keep_alive, test_should_keep_alive);
+	tcase_set_timeout(tcase_http_keep_alive, civetweb_min_test_timeout);
+	suite_add_tcase(suite, tcase_http_keep_alive);
+
+	tcase_add_test(tcase_url_parsing_1, test_match_prefix);
+	tcase_set_timeout(tcase_url_parsing_1, civetweb_min_test_timeout);
+	suite_add_tcase(suite, tcase_url_parsing_1);
+
+	tcase_add_test(tcase_url_parsing_2,
 	               test_remove_double_dots_and_double_slashes);
 	               test_remove_double_dots_and_double_slashes);
-	tcase_add_test(tcase_url_parsing, test_is_valid_uri);
-	tcase_set_timeout(tcase_url_parsing, civetweb_min_test_timeout);
-	suite_add_tcase(suite, tcase_url_parsing);
-
-	tcase_add_test(tcase_internal_parse, test_next_option);
-	tcase_add_test(tcase_internal_parse, test_skip_quoted);
-	tcase_add_test(tcase_internal_parse, test_mg_strcasestr);
-	tcase_add_test(tcase_internal_parse, test_alloc_vprintf);
-	tcase_add_test(tcase_internal_parse, test_mg_vsnprintf);
-	tcase_add_test(tcase_internal_parse, test_parse_port_string);
-	tcase_set_timeout(tcase_internal_parse, civetweb_min_test_timeout);
-	suite_add_tcase(suite, tcase_internal_parse);
+	tcase_set_timeout(tcase_url_parsing_2, civetweb_min_test_timeout);
+	suite_add_tcase(suite, tcase_url_parsing_2);
+
+	tcase_add_test(tcase_url_parsing_3, test_is_valid_uri);
+	tcase_set_timeout(tcase_url_parsing_3, civetweb_min_test_timeout);
+	suite_add_tcase(suite, tcase_url_parsing_3);
+
+	tcase_add_test(tcase_internal_parse_1, test_next_option);
+	tcase_set_timeout(tcase_internal_parse_1, civetweb_min_test_timeout);
+	suite_add_tcase(suite, tcase_internal_parse_1);
+
+	tcase_add_test(tcase_internal_parse_2, test_skip_quoted);
+	tcase_set_timeout(tcase_internal_parse_2, civetweb_min_test_timeout);
+	suite_add_tcase(suite, tcase_internal_parse_2);
+
+	tcase_add_test(tcase_internal_parse_3, test_mg_strcasestr);
+	tcase_set_timeout(tcase_internal_parse_3, civetweb_min_test_timeout);
+	suite_add_tcase(suite, tcase_internal_parse_3);
+
+	tcase_add_test(tcase_internal_parse_4, test_alloc_vprintf);
+	tcase_set_timeout(tcase_internal_parse_4, civetweb_min_test_timeout);
+	suite_add_tcase(suite, tcase_internal_parse_4);
+
+	tcase_add_test(tcase_internal_parse_5, test_mg_vsnprintf);
+	tcase_set_timeout(tcase_internal_parse_5, civetweb_min_test_timeout);
+	suite_add_tcase(suite, tcase_internal_parse_5);
+
+	tcase_add_test(tcase_internal_parse_6, test_parse_port_string);
+	tcase_set_timeout(tcase_internal_parse_6, civetweb_min_test_timeout);
+	suite_add_tcase(suite, tcase_internal_parse_6);
 
 
 	tcase_add_test(tcase_encode_decode, test_encode_decode);
 	tcase_add_test(tcase_encode_decode, test_encode_decode);
 	tcase_set_timeout(tcase_encode_decode, civetweb_min_test_timeout);
 	tcase_set_timeout(tcase_encode_decode, civetweb_min_test_timeout);