fix: Use Qt_QPA_PLATFORM=offscreen for testing

Otherwise the tests always popped up new windows, which was very annoying as they kept grabbing focus on my setup. Fix #53
KDAB · Jul 9, 2024 · f8c61a1 · f8c61a1
1 parent 494a62d
commit f8c61a1
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 3 deletions.
diff --git a/3rdparty-kdab/mfc-utils b/3rdparty-kdab/mfc-utils
diff --git a/3rdparty-kdab/photonwidgets b/3rdparty-kdab/photonwidgets
diff --git a/tests/CMakeLists.txt b/tests/CMakeLists.txt
@@ -33,6 +33,7 @@ function(add_knut_test_variadic name)
   target_include_directories(${name} PRIVATE ${CMAKE_CURRENT_SOURCE_DIR}/../src)
 
   add_test(${name} ${CMAKE_BINARY_DIR}/bin/${name})
+  set_property(TEST ${name} PROPERTY ENVIRONMENT "QT_QPA_PLATFORM=offscreen")
 endfunction()
 
 # * Create a test, with one source and arbitrary libs

diff --git a/tests/tst_knut.cpp b/tests/tst_knut.cpp
@@ -36,7 +36,11 @@ class TestKnut : public QObject
 private:
     void run_knut(const QStringList &arguments)
     {
-        const int failedTests = QProcess::execute(KNUT_BINARY_PATH, arguments);
+        QProcess knut;
+        // Propagate the process environment.
+        // This allows us to e.g. pass QT_QPA_PLATFORM=offscreen to avoid window popups
+        knut.setProcessEnvironment(QProcessEnvironment::systemEnvironment());
+        const int failedTests = knut.execute(KNUT_BINARY_PATH, arguments);
 
         switch (failedTests) {
         case -2: