From 50acffa593c805d084439466e20f48810b59159e Mon Sep 17 00:00:00 2001
From: "Troy D. Straszheim" <troy@resophonic.com>
Date: Wed, 8 Apr 2009 15:46:14 +0000
Subject: [PATCH] Tune up python tests for removal of our ctest hacks

[SVN r52255]
---
 CMakeLists.txt      | 49 ++++++++++++++++++++-------------------------
 test/CMakeLists.txt |  9 +++------
 test/pyrun.py       |  7 +++++++
 3 files changed, 32 insertions(+), 33 deletions(-)
 create mode 100644 test/pyrun.py

diff --git a/CMakeLists.txt b/CMakeLists.txt
index e057f349..da1164e5 100644
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -40,36 +40,31 @@ if (PYTHON_LIBRARIES)
       "" 
       ${ARGN})
       
-    #TODO: The target properties are NOT being set correctly for the test libraries  
-    if (FALSE)
-        # Create the library target itself
-        add_library(${MODULE_NAME} MODULE ${BPL_EXT_DEFAULT_ARGS} )
+    # Create the library target itself
+    add_library(${MODULE_NAME} MODULE ${BPL_EXT_DEFAULT_ARGS})
     
-        # Miscellaneous target properties
-        set_target_properties(${MODULE_NAME} PROPERTIES PREFIX "")
+    # Link against Boost.Python library
+    target_link_libraries(${MODULE_NAME} boost_python-mt-shared)
     
-        # Link against Boost.Python library
-        target_link_libraries(${MODULE_NAME} boost_python-static)
-    
-        # Link against Python libraries
-        target_link_libraries(${MODULE_NAME} ${PYTHON_LIBRARIES})
-    endif(FALSE)
+    # Link against Python libraries
+    target_link_libraries(${MODULE_NAME} ${PYTHON_LIBRARIES})
 
-    boost_add_library(
-      ${MODULE_NAME}
-      ${BPL_EXT_DEFAULT_ARGS}
-      MODULE
-      LINK_LIBS ${PYTHON_LIBRARIES}
-      DEPENDS boost_python
-      NO_STATIC
-      NO_SINGLE_THREADED
-    )
-  
-  set_target_properties(${MODULE_NAME}-mt-shared
-    PROPERTIES
-    PREFIX ""
-    OUTPUT_NAME ${MODULE_NAME}
-    )
+    #    boost_add_library(
+    #      ${MODULE_NAME}
+    #      ${BPL_EXT_DEFAULT_ARGS}
+    #      MODULE
+    #      LINK_LIBS ${PYTHON_LIBRARIES}
+    #      DEPENDS boost_python
+    #      NO_STATIC
+    #      NO_SINGLE_THREADED
+    #    )
+    
+    set_target_properties(${MODULE_NAME}
+      PROPERTIES
+      PREFIX ""
+      OUTPUT_NAME ${MODULE_NAME}
+      LIBRARY_OUTPUT_DIRECTORY ${CMAKE_LIBRARY_OUTPUT_DIRECTORY}/test
+      )
 
   endmacro(boost_python_extension)
 #--
diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt
index fd86d455..74a8ee9a 100644
--- a/test/CMakeLists.txt
+++ b/test/CMakeLists.txt
@@ -36,15 +36,12 @@ macro(bpl_test TESTNAME)
   set(BPL_TEST_PYSOURCE ${CMAKE_CURRENT_SOURCE_DIR}/${BPL_TEST_PYSOURCE})
     
   # Run the test itself
-  file(TO_NATIVE_PATH "${LIBRARY_OUTPUT_PATH}" PYTHONPATH)
+  file(TO_NATIVE_PATH "${CMAKE_LIBRARY_OUTPUT_DIRECTORY}/test" PYTHONPATH)
   if(WIN32 AND NOT UNIX)
     string(REPLACE "\\" "\\\\" PYTHONPATH "${PYTHONPATH}")
   endif(WIN32 AND NOT UNIX)
-  add_test("${PROJECT_NAME}::${TESTNAME}"
-    ${PYTHON_EXECUTABLE} 
-    "${CMAKE_CURRENT_SOURCE_DIR}/pyrun.py" 
-    "${PYTHONPATH}"
-    ${BPL_TEST_PYSOURCE} ${BPL_TEST_ARGS})
+
+  add_test("Python::${TESTNAME}" ${PYTHON_EXECUTABLE} ${CMAKE_CURRENT_SOURCE_DIR}/pyrun.py ${PYTHONPATH} ${BPL_TEST_PYSOURCE} ${BPL_TEST_ARGS})
 endmacro(bpl_test)
 
 macro(py_run TESTNAME)
diff --git a/test/pyrun.py b/test/pyrun.py
new file mode 100644
index 00000000..e033069e
--- /dev/null
+++ b/test/pyrun.py
@@ -0,0 +1,7 @@
+import sys
+
+pythonpath = sys.argv[1]
+scriptfile = sys.argv[2]
+sys.argv = sys.argv[2:]
+sys.path.append(pythonpath)
+execfile(scriptfile)