QSound/QSoundEffect: Use QFINDTESTDATA to locate test data.
authorFriedemann Kleint <Friedemann.Kleint@nokia.com>
Tue, 3 Jan 2012 12:41:33 +0000 (13:41 +0100)
committerQt by Nokia <qt-info@nokia.com>
Thu, 5 Jan 2012 03:01:51 +0000 (04:01 +0100)
Replace SRCDIR define by QFINDTESTDATA, which is the standard
method of locating test data. Remove apparently unused code path
within QT_QSOUNDEFFECT_USEAPPLICATIONPATH.

Change-Id: Id7120e75af727839ad2d1d2b3ef25c7ef2122fa5
Reviewed-by: Mithra Pattison <mithra.pattison@nokia.com>
Reviewed-by: Michael Goddard <michael.goddard@nokia.com>
tests/auto/integration/qsound/qsound.pro
tests/auto/integration/qsound/tst_qsound.cpp
tests/auto/integration/qsoundeffect/qsoundeffect.pro
tests/auto/integration/qsoundeffect/tst_qsoundeffect.cpp

index da4f6d9..d8960ab 100644 (file)
@@ -7,6 +7,3 @@ CONFIG += no_private_qt_headers_warning
 # CONFIG += testcase
 
 SOURCES += tst_qsound.cpp
-
-DEFINES += SRCDIR=\\\"$$PWD/\\\"
-
index 588619f..1cbf4f6 100644 (file)
@@ -67,7 +67,10 @@ private:
 
 void tst_QSound::initTestCase()
 {
-    sound = new QSound(QString(SRCDIR "test.wav"), this);
+    const QString testFileName = QStringLiteral("test.wav");
+    const QString fullPath = QFINDTESTDATA(testFileName);
+    QVERIFY2(!fullPath.isEmpty(), qPrintable(QStringLiteral("Unable to locate ") + testFileName));
+    sound = new QSound(fullPath, this);
 
     QVERIFY(!sound->fileName().isEmpty());
     QCOMPARE(sound->loops(),1);
@@ -120,7 +123,11 @@ void tst_QSound::testStop()
 void tst_QSound::testStaticPlay()
 {
     // Check that you hear sound with static play also.
-    QSound::play(QString(SRCDIR "test2.wav"));
+    const QString testFileName = QStringLiteral("test2.wav");
+    const QString fullPath = QFINDTESTDATA(testFileName);
+    QVERIFY2(!fullPath.isEmpty(), qPrintable(QStringLiteral("Unable to locate ") + testFileName));
+
+    QSound::play(fullPath);
 
     QTest::qWait(1000);
 }
index 4d7f083..352273a 100644 (file)
@@ -8,8 +8,6 @@ CONFIG += no_private_qt_headers_warning
 
 SOURCES += tst_qsoundeffect.cpp
 
-DEFINES += SRCDIR=\\\"$$PWD/\\\"
-
 unix:!mac {
     !contains(QT_CONFIG, pulseaudio) {
         DEFINES += QT_MULTIMEDIA_QMEDIAPLAYER
index f2c576a..771110c 100644 (file)
@@ -75,11 +75,11 @@ private:
 
 void tst_QSoundEffect::initTestCase()
 {
-#ifdef QT_QSOUNDEFFECT_USEAPPLICATIONPATH
-    url = QUrl::fromLocalFile(QCoreApplication::applicationDirPath() + QString("/test.wav"));
-#else
-    url = QUrl::fromLocalFile(QString(SRCDIR "test.wav"));
-#endif
+    const QString testFileName = QStringLiteral("test.wav");
+    const QString fullPath = QFINDTESTDATA(testFileName);
+    QVERIFY2(!fullPath.isEmpty(), qPrintable(QStringLiteral("Unable to locate ") + testFileName));
+
+    url = QUrl::fromLocalFile(fullPath);
 
     sound = new QSoundEffect(this);