From 661a157553d46826d16da1201d638741211fd737 Mon Sep 17 00:00:00 2001
From: Richard Roberts <richard.jw.roberts@gmail.com>
Date: Thu, 13 Feb 2014 01:34:31 -0500
Subject: [PATCH 01/10] Working on standardizing/simplifying building unit
 tests and examples.  Much simpler cmake function to add a glob of tests,
 doing away with convenience libraries, and removing other options that we
 never change from their defaults.

---
 CMakeLists.txt                                |   5 +-
 cmake/GtsamTesting.cmake                      | 109 +++++++++++++++++-
 gtsam/base/CMakeLists.txt                     |  10 +-
 gtsam/base/tests/CMakeLists.txt               |   1 +
 gtsam/discrete/CMakeLists.txt                 |   7 +-
 gtsam/discrete/tests/CMakeLists.txt           |   1 +
 gtsam/geometry/CMakeLists.txt                 |  10 +-
 gtsam/geometry/tests/CMakeLists.txt           |   1 +
 gtsam/inference/CMakeLists.txt                |  10 +-
 gtsam/inference/tests/CMakeLists.txt          |   1 +
 gtsam/linear/CMakeLists.txt                   |  15 +--
 gtsam/linear/tests/CMakeLists.txt             |   6 +
 gtsam/navigation/CMakeLists.txt               |   7 +-
 gtsam/navigation/tests/CMakeLists.txt         |   1 +
 gtsam/nonlinear/CMakeLists.txt                |  10 +-
 gtsam/nonlinear/tests/CMakeLists.txt          |   1 +
 gtsam/slam/CMakeLists.txt                     |  11 +-
 gtsam/slam/tests/CMakeLists.txt               |   6 +
 gtsam/symbolic/CMakeLists.txt                 |  10 +-
 gtsam/symbolic/tests/CMakeLists.txt           |   1 +
 gtsam_unstable/base/CMakeLists.txt            |  10 +-
 gtsam_unstable/base/tests/CMakeLists.txt      |   1 +
 gtsam_unstable/discrete/CMakeLists.txt        |  13 +--
 gtsam_unstable/discrete/tests/CMakeLists.txt  |   1 +
 gtsam_unstable/dynamics/CMakeLists.txt        |  11 +-
 gtsam_unstable/dynamics/tests/CMakeLists.txt  |   1 +
 gtsam_unstable/geometry/CMakeLists.txt        |  11 +-
 gtsam_unstable/geometry/tests/CMakeLists.txt  |   1 +
 gtsam_unstable/nonlinear/CMakeLists.txt       |  15 +--
 gtsam_unstable/nonlinear/tests/CMakeLists.txt |   1 +
 gtsam_unstable/slam/CMakeLists.txt            |  13 +--
 gtsam_unstable/slam/tests/CMakeLists.txt      |   7 ++
 tests/CMakeLists.txt                          |  31 ++---
 wrap/CMakeLists.txt                           |   6 +-
 wrap/tests/CMakeLists.txt                     |   1 +
 35 files changed, 164 insertions(+), 182 deletions(-)
 create mode 100644 gtsam/base/tests/CMakeLists.txt
 create mode 100644 gtsam/discrete/tests/CMakeLists.txt
 create mode 100644 gtsam/geometry/tests/CMakeLists.txt
 create mode 100644 gtsam/inference/tests/CMakeLists.txt
 create mode 100644 gtsam/linear/tests/CMakeLists.txt
 create mode 100644 gtsam/navigation/tests/CMakeLists.txt
 create mode 100644 gtsam/nonlinear/tests/CMakeLists.txt
 create mode 100644 gtsam/slam/tests/CMakeLists.txt
 create mode 100644 gtsam/symbolic/tests/CMakeLists.txt
 create mode 100644 gtsam_unstable/base/tests/CMakeLists.txt
 create mode 100644 gtsam_unstable/discrete/tests/CMakeLists.txt
 create mode 100644 gtsam_unstable/dynamics/tests/CMakeLists.txt
 create mode 100644 gtsam_unstable/geometry/tests/CMakeLists.txt
 create mode 100644 gtsam_unstable/nonlinear/tests/CMakeLists.txt
 create mode 100644 gtsam_unstable/slam/tests/CMakeLists.txt
 create mode 100644 wrap/tests/CMakeLists.txt

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 337fd19a4..22e58564b 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -46,7 +46,6 @@ endif()
 # Set up options
 
 # Configurable Options
-option(GTSAM_BUILD_TESTS                 "Enable/Disable building of tests"          ON)
 option(GTSAM_BUILD_TIMING                "Enable/Disable building of timing scripts" OFF) # These do not currently work
 option(GTSAM_BUILD_EXAMPLES              "Enable/Disable building of examples"       ON)
 if(GTSAM_UNSTABLE_AVAILABLE)
@@ -83,9 +82,8 @@ endif()
 set(CPACK_SOURCE_GENERATOR "TGZ" CACHE STRING "CPack Default Source Generator")
 set(CPACK_GENERATOR        "TGZ" CACHE STRING "CPack Default Binary Generator")
 
-# Flags to determine whether tests and examples are build during 'make install'
+# Flags to determine whether examples are build during 'make install'
 # Note that these remove the targets from the 'all'
-option(GTSAM_DISABLE_TESTS_ON_INSTALL "Disables building tests during install" ON)
 option(GTSAM_DISABLE_EXAMPLES_ON_INSTALL "Disables building examples during install" OFF)
 
 # Pull in infrastructure
