From 02d2807b30321ed31d7e23a8a8e19f3ea821ab0e Mon Sep 17 00:00:00 2001 From: Ken Payson <kpayson@google.com> 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/ServerTest.php | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/php/tests/unit_tests/ServerTest.php') diff --git a/src/php/tests/unit_tests/ServerTest.php b/src/php/tests/unit_tests/ServerTest.php index f2346ab113..5f40202f18 100644 --- a/src/php/tests/unit_tests/ServerTest.php +++ b/src/php/tests/unit_tests/ServerTest.php @@ -67,9 +67,9 @@ class ServerTest extends PHPUnit_Framework_TestCase public function testRequestCall() { $this->server = new Grpc\Server(); - $port = $this->server->addHttp2Port('0.0.0.0:8888'); + $port = $this->server->addHttp2Port('0.0.0.0:0'); $this->server->start(); - $channel = new Grpc\Channel('localhost:8888', + $channel = new Grpc\Channel('localhost:' . $port, ['credentials' => Grpc\ChannelCredentials::createInsecure()]); $deadline = Grpc\Timeval::infFuture(); -- cgit v1.2.3