From 02d2807b30321ed31d7e23a8a8e19f3ea821ab0e Mon Sep 17 00:00:00 2001 From: Ken Payson Date: Thu, 1 Dec 2016 09:57:30 -0800 Subject: Fixes 2 issues: 1. In the State change test, 123->0 will hit a c-core GPR_ASSERT if debugging is enabled because 123 doesn't map to a state that can be printed. 2. In the Server test, 8888 is sometimes unavailable, so we just use a wildcard port. --- src/php/tests/unit_tests/ChannelTest.php | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/php/tests/unit_tests/ChannelTest.php') diff --git a/src/php/tests/unit_tests/ChannelTest.php b/src/php/tests/unit_tests/ChannelTest.php index 4b35b1a28c..fa33d38911 100644 --- a/src/php/tests/unit_tests/ChannelTest.php +++ b/src/php/tests/unit_tests/ChannelTest.php @@ -99,7 +99,7 @@ class ChannelTest extends PHPUnit_Framework_TestCase $this->channel = new Grpc\Channel('localhost:0', ['credentials' => Grpc\ChannelCredentials::createInsecure()]); $time = new Grpc\Timeval(1000); - $state = $this->channel->watchConnectivityState(123, $time); + $state = $this->channel->watchConnectivityState(1, $time); $this->assertTrue($state); unset($time); } -- cgit v1.2.3