diff options
Diffstat (limited to 'tests/auto/corelib/io')
-rw-r--r-- | tests/auto/corelib/io/qprocess/tst_qprocess.cpp | 47 |
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() |