test1531: Add timeout

Previously, the macro TEST_HANG_TIMEOUT was unused, but since there is
looping going on, we might as well add timing instead of removing it.

Closes #2853

diff --git a/tests/libtest/lib1531.c b/tests/libtest/lib1531.c
index 5ee617e52a3ab54c3823420ef8420812342dd1ce..953f062d8c43cd407c201e60725a1fe57772986c 100644 (file)
--- a/tests/libtest/lib1531.c
+++ b/tests/libtest/lib1531.c
@@ -39,6 +39,8 @@ int test(char *URL)
   int msgs_left; /* how many messages are left */
   int res = CURLE_OK;
 
+  start_test_timing();
+
   global_init(CURL_GLOBAL_ALL);
 
   /* Allocate one CURL handle per transfer */
@@ -59,6 +61,8 @@ int test(char *URL)
   /* we start some action by calling perform right away */
   curl_multi_perform(multi_handle, &still_running);
 
+  abort_on_test_timeout();
+
   do {
     struct timeval timeout;
     int rc; /* select() return code */
@@ -127,6 +131,8 @@ int test(char *URL)
       curl_multi_perform(multi_handle, &still_running);
       break;
     }
+
+    abort_on_test_timeout();
   } while(still_running);
 
   /* See how the transfers went */
@@ -136,14 +142,17 @@ int test(char *URL)
       printf("HTTP transfer completed with status %d\n", msg->data.result);
       break;
     }
+
+    abort_on_test_timeout();
   } while(msg);
 
+test_cleanup:
   curl_multi_cleanup(multi_handle);
 
   /* Free the CURL handles */
   curl_easy_cleanup(easy);
   curl_global_cleanup();
 
-  return 0;
+  return res;
 }