Use macro to allow Qt Creator to detect tests

tests/CMakeLists.txt

@@ -12,6 +12,10 @@ set(QT_LIBRARIES Qt5::Core Qt5::Gui Qt5::Test Qt5::Widgets)
 file(GLOB_RECURSE SOURCES "src/*.cpp")
 include_directories("src/" "../lib/src/")
 
+if(MSVC OR CMAKE_CXX_COMPILER MATCHES "cl\\.exe")
+	set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} /wd4005") # Suppress "macro redifinition" warnings
+endif()
+
 add_executable(${PROJECT_NAME} ${SOURCES})
 add_test(${PROJECT_NAME} ${PROJECT_NAME})
 target_link_libraries(${PROJECT_NAME} ${QT_LIBRARIES} lib)

tests/src/commands/sql-worker-test.cpp

@@ -77,4 +77,4 @@ void SqlWorkerTest::testExecCreateAndInsert()
 }
 
 
-static SqlWorkerTest instance;
+QTEST_MAIN(SqlWorkerTest)

tests/src/downloader/extension-rotator-test.cpp

@@ -77,4 +77,4 @@ void ExtensionRotatorTest::testCopyConstructor()
 }
 
 
-static ExtensionRotatorTest instance;
+QTEST_MAIN(ExtensionRotatorTest)

tests/src/downloader/file-downloader-test.cpp

@@ -76,4 +76,4 @@ void FileDownloaderTest::testFailedStart()
 }
 
 
-static FileDownloaderTest instance;
+QTEST_MAIN(FileDownloaderTest)

tests/src/downloader/image-downloader-test.cpp

@@ -168,4 +168,4 @@ void ImageDownloaderTest::assertDownload(QSharedPointer<Image> img, ImageDownloa
 }
 
 
-static ImageDownloaderTest instance;
+QTEST_MAIN(ImageDownloaderTest)

tests/src/functions-test.cpp

@@ -299,4 +299,4 @@ void FunctionsTest::assertFixFilename(int platform, const QString &filename, con
 	QCOMPARE(actual, expected);
 }
 
-static FunctionsTest instance;
+QTEST_MAIN(FunctionsTest)

tests/src/integration/behoimi-test.cpp

@@ -109,4 +109,4 @@ void BehoimiTest::testJsonTags()
 }
 
 
-static BehoimiTest instance;
+QTEST_MAIN(BehoimiTest)

tests/src/integration/booru-org-test.cpp

@@ -38,4 +38,4 @@ void BooruOrgTest::testPageTags()
 }
 
 
-static BooruOrgTest instance;
+QTEST_MAIN(BooruOrgTest)

tests/src/integration/danbooru-test.cpp

@@ -90,4 +90,4 @@ void DanbooruTest::testJsonTags()
 }
 
 
-static DanbooruTest instance;
+QTEST_MAIN(DanbooruTest)

tests/src/integration/derpibooru-test.cpp

@@ -65,4 +65,4 @@ void DerpibooruTest::testJsonTags()
 }
 
 
-static DerpibooruTest instance;
+QTEST_MAIN(DerpibooruTest)

tests/src/integration/e621-test.cpp

@@ -93,4 +93,4 @@ void E621Test::testJsonTags()
 }
 
 
-static E621Test instance;
+QTEST_MAIN(E621Test)

tests/src/integration/gelbooru-test.cpp

@@ -68,4 +68,4 @@ void GelbooruTest::testHtmlTags()
 }
 
 
-static GelbooruTest instance;
+QTEST_MAIN(GelbooruTest)

tests/src/integration/sankaku-test.cpp

@@ -44,4 +44,4 @@ void SankakuTest::testAnimatedUrls()
 }
 
 
-static SankakuTest instance;
+QTEST_MAIN(SankakuTest)

tests/src/integration/zerochan-test.cpp

@@ -42,4 +42,4 @@ void ZerochanTest::testRss()
 }
 
 
-static ZerochanTest instance;
+QTEST_MAIN(ZerochanTest)

tests/src/loader/token-test.cpp

@@ -35,4 +35,4 @@ void TokenTest::testLazyWithoutCaching()
 }
 
 
-static TokenTest instance;
+QTEST_MAIN(TokenTest)

tests/src/main.cpp

