Make ChildProcess UnitTest manual since it malfunctions

This commit is contained in:
Vinnie Falco
2013-09-12 10:51:07 -07:00
parent 02acf7d6d0
commit bfdda3212a

View File

@@ -61,8 +61,6 @@ String ChildProcess::readAllProcessOutput()
class ChildProcessTests : public UnitTest class ChildProcessTests : public UnitTest
{ {
public: public:
ChildProcessTests() : UnitTest ("ChildProcess", "beast") {}
void runTest() void runTest()
{ {
beginTestCase ("Child Processes"); beginTestCase ("Child Processes");
@@ -76,10 +74,25 @@ public:
expect (p.start ("ls /")); expect (p.start ("ls /"));
#endif #endif
if (! p.waitForProcessToFinish (10 * 1000))
{
if (p.kill ())
p.waitForProcessToFinish (-1);
}
//String output (p.readAllProcessOutput()); //String output (p.readAllProcessOutput());
//expect (output.isNotEmpty()); //expect (output.isNotEmpty());
#endif #endif
} }
// VFALCO NOTE I had to disable this test because it was leaving
// behind a zombie process and making other unit tests fail.
// It doesnt happen with a debugger attached, or if the
// unit test is run individually.
//
ChildProcessTests() : UnitTest ("ChildProcess", "beast", runManual)
{
}
}; };
static ChildProcessTests childProcessTests; static ChildProcessTests childProcessTests;