[tor-commits] [tor/master] Integrate backtrace test into the automake test suite.

[nickm at torproject.org]

commit 0e89abfa73ee8494f1e654823ab68282858038b0
Author: cypherpunks <cypherpunks at torproject.org>
Date:   Thu Mar 5 11:05:17 2015 +0100

    Integrate backtrace test into the automake test suite.
    
    For this to work bt_test.py now returns an exit code indicating success or
    failure. Additionally, check-local and its specific dependencies are now
    obsolete so they are removed.
---
 .gitignore             |    1 +
 configure.ac           |    1 +
 src/test/bt_test.py    |    4 ++--
 src/test/include.am    |   16 +---------------
 src/test/test_bt.sh.in |    9 +++++++++
 5 files changed, 14 insertions(+), 17 deletions(-)

diff --git a/.gitignore b/.gitignore
index da5fc47..6139201 100644
--- a/.gitignore
+++ b/.gitignore
@@ -176,6 +176,7 @@ cscope.*
 /src/test/test_workqueue.exe
 /src/test/test_zero_length_keys.sh
 /src/test/test_ntor.sh
+/src/test/test_bt.sh
 
 # /src/tools/
 /src/tools/tor-checkkey
diff --git a/configure.ac b/configure.ac
index 184211b..098a15c 100644
--- a/configure.ac
+++ b/configure.ac
@@ -1664,6 +1664,7 @@ AC_CONFIG_FILES([
         scripts/maint/updateVersions.pl
         src/test/test_zero_length_keys.sh
         src/test/test_ntor.sh
+        src/test/test_bt.sh
 ])
 
 if test x$asciidoc = xtrue && test "$ASCIIDOC" = "none" ; then
diff --git a/src/test/bt_test.py b/src/test/bt_test.py
index 0afe797..e694361 100755
--- a/src/test/bt_test.py
+++ b/src/test/bt_test.py
@@ -36,7 +36,7 @@ LINES = sys.stdin.readlines()
 for I in range(len(LINES)):
     if matches(LINES[I:], FUNCNAMES):
         print("OK")
-        break
+        sys.exit(0)
 else:
     print("BAD")
-
+    sys.exit(1)
diff --git a/src/test/include.am b/src/test/include.am
index 0ed3cb3..3c9eed8 100644
--- a/src/test/include.am
+++ b/src/test/include.am
@@ -2,7 +2,7 @@ TESTS += src/test/test src/test/test-slow src/test/test-memwipe \
 	src/test/test_zero_length_keys.sh
 
 if USEPYTHON
-TESTS += src/test/test_ntor.sh
+TESTS += src/test/test_ntor.sh src/test/test_bt.sh
 endif
 
 TEST_EXTENSIONS = .sh
@@ -154,13 +154,6 @@ src_test_test_ntor_cl_LDADD = src/or/libtor.a src/common/libor.a \
 	@TOR_OPENSSL_LIBS@ @TOR_LIB_WS32@ @TOR_LIB_GDI@ @CURVE25519_LIBS@
 src_test_test_ntor_cl_AM_CPPFLAGS =	       \
 	-I"$(top_srcdir)/src/or"
-NTOR_TEST_DEPS=src/test/test-ntor-cl
-
-if COVERAGE_ENABLED
-CMDLINE_TEST_TOR = $(top_builddir)/src/or/tor-cov
-else
-CMDLINE_TEST_TOR = $(top_builddir)/src/or/tor
-endif
 
 noinst_PROGRAMS += src/test/test-bt-cl
 src_test_test_bt_cl_SOURCES = src/test/test_bt_cl.c
@@ -170,13 +163,6 @@ src_test_test_bt_cl_LDADD = src/common/libor-testing.a \
 src_test_test_bt_cl_CFLAGS = $(AM_CFLAGS) $(TEST_CFLAGS)
 src_test_test_bt_cl_CPPFLAGS= $(src_test_AM_CPPFLAGS) $(TEST_CPPFLAGS)
 
-
-check-local: $(NTOR_TEST_DEPS) $(CMDLINE_TEST_TOR)
-if USEPYTHON
-	$(top_builddir)/src/test/test-bt-cl assert | $(PYTHON) $(top_srcdir)/src/test/bt_test.py
-	$(top_builddir)/src/test/test-bt-cl crash | $(PYTHON) $(top_srcdir)/src/test/bt_test.py
-endif
-
 EXTRA_DIST += \
 	src/test/bt_test.py \
 	src/test/ntor_ref.py \
diff --git a/src/test/test_bt.sh.in b/src/test/test_bt.sh.in
new file mode 100644
index 0000000..d249875
--- /dev/null
+++ b/src/test/test_bt.sh.in
@@ -0,0 +1,9 @@
+#!/bin/sh
+# Test backtrace functionality.
+
+exitcode=0
+
+ at builddir@/src/test/test-bt-cl assert | @PYTHON@ @abs_top_srcdir@/src/test/bt_test.py || exitcode=1
+ at builddir@/src/test/test-bt-cl crash | @PYTHON@ @abs_top_srcdir@/src/test/bt_test.py || exitcode=1
+
+exit ${exitcode}