[tor-commits] [tor/master] Make test_workqueue.c faster, and on-by-default.

[nickm at torproject.org]

commit 19440b9e58b3d824057abd3a0faf08fd7cb891ff
Author: Nick Mathewson <nickm at torproject.org>
Date:   Fri Jul 3 14:37:16 2015 -0400

    Make test_workqueue.c faster, and on-by-default.
    
    Instead of having a 30-second timer be the only way to end the test,
    add a 2 second shutdown timer when the test is actually about to be over.
---
 changes/faster-workqueue-test |    3 +++
 src/test/include.am           |    2 +-
 src/test/test_workqueue.c     |    7 ++++++-
 3 files changed, 10 insertions(+), 2 deletions(-)

diff --git a/changes/faster-workqueue-test b/changes/faster-workqueue-test
new file mode 100644
index 0000000..16489e4
--- /dev/null
+++ b/changes/faster-workqueue-test
@@ -0,0 +1,3 @@
+  o Testing:
+    - The test_workqueue program now runs faster, and is enabled by
+      default as a part of "make check".
diff --git a/src/test/include.am b/src/test/include.am
index 0ef556a..7406c84 100644
--- a/src/test/include.am
+++ b/src/test/include.am
@@ -6,7 +6,7 @@ TESTSCRIPTS += src/test/test_ntor.sh src/test/test_bt.sh
 endif
 
 TESTS += src/test/test src/test/test-slow src/test/test-memwipe \
-	$(TESTSCRIPTS)
+	src/test/test_workqueue $(TESTSCRIPTS)
 
 ### This is a lovely feature, but it requires automake >= 1.12, and Tor
 ###  doesn't require that yet.  Below is a kludge to work around.
diff --git a/src/test/test_workqueue.c b/src/test/test_workqueue.c
index 77053b0..edefb29 100644
--- a/src/test/test_workqueue.c
+++ b/src/test/test_workqueue.c
@@ -214,6 +214,7 @@ add_n_work_items(threadpool_t *tp, int n)
   while (n_queued++ < n) {
     ent = add_work(tp);
     if (! ent) {
+      puts("Z");
       tor_event_base_loopexit(tor_libevent_get_base(), NULL);
       return -1;
     }
@@ -287,6 +288,10 @@ replysock_readable_cb(tor_socket_t sock, short what, void *arg)
     shutting_down = 1;
     threadpool_queue_update(tp, NULL,
                              workqueue_do_shutdown, NULL, NULL);
+    {
+      struct timeval limit = { 2, 0 };
+      tor_event_base_loopexit(tor_libevent_get_base(), &limit);
+    }
   }
 }
 
@@ -392,7 +397,7 @@ main(int argc, char **argv)
   }
 
   {
-    struct timeval limit = { 30, 0 };
+    struct timeval limit = { 180, 0 };
     tor_event_base_loopexit(tor_libevent_get_base(), &limit);
   }