@@ -375,7 +373,6 @@ print_config_flag(${GTSAM_BUILD_TYPE_POSTFIXES}        "Put build type in librar
 if(GTSAM_UNSTABLE_AVAILABLE)
     print_config_flag(${GTSAM_BUILD_UNSTABLE}           "Build libgtsam_unstable        ")
 endif()
-print_config_flag(${GTSAM_DISABLE_TESTS_ON_INSTALL}     "No tests in all or install     ")
 print_config_flag(${GTSAM_DISABLE_EXAMPLES_ON_INSTALL}  "No examples in all or install  ")
 string(TOUPPER "${CMAKE_BUILD_TYPE}" cmake_build_type_toupper)
 if(NOT MSVC AND NOT XCODE_VERSION)
diff --git a/cmake/GtsamTesting.cmake b/cmake/GtsamTesting.cmake
index f1b7c5e54..dce09449e 100644
--- a/cmake/GtsamTesting.cmake
+++ b/cmake/GtsamTesting.cmake
@@ -2,8 +2,13 @@
 
 enable_testing()
 
+option(GTSAM_BUILD_TESTS                 "Enable/Disable building of tests"          ON)
+
 # Enable make check (http://www.cmake.org/Wiki/CMakeEmulateMakeCheck)
-add_custom_target(check COMMAND ${CMAKE_CTEST_COMMAND} -C $<CONFIGURATION> --output-on-failure)
+if(GTSAM_BUILD_TESTS)
+    add_custom_target(check COMMAND ${CMAKE_CTEST_COMMAND} -C $<CONFIGURATION> --output-on-failure)
+endif()
+
 add_custom_target(timing)
 
 # Add option for combining unit tests
@@ -13,6 +18,108 @@ else()
 	option(GTSAM_SINGLE_TEST_EXE "Combine unit tests into single executable (faster compile)" OFF)
 endif()
 
+# Macro for adding glob(s) of tests relative to the current directory.  Automatically
+# links the tests with CppUnitLite.  Separate multiple globPatterns, linkLibraries,
+# and excludedFiles using a semicolon, e.g. "testThings*.cpp;testOthers*.cpp".
+# Usage example:  gtsamAddTestsGlob(basic "test*.cpp" "testBroken.cpp" "gtsam;GeographicLib")
+macro(gtsamAddTestsGlob groupName globPatterns excludedFiles linkLibraries)
+	if(GTSAM_BUILD_TESTS)
+		# Add group target if it doesn't already exist
+	    if(NOT TARGET check.${groupName})
+			add_custom_target(check.${groupName} COMMAND ${CMAKE_CTEST_COMMAND} -C $<CONFIGURATION> --output-on-failure)
+		endif()
+	
+	    # Get all script files relative to the currect directory
+	    set(script_files_relative "")
+	    foreach(one_pattern IN ITEMS ${globPatterns})
+			message(STATUS "Filling test group ${groupName}")
+	        file(GLOB RELATIVE ${CMAKE_CURRENT_SOURCE_DIR} one_script_files "${one_pattern}")
+	        list(APPEND script_files_relative "${one_script_files}")
+	    endforeach()
+
+	    # Remove excluded scripts from the list
+	    if(excludedFiles)
+	    	list(REMOVE_ITEM script_files_relative excludedFiles)
+	    endif()
+	
+		# Get absolute paths
+		set(script_files "")
+		foreach(script_file IN ITEMS ${script_files_relative})
+			list(APPEND script_files "${CMAKE_CURRENT_SOURCE_DIR}/${script_file}")
+		endforeach()
+
+		# Separate into source files and headers (allows for adding headers to show up in
+		# MSVC and Xcode projects).
+		set(script_srcs "")
+		set(script_headers "")
+		foreach(script_file IN ITEMS ${script_files})
+			get_filename_component(script_ext ${script_file} EXT)
+			if(script_ext MATCHES "(h|H)")
+				list(APPEND script_headers ${script_file})
+			else()
+				list(APPEND script_srcs ${script_file})
+			endif()
+		endforeach()
+	
+		# Don't put test files in folders in MSVC and Xcode because they're already grouped
+		source_group("" FILES ${script_srcs} ${script_headers})
+
+		if(NOT GTSAM_SINGLE_TEST_EXE)
+			foreach(script_src IN ITEMS ${script_srcs})
+				# Get test base name
+				get_filename_component(script_name ${script_src} NAME_WE)
+			
+				# Add executable
+				add_executable(script_name ${script_src} ${script_headers})
+				target_link_libraries(${script_name} CppUnitLite ${linkLibraries})
+			
+				# Add target dependencies
+				add_test(NAME ${script_name} COMMAND ${script_name})
+				add_dependencies(check.${groupName} ${script_name})
+				add_dependencies(check ${script_name})
+				if(NOT MSVC AND NOT XCODE_VERSION)
+				  add_custom_target(${script_name}.run ${EXECUTABLE_OUTPUT_PATH}${script_name})
+				endif()
+			
+				# Add TOPSRCDIR
+				set_property(SOURCE ${script_src} APPEND PROPERTY COMPILE_DEFINITIONS "TOPSRCDIR=\"${PROJECT_SOURCE_DIR}\"")
+			
+				# Exclude from 'make all' and 'make install'
+				set_target_properties(${script_name} PROPERTIES EXCLUDE_FROM_ALL ON)
+			
+				# Configure target folder (for MSVC and Xcode)
+				set_property(TARGET ${script_name} PROPERTY FOLDER "Unit tests/${groupName}")
+			endforeach()
+		else()
+			# Create single unit test exe from all test scripts
+			set(target_name check_${groupName}_program)
+		
+			# Add executable
+			add_executable(${target_name} ${script_srcs} ${script_headers})
+			target_link_libraries(${target_name} CppUnitLite ${linkLibraries})
+		
+			# Only have a main function in one script - use preprocessor
+			set(rest_script_srcs ${script_srcs})
+			list(REMOVE_AT rest_script_srcs 0)
+			set_property(SOURCE ${rest_script_srcs} APPEND PROPERTY COMPILE_DEFINITIONS "main=static no_main")
+		
+			# Add target dependencies
+			add_test(NAME ${target_name} COMMAND ${target_name})
+			add_dependencies(check.${groupName} ${target_name})
+			add_dependencies(check ${target_name})
+		
+			# Add TOPSRCDIR
+			set_property(SOURCE ${script_srcs} APPEND PROPERTY COMPILE_DEFINITIONS "TOPSRCDIR=\"${PROJECT_SOURCE_DIR}\"")
+		
+			# Excluse from 'make all' and 'make install'
+			set_target_properties(${target_name} PROPERTIES EXCLUDE_FROM_ALL ON)
+
+			# Configure target folder (for MSVC and Xcode)
+			set_property(TARGET ${script_name} PROPERTY FOLDER "Unit tests")
+		endif()
+	endif()
+endmacro()
+
 # Macro for adding categorized tests in a "tests" folder, with 
 # optional exclusion of tests and convenience library linking options
 #  
diff --git a/gtsam/base/CMakeLists.txt b/gtsam/base/CMakeLists.txt
index 94e2473ed..50145846e 100644
--- a/gtsam/base/CMakeLists.txt
+++ b/gtsam/base/CMakeLists.txt
@@ -5,16 +5,8 @@ install(FILES ${base_headers} DESTINATION include/gtsam/base)
 file(GLOB base_headers_tree "treeTraversal/*.h")
 install(FILES ${base_headers_tree} DESTINATION include/gtsam/base/treeTraversal)
 
-# Files to exclude from compilation of tests and timing scripts
-set(base_excluded_files
-# "${CMAKE_CURRENT_SOURCE_DIR}/tests/testTypedDiscreteFactor.cpp" # Example of excluding a test
-#    ""  # Add to this list, with full path, to exclude
-)
-
 # Build tests
-if (GTSAM_BUILD_TESTS)
-    gtsam_add_subdir_tests(base "gtsam" "gtsam" "${base_excluded_files}") 
-endif(GTSAM_BUILD_TESTS)
+add_subdirectory(tests)
 
 # Build timing scripts
 if (GTSAM_BUILD_TIMING)
diff --git a/gtsam/base/tests/CMakeLists.txt b/gtsam/base/tests/CMakeLists.txt
new file mode 100644
index 000000000..8e99ef5ba
--- /dev/null
+++ b/gtsam/base/tests/CMakeLists.txt
@@ -0,0 +1 @@
+gtsamAddTestsGlob(base "test*.cpp" "" "gtsam")
diff --git a/gtsam/discrete/CMakeLists.txt b/gtsam/discrete/CMakeLists.txt
index 3f1a57e0b..3c5602080 100644
--- a/gtsam/discrete/CMakeLists.txt
+++ b/gtsam/discrete/CMakeLists.txt
@@ -4,13 +4,8 @@ file(GLOB discrete_headers "*.h")
 # FIXME: exclude headers
 install(FILES ${discrete_headers} DESTINATION include/gtsam/discrete)
 
-# Exclude tests that don't work
-set (discrete_excluded_tests "")
-
 # Add all tests
-if (GTSAM_BUILD_TESTS)
-    gtsam_add_subdir_tests(discrete "gtsam" "gtsam" "${discrete_excluded_tests}")
-endif()
+add_subdirectory(tests)
 
 # Build timing scripts
 #if (GTSAM_BUILD_TIMING)
diff --git a/gtsam/discrete/tests/CMakeLists.txt b/gtsam/discrete/tests/CMakeLists.txt
new file mode 100644
index 000000000..e968fac91
--- /dev/null
+++ b/gtsam/discrete/tests/CMakeLists.txt
@@ -0,0 +1 @@
+gtsamAddTestsGlob(discrete "test*.cpp" "" "gtsam")
diff --git a/gtsam/geometry/CMakeLists.txt b/gtsam/geometry/CMakeLists.txt
index 96b84193b..f72f965ea 100644
--- a/gtsam/geometry/CMakeLists.txt
+++ b/gtsam/geometry/CMakeLists.txt
@@ -2,16 +2,8 @@
 file(GLOB geometry_headers "*.h")
 install(FILES ${geometry_headers} DESTINATION include/gtsam/geometry)
 
-# Files to exclude from compilation of tests and timing scripts
-set(geometry_excluded_files
-# "${CMAKE_CURRENT_SOURCE_DIR}/tests/testTypedDiscreteFactor.cpp" # Example of excluding a test     
-    ""  # Add to this list, with full path, to exclude
-)
-
 # Build tests
-if (GTSAM_BUILD_TESTS)
-    gtsam_add_subdir_tests(geometry "gtsam" "gtsam" "${geometry_excluded_files}") 
-endif(GTSAM_BUILD_TESTS)
+add_subdirectory(tests)
 
 # Build timing scripts
 if (GTSAM_BUILD_TIMING)
diff --git a/gtsam/geometry/tests/CMakeLists.txt b/gtsam/geometry/tests/CMakeLists.txt
new file mode 100644
index 000000000..b1499ee9d
--- /dev/null
+++ b/gtsam/geometry/tests/CMakeLists.txt
@@ -0,0 +1 @@
+gtsamAddTestsGlob(geometry "test*.cpp" "" "gtsam")
diff --git a/gtsam/inference/CMakeLists.txt b/gtsam/inference/CMakeLists.txt
index d5c994b00..d5c37d976 100644
--- a/gtsam/inference/CMakeLists.txt
+++ b/gtsam/inference/CMakeLists.txt
@@ -2,16 +2,8 @@
 file(GLOB inference_headers "*.h")
 install(FILES ${inference_headers} DESTINATION include/gtsam/inference)
 
-# Files to exclude from compilation of tests and timing scripts
-set(inference_excluded_files
-# "${CMAKE_CURRENT_SOURCE_DIR}/tests/testTypedDiscreteFactor.cpp" # Example of excluding a test     
-    ""  # Add to this list, with full path, to exclude
-)
-
 # Build tests
-if (GTSAM_BUILD_TESTS)
-    gtsam_add_subdir_tests(inference "gtsam" "gtsam" "${inference_excluded_files}") 
-endif(GTSAM_BUILD_TESTS)
+add_subdirectory(tests)
 
 # Build timing scripts
 if (GTSAM_BUILD_TIMING)
diff --git a/gtsam/inference/tests/CMakeLists.txt b/gtsam/inference/tests/CMakeLists.txt
new file mode 100644
index 000000000..aaa01d775
--- /dev/null
+++ b/gtsam/inference/tests/CMakeLists.txt
@@ -0,0 +1 @@
+gtsamAddTestsGlob(inference "test*.cpp" "" "gtsam")
diff --git a/gtsam/linear/CMakeLists.txt b/gtsam/linear/CMakeLists.txt
index 4a93e2875..29eb60b93 100644
--- a/gtsam/linear/CMakeLists.txt
+++ b/gtsam/linear/CMakeLists.txt
@@ -2,21 +2,8 @@
 file(GLOB linear_headers "*.h")
 install(FILES ${linear_headers} DESTINATION include/gtsam/linear)
 
-# Files to exclude from compilation of tests and timing scripts
-set(linear_excluded_files
-# "${CMAKE_CURRENT_SOURCE_DIR}/tests/testTypedDiscreteFactor.cpp" # Example of excluding a test
-#    ""  # Add to this list, with full path, to exclude
-)
-
 # Build tests
-if (GTSAM_BUILD_TESTS)
-    gtsam_add_subdir_tests(linear "gtsam" "gtsam" "${linear_excluded_files}") 
-endif(GTSAM_BUILD_TESTS)
-
-if(MSVC)
-	set_property(SOURCE "${CMAKE_CURRENT_SOURCE_DIR}/tests/testSerializationLinear.cpp"
-		APPEND PROPERTY COMPILE_FLAGS "/bigobj")
-endif()
+add_subdirectory(tests)
 
 # Build timing scripts
 if (GTSAM_BUILD_TIMING)
diff --git a/gtsam/linear/tests/CMakeLists.txt b/gtsam/linear/tests/CMakeLists.txt
new file mode 100644
index 000000000..d1aafb4ea
--- /dev/null
+++ b/gtsam/linear/tests/CMakeLists.txt
@@ -0,0 +1,6 @@
+gtsamAddTestsGlob(linear "test*.cpp" "" "gtsam")
+
+if(MSVC)
+	set_property(SOURCE "${CMAKE_CURRENT_SOURCE_DIR}/testSerializationLinear.cpp"
+		APPEND PROPERTY COMPILE_FLAGS "/bigobj")
+endif()
diff --git a/gtsam/navigation/CMakeLists.txt b/gtsam/navigation/CMakeLists.txt
index ed599cf15..3e82af774 100644
--- a/gtsam/navigation/CMakeLists.txt
+++ b/gtsam/navigation/CMakeLists.txt
@@ -2,13 +2,8 @@
 file(GLOB navigation_headers "*.h")
 install(FILES ${navigation_headers} DESTINATION include/gtsam/navigation)
 
-# Exclude tests that don't work
-set (navigation_excluded_tests "")
-
 # Add all tests
-if (GTSAM_BUILD_TESTS)
-	gtsam_add_subdir_tests(navigation "gtsam" "gtsam" "${navigation_excluded_tests}") 
-endif()
+add_subdirectory(tests)
 
 # Build timing scripts
 if (GTSAM_BUILD_TIMING)
diff --git a/gtsam/navigation/tests/CMakeLists.txt b/gtsam/navigation/tests/CMakeLists.txt
new file mode 100644
index 000000000..4fd0b00d0
--- /dev/null
+++ b/gtsam/navigation/tests/CMakeLists.txt
@@ -0,0 +1 @@
+gtsamAddTestsGlob(navigation "test*.cpp" "" "gtsam")
diff --git a/gtsam/nonlinear/CMakeLists.txt b/gtsam/nonlinear/CMakeLists.txt
index f4cf0b585..b4d288104 100644
--- a/gtsam/nonlinear/CMakeLists.txt
+++ b/gtsam/nonlinear/CMakeLists.txt
@@ -2,16 +2,8 @@
 file(GLOB nonlinear_headers "*.h")
 install(FILES ${nonlinear_headers} DESTINATION include/gtsam/nonlinear)
 
-# Files to exclude from compilation of tests and timing scripts
-set(nonlinear_excluded_files
-# "${CMAKE_CURRENT_SOURCE_DIR}/tests/testTypedDiscreteFactor.cpp" # Example of excluding a test     
-    ""  # Add to this list, with full path, to exclude
-)
-
 # Build tests
-if (GTSAM_BUILD_TESTS)
-    gtsam_add_subdir_tests(nonlinear "gtsam" "gtsam" "${nonlinear_excluded_files}") 
-endif(GTSAM_BUILD_TESTS)
+add_subdirectory(tests)
 
 # Build timing scripts
 if (GTSAM_BUILD_TIMING)
diff --git a/gtsam/nonlinear/tests/CMakeLists.txt b/gtsam/nonlinear/tests/CMakeLists.txt
new file mode 100644
index 000000000..69a3700f2
--- /dev/null
+++ b/gtsam/nonlinear/tests/CMakeLists.txt
@@ -0,0 +1 @@
+gtsamAddTestsGlob(nonlinear "test*.cpp" "" "gtsam")
diff --git a/gtsam/slam/CMakeLists.txt b/gtsam/slam/CMakeLists.txt
index 1c20e3720..b348c4aa3 100644
--- a/gtsam/slam/CMakeLists.txt
+++ b/gtsam/slam/CMakeLists.txt
@@ -4,19 +4,10 @@ set (slam_excluded_headers #"")
 )
 
 file(GLOB slam_headers "*.h")
-list(REMOVE_ITEM slam_headers ${slam_excluded_headers})
 install(FILES ${slam_headers} DESTINATION include/gtsam/slam)
 
-# Files to exclude from compilation of tests and timing scripts
-set(slam_excluded_files
- "${CMAKE_CURRENT_SOURCE_DIR}/tests/testSerialization.cpp"      
-#    ""  # Add to this list, with full path, to exclude
-)
-
 # Build tests
-if (GTSAM_BUILD_TESTS)
-    gtsam_add_subdir_tests(slam "gtsam" "gtsam" "${slam_excluded_files}") 
-endif(GTSAM_BUILD_TESTS)
+add_subdirectory(tests)
 
 # Build timing scripts
 if (GTSAM_BUILD_TIMING)
diff --git a/gtsam/slam/tests/CMakeLists.txt b/gtsam/slam/tests/CMakeLists.txt
new file mode 100644
index 000000000..d59128c2f
--- /dev/null
+++ b/gtsam/slam/tests/CMakeLists.txt
@@ -0,0 +1,6 @@
+# Files to exclude
+set(slam_excluded_files
+    testSerialization.cpp
+)
+
+gtsamAddTestsGlob(slam "test*.cpp" "${slam_excluded_files}" "gtsam")
diff --git a/gtsam/symbolic/CMakeLists.txt b/gtsam/symbolic/CMakeLists.txt
index a683f0711..6535755f8 100644
--- a/gtsam/symbolic/CMakeLists.txt
+++ b/gtsam/symbolic/CMakeLists.txt
@@ -2,16 +2,8 @@
 file(GLOB symbolic_headers "*.h")
 install(FILES ${symbolic_headers} DESTINATION include/gtsam/symbolic)
 
-# Files to exclude from compilation of tests and timing scripts
-set(symbolic_excluded_files
-# "${CMAKE_CURRENT_SOURCE_DIR}/tests/testTypedDiscreteFactor.cpp" # Example of excluding a test     
-    ""  # Add to this list, with full path, to exclude
-)
-
 # Build tests
-if (GTSAM_BUILD_TESTS)
-    gtsam_add_subdir_tests(symbolic "gtsam" "gtsam" "${symbolic_excluded_files}") 
-endif(GTSAM_BUILD_TESTS)
+add_subdirectory(tests)
 
 # Build timing scripts
 if (GTSAM_BUILD_TIMING)
diff --git a/gtsam/symbolic/tests/CMakeLists.txt b/gtsam/symbolic/tests/CMakeLists.txt
new file mode 100644
index 000000000..6bef7a109
--- /dev/null
+++ b/gtsam/symbolic/tests/CMakeLists.txt
@@ -0,0 +1 @@
+gtsamAddTestsGlob(symbolic "test*.cpp" "" "gtsam")
diff --git a/gtsam_unstable/base/CMakeLists.txt b/gtsam_unstable/base/CMakeLists.txt
index 22737d533..d83e97d26 100644
--- a/gtsam_unstable/base/CMakeLists.txt
+++ b/gtsam_unstable/base/CMakeLists.txt
@@ -2,16 +2,8 @@
 file(GLOB base_headers "*.h")
 install(FILES ${base_headers} DESTINATION include/gtsam_unstable/base)
 
-set (base_full_libs
-    gtsam
-    gtsam_unstable)
-
-# Exclude tests that don't work
-set (base_excluded_tests "")
-
 # Add all tests
-gtsam_add_subdir_tests(base_unstable "${base_full_libs}" "${base_full_libs}" "${base_excluded_tests}") 
-add_dependencies(check.unstable check.base_unstable)
+add_subdirectory(tests)
 
 # Build timing scripts
 if (GTSAM_BUILD_TIMING)
diff --git a/gtsam_unstable/base/tests/CMakeLists.txt b/gtsam_unstable/base/tests/CMakeLists.txt
new file mode 100644
index 000000000..d103ec578
--- /dev/null
+++ b/gtsam_unstable/base/tests/CMakeLists.txt
@@ -0,0 +1 @@
+gtsamAddTestsGlob(base_unstable "test*.cpp" "" "gtsam_unstable")
diff --git a/gtsam_unstable/discrete/CMakeLists.txt b/gtsam_unstable/discrete/CMakeLists.txt
index f6cfcb1a8..1072999b2 100644
--- a/gtsam_unstable/discrete/CMakeLists.txt
+++ b/gtsam_unstable/discrete/CMakeLists.txt
@@ -2,19 +2,8 @@
 file(GLOB discrete_headers "*.h")
 install(FILES ${discrete_headers} DESTINATION include/gtsam_unstable/discrete)
 
-set (discrete_full_libs
-    gtsam
-    gtsam_unstable)
-
-# Exclude tests that don't work
-#set (discrete_excluded_tests 
-#"${CMAKE_CURRENT_SOURCE_DIR}/tests/testScheduler.cpp"
-#)
-
-
 # Add all tests
-gtsam_add_subdir_tests(discrete_unstable "${discrete_full_libs}" "${discrete_full_libs}" "${discrete_excluded_tests}") 
-add_dependencies(check.unstable check.discrete_unstable)
+add_subdirectory(tests)
 
 # List examples to build - comment out here to exclude from compilation
 set(discrete_unstable_examples
diff --git a/gtsam_unstable/discrete/tests/CMakeLists.txt b/gtsam_unstable/discrete/tests/CMakeLists.txt
new file mode 100644
index 000000000..2687a760c
--- /dev/null
+++ b/gtsam_unstable/discrete/tests/CMakeLists.txt
@@ -0,0 +1 @@
+gtsamAddTestsGlob(discrete_unstable "test*.cpp" "" "gtsam_unstable")
diff --git a/gtsam_unstable/dynamics/CMakeLists.txt b/gtsam_unstable/dynamics/CMakeLists.txt
index 382b2b4cf..66aef9455 100644
--- a/gtsam_unstable/dynamics/CMakeLists.txt
+++ b/gtsam_unstable/dynamics/CMakeLists.txt
@@ -2,14 +2,5 @@
 file(GLOB dynamics_headers "*.h")
 install(FILES ${dynamics_headers} DESTINATION include/gtsam_unstable/dynamics)
 
-# Components to link tests in this subfolder against
-set (dynamics_full_libs
-    gtsam
-    gtsam_unstable)
-
-# Exclude tests that don't work
-set (dynamics_excluded_tests "")
-
 # Add all tests
-gtsam_add_subdir_tests(dynamics_unstable "${dynamics_full_libs}" "${dynamics_full_libs}" "${dynamics_excluded_tests}") 
-add_dependencies(check.unstable check.dynamics_unstable)
+add_subdirectory(tests)
diff --git a/gtsam_unstable/dynamics/tests/CMakeLists.txt b/gtsam_unstable/dynamics/tests/CMakeLists.txt
new file mode 100644
index 000000000..493cef4fa
--- /dev/null
+++ b/gtsam_unstable/dynamics/tests/CMakeLists.txt
@@ -0,0 +1 @@
+gtsamAddTestsGlob(dynamics_unstable "test*.cpp" "" "gtsam_unstable")
diff --git a/gtsam_unstable/geometry/CMakeLists.txt b/gtsam_unstable/geometry/CMakeLists.txt
index c7ee3902f..c8b7e250f 100644
--- a/gtsam_unstable/geometry/CMakeLists.txt
+++ b/gtsam_unstable/geometry/CMakeLists.txt
@@ -2,14 +2,5 @@
 file(GLOB geometry_headers "*.h")
 install(FILES ${geometry_headers} DESTINATION include/gtsam_unstable/geometry)
 
-# Components to link tests in this subfolder against
-set (geometry_full_libs
-    gtsam
-    gtsam_unstable)
-
-# Exclude tests that don't work
-set (geometry_excluded_tests "")
-
 # Add all tests
-gtsam_add_subdir_tests(geometry_unstable "${geometry_full_libs}" "${geometry_full_libs}" "${geometry_excluded_tests}") 
-add_dependencies(check.unstable check.geometry_unstable)
+add_subdirectory(tests)
diff --git a/gtsam_unstable/geometry/tests/CMakeLists.txt b/gtsam_unstable/geometry/tests/CMakeLists.txt
new file mode 100644
index 000000000..e22da8652
--- /dev/null
+++ b/gtsam_unstable/geometry/tests/CMakeLists.txt
@@ -0,0 +1 @@
+gtsamAddTestsGlob(geometry_unstable "test*.cpp" "" "gtsam_unstable")
diff --git a/gtsam_unstable/nonlinear/CMakeLists.txt b/gtsam_unstable/nonlinear/CMakeLists.txt
index daef8983c..9e0cb68e1 100644
--- a/gtsam_unstable/nonlinear/CMakeLists.txt
+++ b/gtsam_unstable/nonlinear/CMakeLists.txt
@@ -2,18 +2,5 @@
 file(GLOB nonlinear_headers "*.h")
 install(FILES ${nonlinear_headers} DESTINATION include/gtsam_unstable/nonlinear)
 
-# Components to link tests in this subfolder against
-set (nonlinear_full_libs
-gtsam
-gtsam_unstable)
-
-# Exclude tests that don't work
-set (nonlinear_excluded_tests #"")
-#"${CMAKE_CURRENT_SOURCE_DIR}/tests/testConcurrentIncrementalFilter.cpp" 
-#"${CMAKE_CURRENT_SOURCE_DIR}/tests/testIncrementalFixedLagSmoother.cpp"
-)
-
-
 # Add all tests
-gtsam_add_subdir_tests(nonlinear_unstable "${nonlinear_full_libs}" "${nonlinear_full_libs}" "${nonlinear_excluded_tests}")
-add_dependencies(check.unstable check.nonlinear_unstable)
+add_subdirectory(tests)
diff --git a/gtsam_unstable/nonlinear/tests/CMakeLists.txt b/gtsam_unstable/nonlinear/tests/CMakeLists.txt
new file mode 100644
index 000000000..5b1fd07d4
--- /dev/null
+++ b/gtsam_unstable/nonlinear/tests/CMakeLists.txt
@@ -0,0 +1 @@
+gtsamAddTestsGlob(nonlinear_unstable "test*.cpp" "" "gtsam_unstable")
diff --git a/gtsam_unstable/slam/CMakeLists.txt b/gtsam_unstable/slam/CMakeLists.txt
index ae4784f70..a86beac63 100644
--- a/gtsam_unstable/slam/CMakeLists.txt
+++ b/gtsam_unstable/slam/CMakeLists.txt
@@ -7,16 +7,5 @@ file(GLOB slam_headers "*.h")
 list(REMOVE_ITEM slam_headers ${slam_excluded_headers})
 install(FILES ${slam_headers} DESTINATION include/gtsam_unstable/slam)
 
-# Components to link tests in this subfolder against
-set (slam_full_libs
-    gtsam
-    gtsam_unstable)
-
-# Exclude tests that don't work
-set (slam_excluded_tests
-    "${CMAKE_CURRENT_SOURCE_DIR}/tests/testSerialization.cpp"
-#    ""  # Add to this list, with full path, to exclude
-)
 # Add all tests
-gtsam_add_subdir_tests(slam_unstable "${slam_full_libs}" "${slam_full_libs}" "${slam_excluded_tests}") 
-add_dependencies(check.unstable check.slam_unstable)
+add_subdirectory(tests)
diff --git a/gtsam_unstable/slam/tests/CMakeLists.txt b/gtsam_unstable/slam/tests/CMakeLists.txt
new file mode 100644
index 000000000..bb5259ef2
--- /dev/null
+++ b/gtsam_unstable/slam/tests/CMakeLists.txt
@@ -0,0 +1,7 @@
+
+# Exclude tests that don't work
+set (slam_excluded_tests
+    testSerialization.cpp
+)
+
+gtsamAddTestsGlob(slam_unstable "test*.cpp" "${slam_excluded_tests}" "gtsam_unstable")
diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
index a6ae2afca..cf81dc762 100644
--- a/tests/CMakeLists.txt
+++ b/tests/CMakeLists.txt
@@ -1,31 +1,19 @@
-# Assemble local libraries
-set (tests_full_libs
-    gtsam
-    CppUnitLite)
-
 # exclude certain files
 # note the source dir on each 
-set (tests_exclude
-    #"${CMAKE_CURRENT_SOURCE_DIR}/testOccupancyGrid.cpp"
-)
+set (tests_exclude "")
 
 if ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "Clang") # might not be best test - Richard & Jason & Frank
 	# clang linker segfaults on large testSerializationSLAM 
-	list (APPEND tests_exclude "${CMAKE_CURRENT_SOURCE_DIR}/testSerializationSLAM.cpp") 
+	list (APPEND tests_exclude "testSerializationSLAM.cpp") 
 endif()
 
 # Build tests
-if (GTSAM_BUILD_TESTS)
-    # Subdirectory target for tests
-    add_custom_target(check.tests COMMAND ${CMAKE_CTEST_COMMAND})
-    set(is_test TRUE)
+gtsamAddTestsGlob(tests "test*.cpp" "${tests_exclude}" "gtsam")
 
-    # Build grouped tests
-    gtsam_add_grouped_scripts("tests"               # Use subdirectory as group label
-    "test*.cpp;*.h" check "Test"                      # Standard for all tests
-    "${tests_full_libs}" "${tests_full_libs}" "${tests_exclude}"  # Pass in linking and exclusion lists
-    ${is_test})                                         # Set all as tests
-endif (GTSAM_BUILD_TESTS)
+if(MSVC)
+	set_property(SOURCE "${CMAKE_CURRENT_SOURCE_DIR}/testSerializationSLAM.cpp"
+		APPEND PROPERTY COMPILE_FLAGS "/bigobj")
+endif()
 
 # Build timing scripts
 if (GTSAM_BUILD_TIMING)
@@ -39,8 +27,3 @@ if (GTSAM_BUILD_TIMING)
     "${tests_full_libs}" "${tests_full_libs}" "${tests_exclude}"   # Pass in linking and exclusion lists
     ${is_test})
 endif (GTSAM_BUILD_TIMING)
