Skip to content

Commit 96b2d51

Browse files
committed
fix tests on windows
1 parent 07e3c16 commit 96b2d51

File tree

5 files changed

+11
-6
lines changed

5 files changed

+11
-6
lines changed

cmake/UsePythonTest.cmake

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,8 @@ MACRO(ADD_PYTHON_TEST TESTNAME FILENAME)
3535
FILE(WRITE ${CMAKE_CURRENT_BINARY_DIR}/${TESTNAME}.cmake
3636
"
3737
IF(WIN32)
38-
SET(ENV{QGIS_PREFIX_PATH} \"${QGIS_OUTPUT_DIRECTORY}/bin/${CMAKE_BUILD_TYPE}\")
39-
SET(ENV{PATH} \"${QGIS_OUTPUT_DIRECTORY}/bin/${CMAKE_BUILD_TYPE};\$ENV{PATH}\")
38+
SET(ENV{QGIS_PREFIX_PATH} \"${QGIS_OUTPUT_DIRECTORY}/bin/\${CMAKE_BUILD_TYPE}\")
39+
SET(ENV{PATH} \"${QGIS_OUTPUT_DIRECTORY}/bin/\${CMAKE_BUILD_TYPE};\$ENV{PATH}\")
4040
SET(ENV{PYTHONPATH} \"${QGIS_OUTPUT_DIRECTORY}/python/;\$ENV{PYTHONPATH}\")
4141
MESSAGE(\"PATH:\$ENV{PATH}\")
4242
ELSE(WIN32)
@@ -63,7 +63,11 @@ IF(import_res)
6363
ENDIF(import_res)
6464
"
6565
)
66-
ADD_TEST(${TESTNAME} ${CMAKE_COMMAND} -P ${CMAKE_CURRENT_BINARY_DIR}/${TESTNAME}.cmake)
66+
IF(CMAKE_CONFIGURATION_TYPES)
67+
ADD_TEST(NAME ${TESTNAME} COMMAND ${CMAKE_COMMAND} -D CMAKE_BUILD_TYPE=$<CONFIGURATION> -P ${CMAKE_CURRENT_BINARY_DIR}/${TESTNAME}.cmake)
68+
ELSE(CMAKE_CONFIGURATION_TYPES)
69+
ADD_TEST(NAME ${TESTNAME} COMMAND ${CMAKE_COMMAND} -P ${CMAKE_CURRENT_BINARY_DIR}/${TESTNAME}.cmake)
70+
ENDIF(CMAKE_CONFIGURATION_TYPES)
6771
ENDMACRO(ADD_PYTHON_TEST)
6872

6973
# Byte compile recursively a directory (DIRNAME)

tests/src/core/testqgscomposermap.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ class TestQgsComposerMap: public QObject
4848

4949
void TestQgsComposerMap::initTestCase()
5050
{
51-
QgsApplication::init( QString() );
51+
QgsApplication::init();
5252
QgsApplication::initQgis();
5353

5454
//create maplayers from testdata and add to layer registry

tests/src/core/testqgsrasterlayer.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ class TestQgsRasterLayer: public QObject
8383
void TestQgsRasterLayer::initTestCase()
8484
{
8585
// init QGIS's paths - true means that all path will be inited from prefix
86-
QgsApplication::init( QString() );
86+
QgsApplication::init();
8787
QgsApplication::initQgis();
8888
// disable any PAM stuff to make sure stats are consistent
8989
CPLSetConfigOption( "GDAL_PAM_ENABLED", "NO" );

tests/src/providers/testqgswcsprovider.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,7 @@ class TestQgsWcsProvider: public QObject
6262
void TestQgsWcsProvider::initTestCase()
6363
{
6464
// init QGIS's paths - true means that all path will be inited from prefix
65-
QgsApplication::init( QString() );
65+
QgsApplication::init();
6666
QgsApplication::initQgis();
6767
QString mySettings = QgsApplication::showSettings();
6868
mySettings = mySettings.replace( "\n", "<br />" );

tests/src/python/test_qgslogger.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ def testLogger(self):
2626
#myLogger.fatal('Aaaargh...fatal'); #kills QGIS not testable
2727
myFile = open(myFilename, 'rt')
2828
myText = myFile.readlines()
29+
myFile.close()
2930
myExpectedText = ['QGIS Logger Unit Test\n',
3031
'This is a debug\n',
3132
'This is a warning\n',

0 commit comments

Comments
 (0)