diff options
| -rw-r--r-- | src/test/seratomarkers2test.cpp | 7 | ||||
| -rw-r--r-- | src/test/seratomarkerstest.cpp | 7 |
2 files changed, 8 insertions, 6 deletions
diff --git a/src/test/seratomarkers2test.cpp b/src/test/seratomarkers2test.cpp index 5131ecea30..7abfb9703c 100644 --- a/src/test/seratomarkers2test.cpp +++ b/src/test/seratomarkers2test.cpp @@ -112,12 +112,13 @@ class SeratoMarkers2Test : public testing::Test { void parseMarkers2DataInDirectory( QDir dir, mixxx::taglib::FileType fileType) { + EXPECT_TRUE(dir.exists()); dir.setFilter(QDir::Files); dir.setNameFilters(QStringList() << "*.octet-stream"); - QFileInfoList list = dir.entryInfoList(); - for (int i = 0; i < list.size(); i++) { - QFileInfo fileInfo = list.at(i); + QFileInfoList fileList = dir.entryInfoList(); + EXPECT_FALSE(fileList.isEmpty()); + for (const QFileInfo& fileInfo : fileList) { qDebug() << "--- File:" << fileInfo.fileName(); QFile file(dir.filePath(fileInfo.fileName())); bool openOk = file.open(QIODevice::ReadOnly); diff --git a/src/test/seratomarkerstest.cpp b/src/test/seratomarkerstest.cpp index 7c08099632..fb9b4a0d30 100644 --- a/src/test/seratomarkerstest.cpp +++ b/src/test/seratomarkerstest.cpp @@ -57,12 +57,13 @@ class SeratoMarkersTest : public testing::Test { void parseMarkersDataInDirectory( QDir dir, mixxx::taglib::FileType fileType) { + EXPECT_TRUE(dir.exists()); dir.setFilter(QDir::Files); dir.setNameFilters(QStringList() << "*.octet-stream"); - QFileInfoList list = dir.entryInfoList(); - for (int i = 0; i < list.size(); i++) { - QFileInfo fileInfo = list.at(i); + QFileInfoList fileList = dir.entryInfoList(); + EXPECT_FALSE(fileList.isEmpty()); + for (const QFileInfo& fileInfo : fileList) { qDebug() << "--- File:" << fileInfo.fileName(); QFile file(dir.filePath(fileInfo.fileName())); bool openOk = file.open(QIODevice::ReadOnly); |