@@ -12,33 +12,28 @@ int main(int argc, char *argv[])
 		QGuiApplication a(argc, argv);
 	#endif
 
+	// A possible format to filter by test suite it to pass their names as arguments
 	QStringList testSuites;
-	testSuites.reserve(argc - 1);
 	for (int i = 1; i < argc; ++i)
-		testSuites.append(argv[i]);
-
-	QMap<QString, int> results;
-	int failed = 0;
+	{
+		QString arg(argv[i]);
+		if (!arg.startsWith('-') && !arg.startsWith("test"))
+			testSuites.append(argv[i]);
+	}
 
+	// Used for networking and finding test resource files
 	setTestModeEnabled(true);
 
+	// Run all selected test suites
+	int errorCode = 0;
 	for (TestSuite *suite : TestSuite::getSuites())
 	{
 		if (!testSuites.isEmpty() && !testSuites.contains(suite->metaObject()->className()))
 			continue;
 
-		int result = QTest::qExec(suite);
-		results.insert(suite->metaObject()->className(), result);
-		if (result != 0)
-		{
-			failed++;
-		}
-	}
-
-	for (auto it = results.begin(); it != results.end(); ++it)
-	{
-		std::cout << '[' << (it.value() != 0 ? "FAIL" : "OK") << "] " << it.key().toStdString() << std::endl;
+		errorCode |= QTest::qExec(suite, argc, argv);
+		std::cout << std::endl;
 	}
 
-	return failed;
+	return errorCode;
 }

tests/src/mixed-settings-test.cpp

@@ -93,4 +93,4 @@ void MixedSettingsTest::testChildKeys()
 }
 
 
-static MixedSettingsTest instance;
+QTEST_MAIN(MixedSettingsTest)

tests/src/models/favorite-test.cpp

@@ -272,4 +272,4 @@ void FavoriteTest::testSortByLastViewed()
 }
 
 
-static FavoriteTest instance;
+QTEST_MAIN(FavoriteTest)

tests/src/models/filename-test.cpp

@@ -740,4 +740,5 @@ void FilenameTest::assertExpand(const QString &format, const QString &expected)
 	QCOMPARE(actual, expected);
 }
 
-static FilenameTest instance;
+
+QTEST_MAIN(FilenameTest)

tests/src/models/filtering/blacklist-test.cpp

@@ -54,4 +54,4 @@ void BlacklistTest::testMatch()
 }
 
 
-static BlacklistTest instance;
+QTEST_MAIN(BlacklistTest)

tests/src/models/filtering/meta-filter-test.cpp

@@ -135,4 +135,4 @@ void MetaFilterTest::testMatchString()
 }
 
 
-static MetaFilterTest instance;
+QTEST_MAIN(MetaFilterTest)

tests/src/models/filtering/post-filter-test.cpp

@@ -106,4 +106,4 @@ void PostFilterTest::testFilterInvert()
 }
 
 
-static PostFilterTest instance;
+QTEST_MAIN(PostFilterTest)

tests/src/models/filtering/tag-filter-test.cpp

@@ -46,4 +46,4 @@ void TagFilterTest::testMatchWildcard()
 }
 
 
-static TagFilterTest instance;
+QTEST_MAIN(TagFilterTest)

tests/src/models/filtering/token-filter-test.cpp

@@ -69,4 +69,4 @@ void TokenFilterTest::testMatchStringList()
 }
 
 
-static TokenFilterTest instance;
+QTEST_MAIN(TokenFilterTest)

tests/src/models/image-test.cpp

@@ -372,4 +372,4 @@ void ImageTest::testSetUrl()
 }
 
 
-static ImageTest instance;
+QTEST_MAIN(ImageTest)

tests/src/models/page-api-test.cpp

@@ -66,4 +66,4 @@ void PageApiTest::testParseUrlAltPage()
 }
 
 
-static PageApiTest instance;
+QTEST_MAIN(PageApiTest)

tests/src/models/page-test.cpp

@@ -46,4 +46,4 @@ void PageTest::testLoadTagsAbort()
 }
 
 
-static PageTest instance;
+QTEST_MAIN(PageTest)

tests/src/models/pool-test.cpp

@@ -7,25 +7,30 @@ void PoolTest::testGetId()
 	Pool pool(123, "Test pool", 1, 2, 3);
 	QCOMPARE(pool.id(), 123);
 }
