minor changes. cannot assume hostname unchanged

src/php/ext/grpc/channel.c

@@ -86,7 +86,8 @@ zend_object_value create_wrapped_grpc_channel(zend_class_entry *class_type
   return retval;
 }
 
-void php_grpc_read_args_array(zval *args_array, grpc_channel_args *args TSRMLS_DC) {
+void php_grpc_read_args_array(zval *args_array,
+                              grpc_channel_args *args TSRMLS_DC) {
   HashTable *array_hash;
   HashPosition array_pointer;
   int args_index;

src/php/tests/unit_tests/CallTest.php

@@ -101,7 +101,6 @@ public function testAddSingleAndMultiValueMetadata()
 
     public function testGetPeer()
     {
-        $this->assertStringStartsWith('localhost:', $this->call->getPeer());
         $this->assertTrue(is_string($this->call->getPeer()));
     }
 

src/php/tests/unit_tests/ChannelTest.php

@@ -91,7 +91,7 @@ public function testGetTarget()
         $this->channel = new Grpc\Channel('localhost:8888',
              ['credentials' => Grpc\ChannelCredentials::createInsecure()]);
         $target = $this->channel->getTarget();
-        $this->assertSame('localhost:8888', $target);
+        $this->assertTrue(is_string($target));
     }
 
     public function testWatchConnectivityState()

src/php/tests/unit_tests/ServerTest.php

@@ -83,7 +83,7 @@ public function testRequestCall()
         $this->assertObjectHasAttribute('method', $c);
         $this->assertSame('dummy_method', $c->method);
         $this->assertObjectHasAttribute('host', $c);
-        $this->assertSame('localhost:8888', $c->host);
+        $this->assertTrue(is_string($c->host));
         $this->assertObjectHasAttribute('absolute_deadline', $c);
         $this->assertObjectHasAttribute('metadata', $c);
 
@@ -100,55 +100,7 @@ private function createSslObj()
 
         return $server_credentials;
     }
-/*
-    //TODO(thinkerou): make cases of addHttp2Port right
-    public function testAddHttp2Port()
-    {
-        $this->server = new Grpc\Server();
-        $port = $this->server->addHttp2Port('127.0.0.1:8080');
-        $this->assertEquals(8080, $port);
-    }
-
-    public function testAddHttp2Port1()
-    {
-        $this->server = new Grpc\Server([]);
-        $port = $this->server->addHttp2Port('127.0.0.1:8080');
-        $this->assertEquals(8080, $port);
-    }
-
-    public function testAddHttp2Port2()
-    {
-        $this->server = new Grpc\Server(['ip' => '127.0.0.1',
-                                          'port' => '8888', ]);
-        $port = $this->server->addHttp2Port('127.0.0.1:8080');
-        $this->assertEquals(8080, $port);
-    }
 
-    public function testAddSecureHttp2Port()
-    {
-        $this->server = new Grpc\Server();
-        $cred = $this->createSslObj();
-        $port = $this->server->addSecureHttp2Port('127.0.0.1:8080', $cred);
-        $this->assertEquals(8080, $port);
-    }
-
-    public function testAddSecureHttp2Port1()
-    {
-        $this->server = new Grpc\Server([]);
-        $cred = $this->createSslObj();
-        $port = $this->server->addSecureHttp2Port('127.0.0.1:8080', $cred);
-        $this->assertEquals(8080, $port);
-    }
-
-    public function testAddSecureHttp2Port2()
-    {
-        $this->server = new Grpc\Server(['ip' => '127.0.0.1',
-                                         'port' => '8888', ]);
-        $cred = $this->createSslObj();
-        $port = $this->server->addSecureHttp2Port('127.0.0.1:8080', $cred);
-        $this->assertEquals(8080, $port);
-    }
-*/
     /**
      * @expectedException InvalidArgumentException
      */
@@ -158,16 +110,6 @@ public function testInvalidConstructor()
         $this->assertNull($this->server);
     }
 
-    /**
-     * @expectedException InvalidArgumentException
-     */
-/*    public function testInvalidConstructor2()
-    {
-        //TODO(thinkerou): it crash when key is long on php7
-        $this->server = new Grpc\server(['0.0.0.0:0']);
-        $this->assertNull($this->server);
-    }
-*/
     /**
      * @expectedException InvalidArgumentException
      */