Updates for unit tests so they actually run and fix application test

qgis · Nov 14, 2011 · e54680d · e54680d
1 parent 1fa8386
commit e54680d
Show file tree

Hide file tree

Showing 17 changed files with 42 additions and 66 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -500,6 +500,8 @@ IF (ENABLE_TESTS)
   #the test data. See CMakeLists in test dirs for more info
   SET (TEST_DATA_DIR "${CMAKE_CURRENT_SOURCE_DIR}/tests/testdata")
   ADD_SUBDIRECTORY(tests)
+  SET (CTEST_BINARY_DIRECTORY "${CMAKE_CURRENT_BINARY_DIR}/output/bin" )
+  MESSAGE (STATUS "Ctest Binary Directory  set to: ${CTEST_BINARY_DIRECTORY}")
 ENDIF (ENABLE_TESTS)
 
 IF (APPLE)

diff --git a/tests/README b/tests/README
diff --git a/tests/src/analysis/CMakeLists.txt b/tests/src/analysis/CMakeLists.txt
@@ -7,8 +7,13 @@ SET (util_SRCS ../core/qgsrenderchecker.cpp)
 # the UI file won't be wrapped!
 INCLUDE_DIRECTORIES(${CMAKE_CURRENT_SOURCE_DIR} 
   ${CMAKE_CURRENT_BINARY_DIR}
-  ${CMAKE_CURRENT_SOURCE_DIR}/../../../src/core/
-  ${CMAKE_CURRENT_SOURCE_DIR}/../../../src/analysis/vector
+  ${CMAKE_SOURCE_DIR}/src/core
+  ${CMAKE_SOURCE_DIR}/src/core/raster
+  ${CMAKE_SOURCE_DIR}/src/core/renderer
+  ${CMAKE_SOURCE_DIR}/src/core/symbology
+  ${CMAKE_SOURCE_DIR}/src/core/symbology-ng
+  ${CMAKE_SOURCE_DIR}/src/analysis
+  ${CMAKE_SOURCE_DIR}/src/analysis/vector
   ${QT_INCLUDE_DIR}
   ${GDAL_INCLUDE_DIR}
   ${PROJ_INCLUDE_DIR}
@@ -34,58 +39,39 @@ ENDIF (WIN32)
 
 # Since the tests are not actually installed, but rather
 # run directly from the build/src/tests dir we need to
-# ensure the omg libs can be found.
+# ensure the qgis libs can be found.
 IF (APPLE)
   # For Mac OS X, the executable must be at the root of the bundle's executable folder
-  SET (CMAKE_INSTALL_NAME_DIR @executable_path/../../../src/analyzer)
+  SET (CMAKE_INSTALL_NAME_DIR @executable_path/../../../src/core)
 ENDIF (APPLE)
 
 #note for tests we should not include the moc of our
 #qtests in the executable file list as the moc is
 #directly included in the sources
 #and should not be compiled twice. Trying to include
 #them in will cause an error at build time 
+
+#No relinking and full RPATH for the install tree
+#See: http://www.cmake.org/Wiki/CMake_RPATH_handling#No_relinking_and_full_RPATH_for_the_install_tree
+
+MACRO (ADD_QGIS_TEST testname testsrc)
+  SET(qgis_${testname}_SRCS ${testsrc} ${util_SRCS})
+  SET(qgis_${testname}_MOC_CPPS ${testsrc})
+  QT4_WRAP_CPP(qgis_${testname}_MOC_SRCS ${qgis_${testname}_MOC_CPPS})
+  ADD_CUSTOM_TARGET(qgis_${testname}moc ALL DEPENDS ${qgis_${testname}_MOC_SRCS})
+  ADD_EXECUTABLE(qgis_${testname} ${qgis_${testname}_SRCS})
+  ADD_DEPENDENCIES(qgis_${testname} qgis_${testname}moc)
+  TARGET_LINK_LIBRARIES(qgis_${testname} ${QT_LIBRARIES} qgis_analysis)
+  ADD_TEST(qgis_${testname} ${CMAKE_CURRENT_BINARY_DIR}/../../../output/bin/qgis_${testname})
+  #SET_TARGET_PROPERTIES(qgis_${testname} PROPERTIES
+  #  INSTALL_RPATH ${CMAKE_INSTALL_PREFIX}/${QGIS_LIB_DIR}
+  #  INSTALL_RPATH_USE_LINK_PATH true )
+ENDMACRO (ADD_QGIS_TEST)
+
 #############################################################
 # Tests:
 
-#
-# QgsVectorAnalyzer test
-#
-SET(qgis_vectoranalyzertest_SRCS testqgsvectoranalyzer.cpp ${util_SRCS})
-SET(qgis_vectoranalyzertest_MOC_CPPS testqgsvectoranalyzer.cpp)
-QT4_WRAP_CPP(qgis_vectoranalyzertest_MOC_SRCS ${qgis_vectoranalyzertest_MOC_CPPS})
-ADD_CUSTOM_TARGET(qgis_vectoranalyzertestmoc ALL DEPENDS ${qgis_vectoranalyzertest_MOC_SRCS})
-ADD_EXECUTABLE(qgis_vectoranalyzertest ${qgis_vectoranalyzertest_SRCS})
-ADD_DEPENDENCIES(qgis_vectoranalyzertest qgis_vectoranalyzertestmoc)
-TARGET_LINK_LIBRARIES(qgis_vectoranalyzertest ${QT_LIBRARIES} qgis_core qgis_analysis)
-  #No relinking and full RPATH for the install tree
-  #See: http://www.cmake.org/Wiki/CMake_RPATH_handling#No_relinking_and_full_RPATH_for_the_install_tree
-SET_TARGET_PROPERTIES(qgis_vectoranalyzertest 
-  # skip the full RPATH for the build tree
-  PROPERTIES SKIP_BUILD_RPATH  TRUE
-  )
-SET_TARGET_PROPERTIES(qgis_vectoranalyzertest 
-  # when building, use the install RPATH already
-  # (so it doesn't need to relink when installing)
-  PROPERTIES BUILD_WITH_INSTALL_RPATH TRUE 
-  )
-SET_TARGET_PROPERTIES(qgis_vectoranalyzertest 
-  # the RPATH to be used when installing
-  PROPERTIES INSTALL_RPATH ${QGIS_LIB_DIR}
-  )
-SET_TARGET_PROPERTIES(qgis_vectoranalyzertest 
-  # add the automatically determined parts of the RPATH
-  # which point to directories outside the build tree to the install RPATH
-  PROPERTIES INSTALL_RPATH_USE_LINK_PATH TRUE
-  )
-IF (APPLE)
-  # For Mac OS X, the executable must be at the root of the bundle's executable folder
-  INSTALL(TARGETS qgis_vectoranalyzertest RUNTIME DESTINATION ${CMAKE_INSTALL_PREFIX})
-  ADD_TEST(qgis_vectoranalyzertest ${CMAKE_INSTALL_PREFIX}/qgis_vectoranalyzertest)
-ELSE (APPLE)
-  INSTALL(TARGETS qgis_vectoranalyzertest RUNTIME DESTINATION ${CMAKE_INSTALL_PREFIX}/bin)
-  ADD_TEST(qgis_vectoranalyzertest ${CMAKE_INSTALL_PREFIX}/bin/qgis_vectoranalyzertest)
-ENDIF (APPLE)
+ADD_QGIS_TEST(analyzertest testqgsvectoranalyzer.cpp)
 
 
 
diff --git a/tests/src/core/CMakeLists.txt b/tests/src/core/CMakeLists.txt
@@ -60,10 +60,10 @@ MACRO (ADD_QGIS_TEST testname testsrc)
   ADD_EXECUTABLE(qgis_${testname} ${qgis_${testname}_SRCS})
   ADD_DEPENDENCIES(qgis_${testname} qgis_${testname}moc)
   TARGET_LINK_LIBRARIES(qgis_${testname} ${QT_LIBRARIES} qgis_core)
-  ADD_TEST(qgis_${testname} qgis_${testname})
-  SET_TARGET_PROPERTIES(qgis_${testname} PROPERTIES
-    INSTALL_RPATH ${CMAKE_INSTALL_PREFIX}/${QGIS_LIB_DIR}
-    INSTALL_RPATH_USE_LINK_PATH true )
+  ADD_TEST(qgis_${testname} ${CMAKE_CURRENT_BINARY_DIR}/../../../output/bin/qgis_${testname})
+  #SET_TARGET_PROPERTIES(qgis_${testname} PROPERTIES
+  #  INSTALL_RPATH ${CMAKE_INSTALL_PREFIX}/${QGIS_LIB_DIR}
+  #  INSTALL_RPATH_USE_LINK_PATH true )
 ENDMACRO (ADD_QGIS_TEST)
 
 #############################################################

diff --git a/tests/src/core/testqgsapplication.cpp b/tests/src/core/testqgsapplication.cpp
@@ -27,7 +27,7 @@ class TestQgsApplication: public QObject
 {
     Q_OBJECT;
   private slots:
-    void checkTheme();
+    void checkPaths();
     void checkGdalSkip();
     void initTestCase();
   private:
@@ -41,20 +41,17 @@ void TestQgsApplication::initTestCase()
   // Runs once before any tests are run
   //
   // init QGIS's paths - true means that all path will be inited from prefix
-  QString qgisPath = QCoreApplication::applicationDirPath();
-  QgsApplication::setPrefixPath( INSTALL_PREFIX, true );
+  //QString qgisPath = QCoreApplication::applicationDirPath();
+  QgsApplication::init( INSTALL_PREFIX );
   qDebug( "%s", QgsApplication::showSettings().toUtf8().constData() );
 };
 
-void TestQgsApplication::checkTheme()
+void TestQgsApplication::checkPaths()
 {
-  QString myIconPath = QgsApplication::defaultThemePath();
-  QPixmap myPixmap;
-  myPixmap.load( myIconPath + "mActionFileNew.png" );
-  qDebug( "Checking if a theme icon exists:" );
-  qDebug( "%s/mIconProjectionDisabled.png", myIconPath.toLocal8Bit().constData() );
-  QVERIFY( !myPixmap.isNull() );
-
+  QString myPath = QgsApplication::authorsFilePath();
+  qDebug( "Checking authors file exists:" );
+  qDebug( "%s", myPath.toLocal8Bit().constData() );
+  QVERIFY( !myPath.isEmpty() );
 };
 
 void TestQgsApplication::checkGdalSkip()

diff --git a/tests/testdata/expected_continuous.png b/tests/testdata/expected_continuous.png
diff --git a/tests/testdata/expected_geometry_bufferCheck.png b/tests/testdata/expected_geometry_bufferCheck.png
diff --git a/tests/testdata/expected_geometry_differenceCheck1.png b/tests/testdata/expected_geometry_differenceCheck1.png
diff --git a/tests/testdata/expected_geometry_differenceCheck2.png b/tests/testdata/expected_geometry_differenceCheck2.png
diff --git a/tests/testdata/expected_geometry_intersectionCheck1.png b/tests/testdata/expected_geometry_intersectionCheck1.png
diff --git a/tests/testdata/expected_geometry_simplifyCheck1.png b/tests/testdata/expected_geometry_simplifyCheck1.png
diff --git a/tests/testdata/expected_geometry_unionCheck1.png b/tests/testdata/expected_geometry_unionCheck1.png
diff --git a/tests/testdata/expected_geometry_unionCheck2.png b/tests/testdata/expected_geometry_unionCheck2.png
diff --git a/tests/testdata/expected_landsat_875.png b/tests/testdata/expected_landsat_875.png
diff --git a/tests/testdata/expected_landsat_basic.png b/tests/testdata/expected_landsat_basic.png
diff --git a/tests/testdata/expected_raster_pseudo.png b/tests/testdata/expected_raster_pseudo.png
diff --git a/tests/testdata/expected_uniquevalue.png b/tests/testdata/expected_uniquevalue.png