+
 void PoolTest::testGetName()
 {
 	Pool pool(123, "Test pool", 1, 2, 3);
 	QCOMPARE(pool.name(), QString("Test pool"));
 }
+
 void PoolTest::testGetCurrent()
 {
 	Pool pool(123, "Test pool", 1, 2, 3);
 	QCOMPARE(pool.current(), 1);
 }
+
 void PoolTest::testGetNext()
 {
 	Pool pool(123, "Test pool", 1, 2, 3);
 	QCOMPARE(pool.next(), 2);
 }
+
 void PoolTest::testGetPrevious()
 {
 	Pool pool(123, "Test pool", 1, 2, 3);
 	QCOMPARE(pool.previous(), 3);
 }
 
-static PoolTest instance;
+
+QTEST_MAIN(PoolTest)

tests/src/models/profile-test.cpp

@@ -216,4 +216,4 @@ void ProfileTest::testMd5ActionKeepDeleted()
 
 
 
-static ProfileTest instance;
+QTEST_MAIN(ProfileTest)

tests/src/models/site-test.cpp

@@ -227,4 +227,4 @@ void SiteTest::testLoginPost()
 }
 
 
-static SiteTest instance;
+QTEST_MAIN(SiteTest)

tests/src/models/source-guesser-test.cpp

@@ -73,4 +73,4 @@ void SourceGuesserTest::testDanbooru2()
 }
 
 
-static SourceGuesserTest instance;
+QTEST_MAIN(SourceGuesserTest)

tests/src/models/source-test.cpp

@@ -90,4 +90,4 @@ void SourceTest::testIgnoreEmptySites()
 }
 
 
-static SourceTest instance;
+QTEST_MAIN(SourceTest)

tests/src/tags/tag-api-test.cpp

@@ -39,4 +39,4 @@ void TagApiTest::testBasic()
 }
 
 
-static TagApiTest instance;
+QTEST_MAIN(TagApiTest)

tests/src/tags/tag-database-in-memory-test.cpp

@@ -113,4 +113,4 @@ void TagDatabaseInMemoryTest::saveData()
 }
 
 
-static TagDatabaseInMemoryTest instance;
+QTEST_MAIN(TagDatabaseInMemoryTest)

tests/src/tags/tag-database-sqlite-test.cpp

@@ -8,4 +8,4 @@ TagDatabaseSqliteTest::TagDatabaseSqliteTest()
 {}
 
 
-static TagDatabaseSqliteTest instance;
+QTEST_MAIN(TagDatabaseSqliteTest)

tests/src/tags/tag-name-format-test.cpp

@@ -53,4 +53,4 @@ void TagNameFormatTest::testUnknown()
 }
 
 
-static TagNameFormatTest instance;
+QTEST_MAIN(TagNameFormatTest)

tests/src/tags/tag-name-test.cpp

@@ -45,4 +45,4 @@ void TagNameTest::testCompare()
 }
 
 
-static TagNameTest instance;
+QTEST_MAIN(TagNameTest)

tests/src/tags/tag-stylist-test.cpp

@@ -143,4 +143,4 @@ void TagStylistTest::assertSort(const QString &sort, const QStringList &expected
 }
 
 
-static TagStylistTest instance;
+QTEST_MAIN(TagStylistTest)

tests/src/tags/tag-test.cpp

@@ -228,4 +228,4 @@ void TagTest::testGetType()
 }
 
 
-static TagTest instance;
+QTEST_MAIN(TagTest)

tests/src/test-suite.h

@@ -7,6 +7,9 @@
 #include "models/profile.h"
 
 
+#define QTEST_MAIN(CLASS_NAME) static CLASS_NAME instance;
+
+
 #ifdef _MSC_VER
 	#include <crtdbg.h>
 

tests/src/updater/source-updater-test.cpp

@@ -39,4 +39,4 @@ void SourceUpdaterTest::testChanged()
 #endif
 
 
-static SourceUpdaterTest instance;
+QTEST_MAIN(SourceUpdaterTest)

tests/src/updater/updater-test.cpp

@@ -55,4 +55,4 @@ void UpdaterTest::testCompareAlphaToOld()
 }
 
 
-static UpdaterTest instance;
+QTEST_MAIN(UpdaterTest)