-
-if(MSVC)
-	set_property(SOURCE "${CMAKE_CURRENT_SOURCE_DIR}/testSerializationSLAM.cpp"
-		APPEND PROPERTY COMPILE_FLAGS "/bigobj")
-endif()
diff --git a/wrap/CMakeLists.txt b/wrap/CMakeLists.txt
index 28f8bea48..03915a662 100644
--- a/wrap/CMakeLists.txt
+++ b/wrap/CMakeLists.txt
@@ -31,9 +31,5 @@ set(GTSAM_EXPORTED_TARGETS "${GTSAM_EXPORTED_TARGETS}" PARENT_SCOPE)
 install(FILES matlab.h DESTINATION include/wrap)
 
 # Build tests
-if (GTSAM_BUILD_TESTS)
-    set(wrap_local_libs wrap_lib ${WRAP_BOOST_LIBRARIES}) 
-    gtsam_add_subdir_tests("wrap" "${wrap_local_libs}" "${wrap_local_libs}" "") 
-endif(GTSAM_BUILD_TESTS)
-
+add_subdirectory(tests)
 
diff --git a/wrap/tests/CMakeLists.txt b/wrap/tests/CMakeLists.txt
new file mode 100644
index 000000000..f3ac1df49
--- /dev/null
+++ b/wrap/tests/CMakeLists.txt
@@ -0,0 +1 @@
+gtsamAddTestsGlob(wrap "test*.cpp" "" "wrap_lib")

