Improved targets for "run all the tests, no, all of them."

diff --git a/Makefile.am b/Makefile.am
index 87a5d8adccf476a1ad2f7836ef4902a1b4d463b9..784ff48dcd79c065109a465739df9a08bdc85446 100644 (file)
--- a/Makefile.am
+++ b/Makefile.am
@@ -69,26 +69,34 @@ doxygen:
 test: all
        $(top_builddir)/src/test/test
 
+need-chutney-path:
+       @if test ! -d "$$CHUTNEY_PATH"; then \
+               echo '$$CHUTNEY_PATH was not set.'; echo; \
+               echo "To run these tests, git clone https://git.torproject.org/chutney.git ; export CHUTNEY_PATH=\`pwd\`/chutney"; \
+               exit 1; \
+       fi
+
 # Note that test-network requires a copy of Chutney in $CHUTNEY_PATH.
 # Chutney can be cloned from https://git.torproject.org/chutney.git .
-test-network: all
+test-network: need-chutney-path all
        $(top_srcdir)/src/test/test-network.sh
 
-test-stem: $(TESTING_TOR_BINARY)
-       @if test -d "$$STEM_SOURCE_DIR"; then \
-               $(PYTHON) "$$STEM_SOURCE_DIR"/run_tests.py --tor $(TESTING_TOR_BINARY) --all --log notice --target RUN_ALL; \
-       else \
+need-stem-path:
+       @if test ! -d "$$STEM_SOURCE_DIR"; then \
                echo '$$STEM_SOURCE_DIR was not set.'; echo; \
                echo "To run these tests, git clone https://git.torproject.org/stem.git/ ; export STEM_SOURCE_DIR=\`pwd\`/stem"; \
+               exit 1; \
        fi
 
-test-stem-full: $(TESTING_TOR_BINARY)
-       @if test -d "$$STEM_SOURCE_DIR"; then \
-               $(PYTHON) "$$STEM_SOURCE_DIR"/run_tests.py --tor $(TESTING_TOR_BINARY) --all --log notice --target RUN_ALL,ONLINE -v; \
-       else \
-               echo '$$STEM_SOURCE_DIR was not set.'; echo; \
-               echo "To run these tests, git clone https://git.torproject.org/stem.git/ ; export STEM_SOURCE_DIR=\`pwd\`/stem"; \
-       fi
+test-stem: need-stem-path $(TESTING_TOR_BINARY)
+       @$(PYTHON) "$$STEM_SOURCE_DIR"/run_tests.py --tor $(TESTING_TOR_BINARY) --all --log notice --target RUN_ALL;
+
+test-stem-full: need-stem-path $(TESTING_TOR_BINARY)
+       @$(PYTHON) "$$STEM_SOURCE_DIR"/run_tests.py --tor $(TESTING_TOR_BINARY) --all --log notice --target RUN_ALL,ONLINE -v;
+
+test-full: need-stem-path need-chutney-path check test-network test-stem
+
+test-full-online: need-stem-path need-chutney-path check test-network test-stem-full
 
 reset-gcov:
        rm -f $(top_builddir)/src/*/*.gcda $(top_builddir)/src/*/*/*.gcda

diff --git a/changes/early-check-paths b/changes/early-check-paths
new file mode 100644 (file)
index 0000000..44bebbe
--- /dev/null
+++ b/changes/early-check-paths
@@ -0,0 +1,3 @@
+  o Testing:
+    - When running test-network or test-stem, check for the absence
+      of stem/chutney before doing any build operations.
\ No newline at end of file

diff --git a/changes/test-full b/changes/test-full
new file mode 100644 (file)
index 0000000..5ff51db
--- /dev/null
+++ b/changes/test-full
@@ -0,0 +1,3 @@
+  o Testing:
+    - Add new 'test-full' and 'test-full-online' targets to run all tests,
+      including integration tests with stem and chutney.