1 files changed, 19 insertions, 28 deletions

diff --git a/tests/auto/corelib/io/qprocess/tst_qprocess.cpp b/tests/auto/corelib/io/qprocess/tst_qprocess.cpp
index a7ecf6cae4..317db0574c 100644
--- a/tests/auto/corelib/io/qprocess/tst_qprocess.cpp
+++ b/tests/auto/corelib/io/qprocess/tst_qprocess.cpp
@@ -63,7 +63,7 @@ private slots:
     void getSetCheck();
     void constructing();
     void simpleStart();
-    void setupChildProcess();
+    void setChildProcessModifier();
     void startWithOpen();
     void startWithOldOpen();
     void execute();
@@ -267,49 +267,40 @@ void tst_QProcess::simpleStart()
     QCOMPARE(qvariant_cast<QProcess::ProcessState>(spy.at(2).at(0)), QProcess::NotRunning);
 }
 
-void tst_QProcess::setupChildProcess()
+static const char messageFromChildProcess[] = "Message from the child process";
+static void childProcessModifier(int fd)
 {
-    /* This test exists because in Qt 5.15, the Unix version of QProcess has
-     * some code that depends on whether it's an actual QProcess or a
-     * derived class */
-    static const char setupChildMessage[] = "Called from setupChildProcess()";
-    class DerivedProcessClass : public QProcess {
-    public:
-        int fd;
-        DerivedProcessClass(int fd) : fd(fd)
-        {
-        }
-
-    protected:
-        void setupChildProcess() override
-        {
-            QT_WRITE(fd, setupChildMessage, sizeof(setupChildMessage) - 1);
-            QT_CLOSE(fd);
-        }
-    };
+    QT_WRITE(fd, messageFromChildProcess, sizeof(messageFromChildProcess) - 1);
+    QT_CLOSE(fd);
+}
 
-    int pipes[2] = { -1 , -1 };
+void tst_QProcess::setChildProcessModifier()
+{
 #ifdef Q_OS_UNIX
+    int pipes[2] = { -1 , -1 };
     QVERIFY(qt_safe_pipe(pipes) == 0);
-#endif
 
-    DerivedProcessClass process(pipes[1]);
+    QProcess process;
+    process.setChildProcessModifier([pipes]() {
+        ::childProcessModifier(pipes[1]);
+    });
     process.start("testProcessNormal/testProcessNormal");
     if (process.state() != QProcess::Starting)
         QCOMPARE(process.state(), QProcess::Running);
     QVERIFY2(process.waitForStarted(5000), qPrintable(process.errorString()));
 
-#ifdef Q_OS_UNIX
-    char buf[sizeof setupChildMessage] = {};
+    char buf[sizeof messageFromChildProcess] = {};
     qt_safe_close(pipes[1]);
-    QCOMPARE(qt_safe_read(pipes[0], buf, sizeof(buf)), qint64(sizeof(setupChildMessage) - 1));
-    QCOMPARE(buf, setupChildMessage);
+    QCOMPARE(qt_safe_read(pipes[0], buf, sizeof(buf)), qint64(sizeof(messageFromChildProcess)) - 1);
+    QCOMPARE(buf, messageFromChildProcess);
     qt_safe_close(pipes[0]);
-#endif
 
     QVERIFY2(process.waitForFinished(5000), qPrintable(process.errorString()));
     QCOMPARE(process.exitStatus(), QProcess::NormalExit);
     QCOMPARE(process.exitCode(), 0);
+#else
+    QSKIP("Unix-only test");
+#endif
 }
 
 void tst_QProcess::startWithOpen()