From 3e3f31c4d1823c1ec9a526a8a2f99174ac01efb1 Mon Sep 17 00:00:00 2001
From: Richard Roberts <richard.jw.roberts@gmail.com>
Date: Thu, 13 Feb 2014 10:56:20 -0500
Subject: [PATCH 02/10] Made FIRST_PASS_DONE variable internal

---
 cmake/GtsamBuildTypes.cmake | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cmake/GtsamBuildTypes.cmake b/cmake/GtsamBuildTypes.cmake
index f00f3b3b1..51e66627d 100644
--- a/cmake/GtsamBuildTypes.cmake
+++ b/cmake/GtsamBuildTypes.cmake
@@ -102,7 +102,7 @@ if(    NOT cmake_build_type_tolower STREQUAL ""
 endif()
 
 # Mark that first pass is done
-set(FIRST_PASS_DONE TRUE CACHE BOOL "Internally used to mark whether cmake has been run multiple times" FORCE)
+set(FIRST_PASS_DONE TRUE CACHE INTERNAL "Internally used to mark whether cmake has been run multiple times")
 mark_as_advanced(FIRST_PASS_DONE) 
 
 # Enable Visual Studio solution folders

From e7018e8c51e88aa2258e8e8e3c25802b02f87f6c Mon Sep 17 00:00:00 2001
From: Richard Roberts <richard.jw.roberts@gmail.com>
Date: Thu, 13 Feb 2014 10:57:09 -0500
Subject: [PATCH 03/10] Fixed bugs in last commit and made
 GTSAM_SINGLE_TEST_EXE default on Xcode too

---
 cmake/GtsamTesting.cmake | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/cmake/GtsamTesting.cmake b/cmake/GtsamTesting.cmake
index dce09449e..7e84c1a4d 100644
--- a/cmake/GtsamTesting.cmake
+++ b/cmake/GtsamTesting.cmake
@@ -12,11 +12,12 @@ endif()
 add_custom_target(timing)
 
 # Add option for combining unit tests
-if(MSVC)
+if(MSVC OR XCODE_VERSION)
 	option(GTSAM_SINGLE_TEST_EXE "Combine unit tests into single executable (faster compile)" ON)
 else()
 	option(GTSAM_SINGLE_TEST_EXE "Combine unit tests into single executable (faster compile)" OFF)
 endif()
+mark_as_advanced(GTSAM_SINGLE_TEST_EXE)
 
 # Macro for adding glob(s) of tests relative to the current directory.  Automatically
 # links the tests with CppUnitLite.  Separate multiple globPatterns, linkLibraries,
@@ -33,13 +34,13 @@ macro(gtsamAddTestsGlob groupName globPatterns excludedFiles linkLibraries)
 	    set(script_files_relative "")
 	    foreach(one_pattern IN ITEMS ${globPatterns})
 			message(STATUS "Filling test group ${groupName}")
-	        file(GLOB RELATIVE ${CMAKE_CURRENT_SOURCE_DIR} one_script_files "${one_pattern}")
-	        list(APPEND script_files_relative "${one_script_files}")
+	        file(GLOB one_script_files RELATIVE ${CMAKE_CURRENT_SOURCE_DIR} "${one_pattern}")
+	        list(APPEND script_files_relative ${one_script_files})		
 	    endforeach()
 
 	    # Remove excluded scripts from the list
-	    if(excludedFiles)
-	    	list(REMOVE_ITEM script_files_relative excludedFiles)
+	    if(NOT "${excludedFiles}" STREQUAL "")
+	    	list(REMOVE_ITEM script_files_relative ${excludedFiles})
 	    endif()
 	
 		# Get absolute paths
@@ -65,14 +66,15 @@ macro(gtsamAddTestsGlob groupName globPatterns excludedFiles linkLibraries)
 		source_group("" FILES ${script_srcs} ${script_headers})
 
 		if(NOT GTSAM_SINGLE_TEST_EXE)
+			# Default for Makefiles - each test in its own executable
 			foreach(script_src IN ITEMS ${script_srcs})
 				# Get test base name
 				get_filename_component(script_name ${script_src} NAME_WE)
 			
 				# Add executable
-				add_executable(script_name ${script_src} ${script_headers})
+				add_executable(${script_name} ${script_src} ${script_headers})
 				target_link_libraries(${script_name} CppUnitLite ${linkLibraries})
-			
+				
 				# Add target dependencies
 				add_test(NAME ${script_name} COMMAND ${script_name})
 				add_dependencies(check.${groupName} ${script_name})
@@ -91,7 +93,7 @@ macro(gtsamAddTestsGlob groupName globPatterns excludedFiles linkLibraries)
 				set_property(TARGET ${script_name} PROPERTY FOLDER "Unit tests/${groupName}")
 			endforeach()
 		else()
-			# Create single unit test exe from all test scripts
+			# Default on MSVC and XCode - combine test group into a single exectuable
 			set(target_name check_${groupName}_program)
 		
 			# Add executable

From b3aa563e1341c0812fdd232d70fde4317cd86800 Mon Sep 17 00:00:00 2001
From: Richard Roberts <richard.jw.roberts@gmail.com>
Date: Thu, 13 Feb 2014 11:14:25 -0500
Subject: [PATCH 04/10] Removed non-existant file to exclude

---
 gtsam/slam/tests/CMakeLists.txt | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/gtsam/slam/tests/CMakeLists.txt b/gtsam/slam/tests/CMakeLists.txt
index d59128c2f..1205388a1 100644
--- a/gtsam/slam/tests/CMakeLists.txt
+++ b/gtsam/slam/tests/CMakeLists.txt
@@ -1,6 +1 @@
-# Files to exclude
-set(slam_excluded_files
-    testSerialization.cpp
-)
-
-gtsamAddTestsGlob(slam "test*.cpp" "${slam_excluded_files}" "gtsam")
+gtsamAddTestsGlob(slam "test*.cpp" "" "gtsam")

From 0e4b5269c53d5c84991f44a2dac990eaebe0bbeb Mon Sep 17 00:00:00 2001
From: Richard Roberts <richard.jw.roberts@gmail.com>
Date: Thu, 13 Feb 2014 11:14:51 -0500
Subject: [PATCH 05/10] Cleaned up gtsamAddTestsGlob function

---
 cmake/GtsamTesting.cmake | 30 +++++++++++++-----------------
 1 file changed, 13 insertions(+), 17 deletions(-)

diff --git a/cmake/GtsamTesting.cmake b/cmake/GtsamTesting.cmake
index 7e84c1a4d..fbde55a28 100644
--- a/cmake/GtsamTesting.cmake
+++ b/cmake/GtsamTesting.cmake
@@ -19,9 +19,10 @@ else()
 endif()
 mark_as_advanced(GTSAM_SINGLE_TEST_EXE)
 
-# Macro for adding glob(s) of tests relative to the current directory.  Automatically
-# links the tests with CppUnitLite.  Separate multiple globPatterns, linkLibraries,
-# and excludedFiles using a semicolon, e.g. "testThings*.cpp;testOthers*.cpp".
+# Macro for adding a group of tests relative to the current directory.
+# globPatterns: e.g. "test*.cpp", or a list of globs, e.g. "testA*.cpp;testB*.cpp".
+# excludedFiles: list of files or globs to exclude, e.g. "testC*.cpp;testBroken.cpp".
+# linkLibraries: list of libraries to link to in addition to CppUnitLite.
 # Usage example:  gtsamAddTestsGlob(basic "test*.cpp" "testBroken.cpp" "gtsam;GeographicLib")
 macro(gtsamAddTestsGlob groupName globPatterns excludedFiles linkLibraries)
 	if(GTSAM_BUILD_TESTS)
@@ -30,25 +31,20 @@ macro(gtsamAddTestsGlob groupName globPatterns excludedFiles linkLibraries)
 			add_custom_target(check.${groupName} COMMAND ${CMAKE_CTEST_COMMAND} -C $<CONFIGURATION> --output-on-failure)
 		endif()
 	
-	    # Get all script files relative to the currect directory
-	    set(script_files_relative "")
-	    foreach(one_pattern IN ITEMS ${globPatterns})
-			message(STATUS "Filling test group ${groupName}")
-	        file(GLOB one_script_files RELATIVE ${CMAKE_CURRENT_SOURCE_DIR} "${one_pattern}")
-	        list(APPEND script_files_relative ${one_script_files})		
-	    endforeach()
+	    # Get all script files
+		message(STATUS "Filling test group ${groupName}")
+        file(GLOB script_files ${globPatterns})
 
 	    # Remove excluded scripts from the list
 	    if(NOT "${excludedFiles}" STREQUAL "")
-	    	list(REMOVE_ITEM script_files_relative ${excludedFiles})
+			file(GLOB excludedFilePaths ${excludedFiles})
+			if("${excludedFilePaths}" STREQUAL "")
+				message(WARNING "The pattern '${excludedFiles}' for excluding tests from group ${groupName} did not match any files")
+			else()
+		    	list(REMOVE_ITEM script_files ${excludedFilePaths})
+			endif()
 	    endif()
 	
-		# Get absolute paths
-		set(script_files "")
-		foreach(script_file IN ITEMS ${script_files_relative})
-			list(APPEND script_files "${CMAKE_CURRENT_SOURCE_DIR}/${script_file}")
-		endforeach()
-
 		# Separate into source files and headers (allows for adding headers to show up in
 		# MSVC and Xcode projects).
 		set(script_srcs "")

From 218500de11b858da3f6bd5b9d937f07eee455b0c Mon Sep 17 00:00:00 2001
From: Richard Roberts <richard.jw.roberts@gmail.com>
Date: Thu, 13 Feb 2014 13:22:56 -0500
Subject: [PATCH 06/10] Cleaned up flags for building examples and tests

---
 CMakeLists.txt           | 21 ++++-----------------
 cmake/GtsamTesting.cmake | 16 +++++++++-------
 2 files changed, 13 insertions(+), 24 deletions(-)

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 22e58564b..da60d1432 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -47,7 +47,6 @@ endif()
 
 # Configurable Options
 option(GTSAM_BUILD_TIMING                "Enable/Disable building of timing scripts" OFF) # These do not currently work
-option(GTSAM_BUILD_EXAMPLES              "Enable/Disable building of examples"       ON)
 if(GTSAM_UNSTABLE_AVAILABLE)
     option(GTSAM_BUILD_UNSTABLE              "Enable/Disable libgtsam_unstable"          ON)
 endif()
@@ -82,17 +81,6 @@ endif()
 set(CPACK_SOURCE_GENERATOR "TGZ" CACHE STRING "CPack Default Source Generator")
 set(CPACK_GENERATOR        "TGZ" CACHE STRING "CPack Default Binary Generator")
 
-# Flags to determine whether examples are build during 'make install'
-# Note that these remove the targets from the 'all'
-option(GTSAM_DISABLE_EXAMPLES_ON_INSTALL "Disables building examples during install" OFF)
-
-# Pull in infrastructure
-if (GTSAM_BUILD_TESTS)
-    enable_testing()
-    include(Dart)
-    include(CTest)
-endif()
-
 ###############################################################################
 # Find boost
 
@@ -362,18 +350,17 @@ set(CPACK_DEBIAN_PACKAGE_DEPENDS "libboost-dev (>= 1.43)") #Example: "libc6 (>=
 message(STATUS "===============================================================")
 message(STATUS "================  Configuration Options  ======================")
 message(STATUS "Build flags                                               ")
-print_config_flag(${GTSAM_BUILD_TIMING}                "Build Timing scripts           ")
-print_config_flag(${GTSAM_BUILD_EXAMPLES}              "Build Examples                 ")
 print_config_flag(${GTSAM_BUILD_TESTS}                 "Build Tests                    ")
+print_config_flag(${GTSAM_BUILD_EXAMPLES_ALWAYS}       "Build examples with 'make all' ")
+print_config_flag(${GTSAM_BUILD_TIMING}                "Build Timing scripts           ")
 if (DOXYGEN_FOUND)
-    print_config_flag(${GTSAM_BUILD_DOCS}                  "Build Docs                     ")
+    print_config_flag(${GTSAM_BUILD_DOCS}              "Build Docs                     ")
 endif()
 print_config_flag(${GTSAM_BUILD_STATIC_LIBRARY}        "Build static GTSAM library instead of shared")
 print_config_flag(${GTSAM_BUILD_TYPE_POSTFIXES}        "Put build type in library name ")
 if(GTSAM_UNSTABLE_AVAILABLE)
-    print_config_flag(${GTSAM_BUILD_UNSTABLE}           "Build libgtsam_unstable        ")
+    print_config_flag(${GTSAM_BUILD_UNSTABLE}          "Build libgtsam_unstable        ")
 endif()
-print_config_flag(${GTSAM_DISABLE_EXAMPLES_ON_INSTALL}  "No examples in all or install  ")
 string(TOUPPER "${CMAKE_BUILD_TYPE}" cmake_build_type_toupper)
 if(NOT MSVC AND NOT XCODE_VERSION)
     message(STATUS "  Build type                     : ${CMAKE_BUILD_TYPE}")
diff --git a/cmake/GtsamTesting.cmake b/cmake/GtsamTesting.cmake
index fbde55a28..6170fd154 100644
--- a/cmake/GtsamTesting.cmake
+++ b/cmake/GtsamTesting.cmake
@@ -3,13 +3,7 @@
 enable_testing()
 
 option(GTSAM_BUILD_TESTS                 "Enable/Disable building of tests"          ON)
-
-# Enable make check (http://www.cmake.org/Wiki/CMakeEmulateMakeCheck)
-if(GTSAM_BUILD_TESTS)
-    add_custom_target(check COMMAND ${CMAKE_CTEST_COMMAND} -C $<CONFIGURATION> --output-on-failure)
-endif()
-
-add_custom_target(timing)
+option(GTSAM_BUILD_EXAMPLES_ALWAYS       "Build examples with 'make all' (build with 'make examples' if not)"       ON)
 
 # Add option for combining unit tests
 if(MSVC OR XCODE_VERSION)
@@ -19,6 +13,14 @@ else()
 endif()
 mark_as_advanced(GTSAM_SINGLE_TEST_EXE)
 
+# Enable make check (http://www.cmake.org/Wiki/CMakeEmulateMakeCheck)
+if(GTSAM_BUILD_TESTS)
+    add_custom_target(check COMMAND ${CMAKE_CTEST_COMMAND} -C $<CONFIGURATION> --output-on-failure)
+endif()
+
+add_custom_target(timing)
+add_custom_target(examples)
+
 # Macro for adding a group of tests relative to the current directory.
 # globPatterns: e.g. "test*.cpp", or a list of globs, e.g. "testA*.cpp;testB*.cpp".
 # excludedFiles: list of files or globs to exclude, e.g. "testC*.cpp;testBroken.cpp".

From 320cfe7b4da281fe42eeef759a9b603e94beb5ba Mon Sep 17 00:00:00 2001
From: Richard Roberts <richard.jw.roberts@gmail.com>
Date: Thu, 13 Feb 2014 13:23:20 -0500
Subject: [PATCH 07/10] Comment cleanups

---
 cmake/GtsamTesting.cmake | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

diff --git a/cmake/GtsamTesting.cmake b/cmake/GtsamTesting.cmake
index 6170fd154..d51445b90 100644
--- a/cmake/GtsamTesting.cmake
+++ b/cmake/GtsamTesting.cmake
@@ -22,7 +22,9 @@ add_custom_target(timing)
 add_custom_target(examples)
 
 # Macro for adding a group of tests relative to the current directory.
-# globPatterns: e.g. "test*.cpp", or a list of globs, e.g. "testA*.cpp;testB*.cpp".
+# groupName: A name that will allow this group of tests to be run independently, e.g.
+#            'basic' causes a 'check.basic' target to be created to run this test group.
+# globPatterns: e.g. "test*.cpp", or a list of globs and files, e.g. "testA*.cpp;testB*.cpp".
 # excludedFiles: list of files or globs to exclude, e.g. "testC*.cpp;testBroken.cpp".
 # linkLibraries: list of libraries to link to in addition to CppUnitLite.
 # Usage example:  gtsamAddTestsGlob(basic "test*.cpp" "testBroken.cpp" "gtsam;GeographicLib")
@@ -111,7 +113,7 @@ macro(gtsamAddTestsGlob groupName globPatterns excludedFiles linkLibraries)
 			# Add TOPSRCDIR
 			set_property(SOURCE ${script_srcs} APPEND PROPERTY COMPILE_DEFINITIONS "TOPSRCDIR=\"${PROJECT_SOURCE_DIR}\"")
 		
-			# Excluse from 'make all' and 'make install'
+			# Exclude from 'make all' and 'make install'
 			set_target_properties(${target_name} PROPERTIES EXCLUDE_FROM_ALL ON)
 
 			# Configure target folder (for MSVC and Xcode)

From 2ccb8921673a2868fa2a3f173d1335fbf519b097 Mon Sep 17 00:00:00 2001
From: Richard Roberts <richard.jw.roberts@gmail.com>
Date: Thu, 13 Feb 2014 13:23:35 -0500
Subject: [PATCH 08/10] Removed unneeded print

---
 cmake/GtsamTesting.cmake | 1 -
 1 file changed, 1 deletion(-)

diff --git a/cmake/GtsamTesting.cmake b/cmake/GtsamTesting.cmake
index d51445b90..aee3ac88b 100644
--- a/cmake/GtsamTesting.cmake
+++ b/cmake/GtsamTesting.cmake
@@ -36,7 +36,6 @@ macro(gtsamAddTestsGlob groupName globPatterns excludedFiles linkLibraries)
 		endif()
 	
 	    # Get all script files
-		message(STATUS "Filling test group ${groupName}")
         file(GLOB script_files ${globPatterns})
 
 	    # Remove excluded scripts from the list

From f283150eb4209d87c8040ebd5184db52a80a3a23 Mon Sep 17 00:00:00 2001
From: Richard Roberts <richard.jw.roberts@gmail.com>
Date: Thu, 13 Feb 2014 13:24:14 -0500
Subject: [PATCH 09/10] Using macro for creating examples to remove cmake code
 copy/paste

---
 cmake/GtsamTesting.cmake                      | 63 +++++++++++++++++++
 examples/CMakeLists.txt                       | 46 ++------------
 gtsam_unstable/discrete/CMakeLists.txt        | 26 +-------
 .../discrete/examples/CMakeLists.txt          |  5 ++
 gtsam_unstable/examples/CMakeLists.txt        | 35 ++---------
 5 files changed, 80 insertions(+), 95 deletions(-)
 create mode 100644 gtsam_unstable/discrete/examples/CMakeLists.txt

diff --git a/cmake/GtsamTesting.cmake b/cmake/GtsamTesting.cmake
index aee3ac88b..b3209e104 100644
--- a/cmake/GtsamTesting.cmake
+++ b/cmake/GtsamTesting.cmake
@@ -121,6 +121,69 @@ macro(gtsamAddTestsGlob groupName globPatterns excludedFiles linkLibraries)
 	endif()
 endmacro()
 
+# Macro for adding scripts - one executable per cpp file.
+# globPatterns: e.g. "*.cpp", or a list of globs and files, e.g. "A*.cpp;B*.cpp".
+# excludedFiles: list of files or globs to exclude, e.g. "testC*.cpp;testBroken.cpp".
+# linkLibraries: list of libraries to link to.
+# Usage example:  gtsamAddScriptsGlob("*.cpp" "Broken.cpp" "gtsam;GeographicLib")
+macro(gtsamAddExamplesGlob globPatterns excludedFiles linkLibraries)
+    # Get all script files
+    file(GLOB script_files ${globPatterns})
+
+    # Remove excluded scripts from the list
+    if(NOT "${excludedFiles}" STREQUAL "")
+		file(GLOB excludedFilePaths ${excludedFiles})
+		if("${excludedFilePaths}" STREQUAL "")
+			message(WARNING "The script exclusion pattern '${excludedFiles}' did not match any files")
+		else()
+	    	list(REMOVE_ITEM script_files ${excludedFilePaths})
+		endif()
+    endif()
+
+	# Separate into source files and headers (allows for adding headers to show up in
+	# MSVC and Xcode projects).
+	set(script_srcs "")
+	set(script_headers "")
+	foreach(script_file IN ITEMS ${script_files})
+		get_filename_component(script_ext ${script_file} EXT)
+		if(script_ext MATCHES "(h|H)")
+			list(APPEND script_headers ${script_file})
+		else()
+			list(APPEND script_srcs ${script_file})
+		endif()
+	endforeach()
+
+	# Don't put test files in folders in MSVC and Xcode because they're already grouped
+	source_group("" FILES ${script_srcs} ${script_headers})
+
+	# Create executables
+	foreach(script_src IN ITEMS ${script_srcs})
+		# Get script base name
+		get_filename_component(script_name ${script_src} NAME_WE)
+
+		# Add executable
+		add_executable(${script_name} ${script_src} ${script_headers})
+		target_link_libraries(${script_name} ${linkLibraries})
+	
+		# Add target dependencies
+		add_dependencies(examples ${script_name})
+		if(NOT MSVC AND NOT XCODE_VERSION)
+		  add_custom_target(${script_name}.run ${EXECUTABLE_OUTPUT_PATH}${script_name})
+		endif()
+
+		# Add TOPSRCDIR
+		set_property(SOURCE ${script_src} APPEND PROPERTY COMPILE_DEFINITIONS "TOPSRCDIR=\"${PROJECT_SOURCE_DIR}\"")
+	
+		if(NOT GTSAM_BUILD_EXAMPLES_ALWAYS)
+			# Exclude from 'make all' and 'make install'
+			set_target_properties(${target_name} PROPERTIES EXCLUDE_FROM_ALL ON)
+		endif()
+
+		# Configure target folder (for MSVC and Xcode)
+		set_property(TARGET ${script_name} PROPERTY FOLDER "Examples")
+	endforeach()
+endmacro()
+
 # Macro for adding categorized tests in a "tests" folder, with 
 # optional exclusion of tests and convenience library linking options
 #  
diff --git a/examples/CMakeLists.txt b/examples/CMakeLists.txt
index d87070330..7251c2b6f 100644
--- a/examples/CMakeLists.txt
+++ b/examples/CMakeLists.txt
@@ -1,42 +1,8 @@
-if(NOT MSVC)
-  add_custom_target(examples)
-endif()
-
-# Build example executables
-FILE(GLOB example_srcs "*.cpp")
-
-set (excluded_examples #"")
-    "${CMAKE_CURRENT_SOURCE_DIR}/DiscreteBayesNet_FG.cpp"
-    "${CMAKE_CURRENT_SOURCE_DIR}/UGM_chain.cpp"
-    "${CMAKE_CURRENT_SOURCE_DIR}/UGM_small.cpp"
-    "${CMAKE_CURRENT_SOURCE_DIR}/elaboratePoint2KalmanFilter.cpp"
+set (excluded_examples
+    DiscreteBayesNet_FG.cpp
+    UGM_chain.cpp
+    UGM_small.cpp
+    elaboratePoint2KalmanFilter.cpp
 )
 
-list(REMOVE_ITEM example_srcs ${excluded_examples})
-
-foreach(example_src ${example_srcs} )
-    get_filename_component(example_base ${example_src} NAME_WE)
-    set( example_bin ${example_base} )
-    message(STATUS "Adding Example ${example_bin}")
-	if(NOT MSVC)
-      add_dependencies(examples ${example_bin})
-	endif()
-    add_executable(${example_bin} ${example_src})
-    
-    # Disable building during make all/install
-    if (GTSAM_DISABLE_EXAMPLES_ON_INSTALL)
-        set_target_properties(${example_bin} PROPERTIES EXCLUDE_FROM_ALL ON)
-    endif()
-    
-    target_link_libraries(${example_bin} gtsam ${Boost_PROGRAM_OPTIONS_LIBRARY})
-	if(NOT MSVC AND NOT XCODE_VERSION)
-      add_custom_target(${example_bin}.run ${EXECUTABLE_OUTPUT_PATH}${example_bin} ${ARGN})
-	endif()
-	
-	# Set up Visual Studio folder
-	if(MSVC)
-	  set_property(TARGET ${example_bin} PROPERTY FOLDER "Examples")
-	endif()
-
-endforeach(example_src)
-
+gtsamAddExamplesGlob("*.cpp" "${excluded_examples}" "gtsam;${Boost_PROGRAM_OPTIONS_LIBRARY}")
diff --git a/gtsam_unstable/discrete/CMakeLists.txt b/gtsam_unstable/discrete/CMakeLists.txt
index 1072999b2..18346a45a 100644
--- a/gtsam_unstable/discrete/CMakeLists.txt
+++ b/gtsam_unstable/discrete/CMakeLists.txt
@@ -5,27 +5,5 @@ install(FILES ${discrete_headers} DESTINATION include/gtsam_unstable/discrete)
 # Add all tests
 add_subdirectory(tests)
 
-# List examples to build - comment out here to exclude from compilation
-set(discrete_unstable_examples
-schedulingExample
-schedulingQuals12
-schedulingQuals13
-)
-
-if (GTSAM_BUILD_EXAMPLES)
-    foreach(example ${discrete_unstable_examples})
-    	add_executable(${example} "examples/${example}.cpp")
-    	
-    	# Disable building during make all/install
-        if (GTSAM_ENABLE_INSTALL_EXAMPLE_FIX)
-            set_target_properties(${example} PROPERTIES EXCLUDE_FROM_ALL ON)
-        endif()
-    	
-		if(NOT MSVC AND NOT XCODE_VERSION)
-			add_dependencies(examples ${example})
-			add_custom_target(${example}.run ${EXECUTABLE_OUTPUT_PATH}${example} ${ARGN})
-		endif()
-    	
-    	target_link_libraries(${example} gtsam gtsam_unstable)
-    endforeach(example)
-endif (GTSAM_BUILD_EXAMPLES)
+# Add examples
+add_subdirectory(examples)
diff --git a/gtsam_unstable/discrete/examples/CMakeLists.txt b/gtsam_unstable/discrete/examples/CMakeLists.txt
new file mode 100644
index 000000000..da06b7dfc
--- /dev/null
+++ b/gtsam_unstable/discrete/examples/CMakeLists.txt
@@ -0,0 +1,5 @@
+set (excluded_examples
+    # fileToExclude.cpp
+)
+
+gtsamAddExamplesGlob("*.cpp" "${excluded_examples}" "gtsam_unstable")
diff --git a/gtsam_unstable/examples/CMakeLists.txt b/gtsam_unstable/examples/CMakeLists.txt
index d95ab7706..da06b7dfc 100644
--- a/gtsam_unstable/examples/CMakeLists.txt
+++ b/gtsam_unstable/examples/CMakeLists.txt
@@ -1,32 +1,5 @@
-if(NOT MSVC)
-  add_custom_target(unstable_examples)
-endif()
-
-# Build example executables
-FILE(GLOB example_srcs "*.cpp")
-foreach(example_src ${example_srcs} )
-    get_filename_component(example_base ${example_src} NAME_WE)
-    set( example_bin ${example_base} )
-    message(STATUS "Adding Example ${example_bin}")
-	if(NOT MSVC)
-      add_dependencies(examples ${example_bin})
-	endif()
-    add_executable(${example_bin} ${example_src})
-    
-    # Disable building during make all/install
-    if (GTSAM_DISABLE_EXAMPLES_ON_INSTALL)
-        set_target_properties(${example_bin} PROPERTIES EXCLUDE_FROM_ALL ON)
-    endif()
-    
-    target_link_libraries(${example_bin} gtsam gtsam_unstable)
-	if(NOT MSVC AND NOT XCODE_VERSION)
-      add_custom_target(${example_bin}.run ${EXECUTABLE_OUTPUT_PATH}${example_bin} ${ARGN})
-	endif()
-	
-	# Set up Visual Studio folder
-	if(MSVC)
-	  set_property(TARGET ${example_bin} PROPERTY FOLDER "Examples")
-	endif()
-
-endforeach(example_src)
+set (excluded_examples
+    # fileToExclude.cpp
+)
 
+gtsamAddExamplesGlob("*.cpp" "${excluded_examples}" "gtsam_unstable")

From 835d020581a8aaa22446efa118b422a7eb5f5d37 Mon Sep 17 00:00:00 2001
From: Richard Roberts <richard.jw.roberts@gmail.com>
Date: Thu, 13 Feb 2014 16:12:14 -0500
Subject: [PATCH 10/10] Added lots of documentation in GtsamTesting.cmake and
 moved old functions to an obsolete file with a warning if they are used

---
 cmake/GtsamTesting.cmake         | 278 ++++++++-----------------------
 cmake/GtsamTestingObsolete.cmake | 195 ++++++++++++++++++++++
 2 files changed, 265 insertions(+), 208 deletions(-)
 create mode 100644 cmake/GtsamTestingObsolete.cmake

diff --git a/cmake/GtsamTesting.cmake b/cmake/GtsamTesting.cmake
index b3209e104..bd8aba766 100644
--- a/cmake/GtsamTesting.cmake
+++ b/cmake/GtsamTesting.cmake
@@ -1,5 +1,64 @@
-# Build macros for using tests
+# This file defines the two macros below for easily adding groups of unit tests and scripts,
+# as well as sets up unit testing and defines several cache options used to control how
+# tests and scripts are built and run.
 
+
+
+###############################################################################
+# Macro:
+#
+# gtsamAddTestsGlob(groupName globPatterns excludedFiles linkLibraries)
+# 
+# Add a group of unit tests.  A list of unit test .cpp files or glob patterns specifies the
+# tests to create.  Tests are assigned into a group name so they can easily by run
+# independently with a make target.  Running 'make check' builds and runs all tests.
+#
+# Usage example:
+#   gtsamAddTestsGlob(basic "test*.cpp" "testBroken.cpp" "gtsam;GeographicLib")
+#
+# Arguments:
+#   groupName:     A name that will allow this group of tests to be run independently, e.g.
+#                  'basic' causes a 'check.basic' target to be created to run this test
+#                  group.
+#   globPatterns:  The list of files or glob patterns from which to create unit tests, with
+#                  one test created for each cpp file.  e.g. "test*.cpp", or
+#                  "testA*.cpp;testB*.cpp;testOneThing.cpp".
+#   excludedFiles: A list of files or globs to exclude, e.g. "testC*.cpp;testBroken.cpp".
+#                  Pass an empty string "" if nothing needs to be excluded.
+#   linkLibraries: The list of libraries to link to in addition to CppUnitLite.
+macro(gtsamAddTestsGlob groupName globPatterns excludedFiles linkLibraries)
+	gtsamAddTestsGlob_impl("${groupName}" "${globPatterns}" "${excludedFiles}" "${linkLibraries}")
+endmacro()
+
+
+###############################################################################
+# Macro:
+#
+# gtsamAddExamplesGlob(globPatterns excludedFiles linkLibraries)
+# 
+# Add scripts that will serve as examples of how to use the library.  A list of files or
+# glob patterns is specified, and one executable will be created for each matching .cpp
+# file.  These executables will not be installed.  They are build with 'make all' if
+# GTSAM_BUILD_EXAMPLES_ALWAYS is enabled.  They may also be built with 'make examples'.
+#
+# Usage example:
+#   gtsamAddExamplesGlob("*.cpp" "BrokenExample.cpp" "gtsam;GeographicLib")
+#
+# Arguments:
+#   globPatterns:  The list of files or glob patterns from which to create unit tests, with
+#                  one test created for each cpp file.  e.g. "*.cpp", or
+#                  "A*.cpp;B*.cpp;MyExample.cpp".
+#   excludedFiles: A list of files or globs to exclude, e.g. "C*.cpp;BrokenExample.cpp".  Pass
+#                  an empty string "" if nothing needs to be excluded.
+#   linkLibraries: The list of libraries to link to.
+macro(gtsamAddExamplesGlob globPatterns excludedFiles linkLibraries)
+	gtsamAddExamplesGlob_impl("${globPatterns}" "${excludedFiles}" "${linkLibraries}")
+endmacro()
+
+
+# Implementation follows:
+
+# Build macros for using tests
 enable_testing()
 
 option(GTSAM_BUILD_TESTS                 "Enable/Disable building of tests"          ON)
@@ -18,17 +77,16 @@ if(GTSAM_BUILD_TESTS)
     add_custom_target(check COMMAND ${CMAKE_CTEST_COMMAND} -C $<CONFIGURATION> --output-on-failure)
 endif()
 
-add_custom_target(timing)
+# Add examples target
 add_custom_target(examples)
 
-# Macro for adding a group of tests relative to the current directory.
-# groupName: A name that will allow this group of tests to be run independently, e.g.
-#            'basic' causes a 'check.basic' target to be created to run this test group.
-# globPatterns: e.g. "test*.cpp", or a list of globs and files, e.g. "testA*.cpp;testB*.cpp".
-# excludedFiles: list of files or globs to exclude, e.g. "testC*.cpp;testBroken.cpp".
-# linkLibraries: list of libraries to link to in addition to CppUnitLite.
-# Usage example:  gtsamAddTestsGlob(basic "test*.cpp" "testBroken.cpp" "gtsam;GeographicLib")
-macro(gtsamAddTestsGlob groupName globPatterns excludedFiles linkLibraries)
+# Include obsolete macros - will be removed in the near future
+include(GtsamTestingObsolete)
+
+
+# Implementations of this file's macros:
+
+macro(gtsamAddTestsGlob_impl groupName globPatterns excludedFiles linkLibraries)
 	if(GTSAM_BUILD_TESTS)
 		# Add group target if it doesn't already exist
 	    if(NOT TARGET check.${groupName})
@@ -121,12 +179,8 @@ macro(gtsamAddTestsGlob groupName globPatterns excludedFiles linkLibraries)
 	endif()
 endmacro()
 
-# Macro for adding scripts - one executable per cpp file.
-# globPatterns: e.g. "*.cpp", or a list of globs and files, e.g. "A*.cpp;B*.cpp".
-# excludedFiles: list of files or globs to exclude, e.g. "testC*.cpp;testBroken.cpp".
-# linkLibraries: list of libraries to link to.
-# Usage example:  gtsamAddScriptsGlob("*.cpp" "Broken.cpp" "gtsam;GeographicLib")
-macro(gtsamAddExamplesGlob globPatterns excludedFiles linkLibraries)
+
+macro(gtsamAddExamplesGlob_impl globPatterns excludedFiles linkLibraries)
     # Get all script files
     file(GLOB script_files ${globPatterns})
 
@@ -183,195 +237,3 @@ macro(gtsamAddExamplesGlob globPatterns excludedFiles linkLibraries)
 		set_property(TARGET ${script_name} PROPERTY FOLDER "Examples")
 	endforeach()
 endmacro()
-
-# Macro for adding categorized tests in a "tests" folder, with 
-# optional exclusion of tests and convenience library linking options
-#  
-# By default, all tests are linked with CppUnitLite and boost
-# Arguments: 
-#   - subdir    The name of the category for this test
-#   - local_libs  A list of convenience libraries to use (if GTSAM_BUILD_CONVENIENCE_LIBRARIES is true)
-#   - full_libs   The main library to link against if not using convenience libraries
-#   - excluded_tests  A list of test files that should not be compiled - use for debugging 
-function(gtsam_add_subdir_tests subdir local_libs full_libs excluded_tests)
-    # Subdirectory target for tests
-    add_custom_target(check.${subdir} COMMAND ${CMAKE_CTEST_COMMAND} -C $<CONFIGURATION> --output-on-failure)
-    set(is_test TRUE)
-
-	# Put check target in Visual Studio solution folder
-	file(RELATIVE_PATH relative_path "${PROJECT_SOURCE_DIR}" "${CMAKE_CURRENT_SOURCE_DIR}")
-	set_property(TARGET check.${subdir} PROPERTY FOLDER "${relative_path}")
-    
-	# Link with CppUnitLite - pulled from gtsam installation
-	list(APPEND local_libs CppUnitLite)
-	list(APPEND full_libs CppUnitLite)
-
-    # Build grouped tests
-    gtsam_add_grouped_scripts("${subdir}"               # Use subdirectory as group label
-    "tests/test*.cpp" check "Test"                      # Standard for all tests
-    "${local_libs}"
-    "${full_libs}" "${excluded_tests}"  # Pass in linking and exclusion lists
-    ${is_test})                                         # Set all as tests
-endfunction()
-
-# Macro for adding categorized timing scripts in a "tests" folder, with 
-# optional exclusion of tests and convenience library linking options
-#  
-# By default, all tests are linked with boost
-# Arguments: 
-#   - subdir    The name of the category for this timing script
-#   - local_libs  A list of convenience libraries to use (if GTSAM_BUILD_CONVENIENCE_LIBRARIES is true)
-#   - full_libs   The main library to link against if not using convenience libraries
-#   - excluded_srcs  A list of timing files that should not be compiled - use for debugging 
-macro(gtsam_add_subdir_timing subdir local_libs full_libs excluded_srcs)
-    # Subdirectory target for timing - does not actually execute the scripts
-    add_custom_target(timing.${subdir})
-    set(is_test FALSE)
-
-    # Build grouped benchmarks
-    gtsam_add_grouped_scripts("${subdir}"               # Use subdirectory as group label
-    "tests/time*.cpp" timing "Timing Benchmark"         # Standard for all timing scripts
-    "${local_libs}" "${full_libs}" "${excluded_srcs}"   # Pass in linking and exclusion lists
-    ${is_test})                                         # Treat as not a test
-endmacro()
-
-# Macro for adding executables matching a pattern - builds one executable for
-# each file matching the pattern.  These exectuables are automatically linked
-# with boost.
-# Arguments:
-#   - pattern    The glob pattern to match source files
-#   - local_libs A list of convenience libraries to use (if GTSAM_BUILD_CONVENIENCE_LIBRARIES is true)
-#   - full_libs  The main library to link against if not using convenience libraries
-#   - excluded_srcs  A list of timing files that should not be compiled - use for debugging
-function(gtsam_add_executables pattern local_libs full_libs excluded_srcs)
-    set(is_test FALSE)
-    
-    if(NOT excluded_srcs)
-        set(excluded_srcs "")
-    endif()
-    
-    # Build executables
-    gtsam_add_grouped_scripts("" "${pattern}" "" "Executable" "${local_libs}" "${full_libs}" "${excluded_srcs}" ${is_test})
-endfunction()
-
-# General-purpose script for adding tests with categories and linking options
-macro(gtsam_add_grouped_scripts group pattern target_prefix pretty_prefix_name local_libs full_libs excluded_srcs is_test) 
-    # Get all script files
-    set(script_files "")
-    foreach(one_pattern ${pattern})
-        file(GLOB one_script_files "${one_pattern}")
-        list(APPEND script_files "${one_script_files}")
-    endforeach()
-
-    # Remove excluded scripts from the list
-    set(exclusions "") # Need to copy out exclusion list for logic to work
-    foreach(one_exclusion ${excluded_srcs})
-        file(GLOB one_exclusion_srcs "${one_exclusion}")
-        list(APPEND exclusions "${one_exclusion_srcs}")
-    endforeach()
-    if(exclusions)
-    	list(REMOVE_ITEM script_files ${exclusions})
-    endif(exclusions)
-
-	# Separate into source files and headers
-	set(script_srcs "")
-	set(script_headers "")
-	foreach(script_file ${script_files})
-		get_filename_component(script_ext ${script_file} EXT)
-		if(script_ext MATCHES "(h|H)")
-			list(APPEND script_headers ${script_file})
-		else()
-			list(APPEND script_srcs ${script_file})
-		endif()
-	endforeach()
-		
-    
-    # Add targets and dependencies for each script
-    if(NOT "${group}" STREQUAL "")
-        message(STATUS "Adding ${pretty_prefix_name}s in ${group}")
-    endif()
-	
-	# Create exe's for each script, unless we're in SINGLE_TEST_EXE mode
-	if(NOT is_test OR NOT GTSAM_SINGLE_TEST_EXE)
-		foreach(script_src ${script_srcs})
-			get_filename_component(script_base ${script_src} NAME_WE)
-			if (script_base) # Check for null filenames and headers
-				set( script_bin ${script_base} )
-				message(STATUS "Adding ${pretty_prefix_name} ${script_bin}") 
-				add_executable(${script_bin} ${script_src} ${script_headers})
-				if(NOT "${target_prefix}" STREQUAL "")
-					if(NOT "${group}" STREQUAL "")
-						add_dependencies(${target_prefix}.${group} ${script_bin})
-					endif()
-					add_dependencies(${target_prefix} ${script_bin})
-				endif()
-				
-				# Add TOPSRCDIR
-				set_property(SOURCE ${script_src} APPEND PROPERTY COMPILE_DEFINITIONS "TOPSRCDIR=\"${PROJECT_SOURCE_DIR}\"")
-
-				# Disable building during make all/install
-				if (GTSAM_DISABLE_TESTS_ON_INSTALL)
-					set_target_properties(${script_bin} PROPERTIES EXCLUDE_FROM_ALL ON)
-				endif()
-				
-				if (is_test)
-					add_test(NAME ${script_base} COMMAND ${script_bin})
-				endif()
-				
-				# Linking and dependendencies
-				if (GTSAM_BUILD_CONVENIENCE_LIBRARIES)
-					target_link_libraries(${script_bin} ${local_libs} ${GTSAM_BOOST_LIBRARIES})
-				else()
-					target_link_libraries(${script_bin} ${full_libs} ${GTSAM_BOOST_LIBRARIES})
-				endif()
-				
-				# Add .run target
-				if(NOT MSVC AND NOT XCODE_VERSION)
-				  add_custom_target(${script_bin}.run ${EXECUTABLE_OUTPUT_PATH}${script_bin} ${ARGN})
-				endif()
-				
-				# Set up Visual Studio folders
-				file(RELATIVE_PATH relative_path "${PROJECT_SOURCE_DIR}" "${CMAKE_CURRENT_SOURCE_DIR}")
-				set_property(TARGET ${script_bin} PROPERTY FOLDER "${relative_path}")
-			endif()
-		endforeach(script_src)
-		
-		if(MSVC)
-			source_group("" FILES ${script_srcs} ${script_headers})
-		endif()
-	else()
-		# Create single unit test exe from all test scripts
-		set(script_bin ${target_prefix}_${group}_prog)
-		add_executable(${script_bin} ${script_srcs} ${script_headers})
-		if (GTSAM_BUILD_CONVENIENCE_LIBRARIES)
-			target_link_libraries(${script_bin} ${local_libs} ${Boost_LIBRARIES})
-		else()
-			target_link_libraries(${script_bin} ${Boost_LIBRARIES} ${full_libs})
-		endif()
-		
-		# Only have a main function in one script
-		set(rest_script_srcs ${script_srcs})
-		list(REMOVE_AT rest_script_srcs 0)
-		set_property(SOURCE ${rest_script_srcs} APPEND PROPERTY COMPILE_DEFINITIONS "main=static no_main")
-			
-		# Add TOPSRCDIR
-		set_property(SOURCE ${script_srcs} APPEND PROPERTY COMPILE_DEFINITIONS "TOPSRCDIR=\"${PROJECT_SOURCE_DIR}\"")
-			
-		# Add test
-		add_dependencies(${target_prefix}.${group} ${script_bin})
-		add_dependencies(${target_prefix} ${script_bin})
-		add_test(NAME ${target_prefix}.${group} COMMAND ${script_bin})
-		
-		# Disable building during make all/install
-		if (GTSAM_DISABLE_TESTS_ON_INSTALL)
-			set_target_properties(${script_bin} PROPERTIES EXCLUDE_FROM_ALL ON)
-		endif()
-		
-		# Set up Visual Studio folders
-		if(MSVC)
-		    file(RELATIVE_PATH relative_path "${PROJECT_SOURCE_DIR}" "${CMAKE_CURRENT_SOURCE_DIR}")
-			set_property(TARGET ${script_bin} PROPERTY FOLDER "${relative_path}")
-			source_group("" FILES ${script_srcs} ${script_headers})
-		endif()
-	endif()
-endmacro()
diff --git a/cmake/GtsamTestingObsolete.cmake b/cmake/GtsamTestingObsolete.cmake
new file mode 100644
index 000000000..f56d138e6
--- /dev/null
+++ b/cmake/GtsamTestingObsolete.cmake
@@ -0,0 +1,195 @@
+
+# Macro for adding categorized tests in a "tests" folder, with 
+# optional exclusion of tests and convenience library linking options
+#  
+# By default, all tests are linked with CppUnitLite and boost
+# Arguments: 
+#   - subdir    The name of the category for this test
+#   - local_libs  A list of convenience libraries to use (if GTSAM_BUILD_CONVENIENCE_LIBRARIES is true)
+#   - full_libs   The main library to link against if not using convenience libraries
+#   - excluded_tests  A list of test files that should not be compiled - use for debugging 
+function(gtsam_add_subdir_tests subdir local_libs full_libs excluded_tests)
+    # Subdirectory target for tests
+    add_custom_target(check.${subdir} COMMAND ${CMAKE_CTEST_COMMAND} -C $<CONFIGURATION> --output-on-failure)
+    set(is_test TRUE)
+
+	# Put check target in Visual Studio solution folder
+	file(RELATIVE_PATH relative_path "${PROJECT_SOURCE_DIR}" "${CMAKE_CURRENT_SOURCE_DIR}")
+	set_property(TARGET check.${subdir} PROPERTY FOLDER "${relative_path}")
+    
+	# Link with CppUnitLite - pulled from gtsam installation
+	list(APPEND local_libs CppUnitLite)
+	list(APPEND full_libs CppUnitLite)
+
+    # Build grouped tests
+    gtsam_add_grouped_scripts("${subdir}"               # Use subdirectory as group label
+    "tests/test*.cpp" check "Test"                      # Standard for all tests
+    "${local_libs}"
+    "${full_libs}" "${excluded_tests}"  # Pass in linking and exclusion lists
+    ${is_test})                                         # Set all as tests
+endfunction()
+
+# Macro for adding categorized timing scripts in a "tests" folder, with 
+# optional exclusion of tests and convenience library linking options
+#  
+# By default, all tests are linked with boost
+# Arguments: 
+#   - subdir    The name of the category for this timing script
+#   - local_libs  A list of convenience libraries to use (if GTSAM_BUILD_CONVENIENCE_LIBRARIES is true)
+#   - full_libs   The main library to link against if not using convenience libraries
+#   - excluded_srcs  A list of timing files that should not be compiled - use for debugging 
+macro(gtsam_add_subdir_timing subdir local_libs full_libs excluded_srcs)
+    # Subdirectory target for timing - does not actually execute the scripts
+    add_custom_target(timing.${subdir})
+    set(is_test FALSE)
+
+    # Build grouped benchmarks
+    gtsam_add_grouped_scripts("${subdir}"               # Use subdirectory as group label
+    "tests/time*.cpp" timing "Timing Benchmark"         # Standard for all timing scripts
+    "${local_libs}" "${full_libs}" "${excluded_srcs}"   # Pass in linking and exclusion lists
+    ${is_test})                                         # Treat as not a test
+endmacro()
+
+# Macro for adding executables matching a pattern - builds one executable for
+# each file matching the pattern.  These exectuables are automatically linked
+# with boost.
+# Arguments:
+#   - pattern    The glob pattern to match source files
+#   - local_libs A list of convenience libraries to use (if GTSAM_BUILD_CONVENIENCE_LIBRARIES is true)
+#   - full_libs  The main library to link against if not using convenience libraries
+#   - excluded_srcs  A list of timing files that should not be compiled - use for debugging
+function(gtsam_add_executables pattern local_libs full_libs excluded_srcs)
+    set(is_test FALSE)
+    
+    if(NOT excluded_srcs)
+        set(excluded_srcs "")
+    endif()
+    
+    # Build executables
+    gtsam_add_grouped_scripts("" "${pattern}" "" "Executable" "${local_libs}" "${full_libs}" "${excluded_srcs}" ${is_test})
+endfunction()
+
+# General-purpose script for adding tests with categories and linking options
+macro(gtsam_add_grouped_scripts group pattern target_prefix pretty_prefix_name local_libs full_libs excluded_srcs is_test)
+	# Print warning about using this obsolete function
+	message(AUTHOR_WARNING "Warning:  Please see GtsamTesting.cmake - obsolete cmake cmake macro for creating unit tests, examples, and scripts was called.  This will be removed in the future.  The new macros are much easier anyway!!")
+	
+    # Get all script files
+    set(script_files "")
+    foreach(one_pattern ${pattern})
+        file(GLOB one_script_files "${one_pattern}")
+        list(APPEND script_files "${one_script_files}")
+    endforeach()
+
+    # Remove excluded scripts from the list
+    set(exclusions "") # Need to copy out exclusion list for logic to work
+    foreach(one_exclusion ${excluded_srcs})
+        file(GLOB one_exclusion_srcs "${one_exclusion}")
+        list(APPEND exclusions "${one_exclusion_srcs}")
+    endforeach()
+    if(exclusions)
+    	list(REMOVE_ITEM script_files ${exclusions})
+    endif(exclusions)
+
+	# Separate into source files and headers
+	set(script_srcs "")
+	set(script_headers "")
+	foreach(script_file ${script_files})
+		get_filename_component(script_ext ${script_file} EXT)
+		if(script_ext MATCHES "(h|H)")
+			list(APPEND script_headers ${script_file})
+		else()
+			list(APPEND script_srcs ${script_file})
+		endif()
+	endforeach()
+		
+    
+    # Add targets and dependencies for each script
+    if(NOT "${group}" STREQUAL "")
+        message(STATUS "Adding ${pretty_prefix_name}s in ${group}")
+    endif()
+	
+	# Create exe's for each script, unless we're in SINGLE_TEST_EXE mode
+	if(NOT is_test OR NOT GTSAM_SINGLE_TEST_EXE)
+		foreach(script_src ${script_srcs})
+			get_filename_component(script_base ${script_src} NAME_WE)
+			if (script_base) # Check for null filenames and headers
+				set( script_bin ${script_base} )
+				message(STATUS "Adding ${pretty_prefix_name} ${script_bin}") 
+				add_executable(${script_bin} ${script_src} ${script_headers})
+				if(NOT "${target_prefix}" STREQUAL "")
+					if(NOT "${group}" STREQUAL "")
+						add_dependencies(${target_prefix}.${group} ${script_bin})
+					endif()
+					add_dependencies(${target_prefix} ${script_bin})
+				endif()
+				
+				# Add TOPSRCDIR
+				set_property(SOURCE ${script_src} APPEND PROPERTY COMPILE_DEFINITIONS "TOPSRCDIR=\"${PROJECT_SOURCE_DIR}\"")
+
+				# Disable building during make all/install
+				if (GTSAM_DISABLE_TESTS_ON_INSTALL)
+					set_target_properties(${script_bin} PROPERTIES EXCLUDE_FROM_ALL ON)
+				endif()
+				
+				if (is_test)
+					add_test(NAME ${script_base} COMMAND ${script_bin})
+				endif()
+				
+				# Linking and dependendencies
+				if (GTSAM_BUILD_CONVENIENCE_LIBRARIES)
+					target_link_libraries(${script_bin} ${local_libs} ${GTSAM_BOOST_LIBRARIES})
+				else()
+					target_link_libraries(${script_bin} ${full_libs} ${GTSAM_BOOST_LIBRARIES})
+				endif()
+				
+				# Add .run target
+				if(NOT MSVC AND NOT XCODE_VERSION)
+				  add_custom_target(${script_bin}.run ${EXECUTABLE_OUTPUT_PATH}${script_bin} ${ARGN})
+				endif()
+				
+				# Set up Visual Studio folders
+				file(RELATIVE_PATH relative_path "${PROJECT_SOURCE_DIR}" "${CMAKE_CURRENT_SOURCE_DIR}")
+				set_property(TARGET ${script_bin} PROPERTY FOLDER "${relative_path}")
+			endif()
+		endforeach(script_src)
+		
+		if(MSVC)
+			source_group("" FILES ${script_srcs} ${script_headers})
+		endif()
+	else()
+		# Create single unit test exe from all test scripts
+		set(script_bin ${target_prefix}_${group}_prog)
+		add_executable(${script_bin} ${script_srcs} ${script_headers})
+		if (GTSAM_BUILD_CONVENIENCE_LIBRARIES)
+			target_link_libraries(${script_bin} ${local_libs} ${Boost_LIBRARIES})
+		else()
+			target_link_libraries(${script_bin} ${Boost_LIBRARIES} ${full_libs})
+		endif()
+		
+		# Only have a main function in one script
+		set(rest_script_srcs ${script_srcs})
+		list(REMOVE_AT rest_script_srcs 0)
+		set_property(SOURCE ${rest_script_srcs} APPEND PROPERTY COMPILE_DEFINITIONS "main=static no_main")
+			
+		# Add TOPSRCDIR
+		set_property(SOURCE ${script_srcs} APPEND PROPERTY COMPILE_DEFINITIONS "TOPSRCDIR=\"${PROJECT_SOURCE_DIR}\"")
+			
+		# Add test
+		add_dependencies(${target_prefix}.${group} ${script_bin})
+		add_dependencies(${target_prefix} ${script_bin})
+		add_test(NAME ${target_prefix}.${group} COMMAND ${script_bin})
+		
+		# Disable building during make all/install
+		if (GTSAM_DISABLE_TESTS_ON_INSTALL)
+			set_target_properties(${script_bin} PROPERTIES EXCLUDE_FROM_ALL ON)
+		endif()
+		
+		# Set up Visual Studio folders
+		if(MSVC)
+		    file(RELATIVE_PATH relative_path "${PROJECT_SOURCE_DIR}" "${CMAKE_CURRENT_SOURCE_DIR}")
+			set_property(TARGET ${script_bin} PROPERTY FOLDER "${relative_path}")
+			source_group("" FILES ${script_srcs} ${script_headers})
+		endif()
+	endif()
+endmacro()