search

brianlmoon / net_gearman

A PHP interface for Gearman
84 stars 46 forks source link

012-client-runSet.phpt #2

Closed till closed 9 years ago

till commented 14 years ago

The 012-client-runSet.phpt test fails for me always.

And I can't figure out why: ** ERROR: process timed out **

Try:

cd tests/
pear run-tests 012-client-runSet.phpt
brianlmoon commented 14 years ago

Well, if you are using GearmanManager for this test, there is a sleep() in the worker. That may be the timeout issue. However, the test was using the wrong result data. Worker will convert a scalar into an array with a single item named result. I don't really like that behavior, but it would be a major BC break now.

till commented 14 years ago

Yeah, you realize we can break BC all we want until 1.0? ;-)

I just merged your change:

till@till-laptop:~/net_gearman/tests$ phpunit --colors --coverage-html ./report AllTests.php 
PHPUnit 3.4.15 by Sebastian Bergmann.

............F

Time: 01:01, Memory: 15.00Mb

There was 1 failure:

1) /usr/home/till/net_gearman/tests/012-client-runSet.phpt
--- Expected
+++ Actual
@@ @@
-int(1274)
+** ERROR: process timed out **

What do you mean by using "GearmanManager"?