Merge pull request #3858 from enisoc/php-close

PHP: Fix property reference in BaseStub::close()

src/php/lib/Grpc/BaseStub.php

@@ -123,7 +123,7 @@ class BaseStub {
    * Close the communication channel associated with this stub
    */
   public function close() {
-    $channel->close();
+    $this->channel->close();
   }
 
   /**

src/php/tests/generated_code/AbstractGeneratedCodeTest.php

@@ -32,7 +32,7 @@
  *
  */
 require_once realpath(dirname(__FILE__) . '/../../vendor/autoload.php');
-require 'math.php';
+require_once dirname(__FILE__) . '/math.php';
 abstract class AbstractGeneratedCodeTest extends PHPUnit_Framework_TestCase {
   /* These tests require that a server exporting the math service must be
    * running on $GRPC_TEST_HOST */

src/php/tests/generated_code/GeneratedCodeTest.php

@@ -31,11 +31,15 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  */
-require 'AbstractGeneratedCodeTest.php';
+require_once dirname(__FILE__) . '/AbstractGeneratedCodeTest.php';
 
 class GeneratedCodeTest extends AbstractGeneratedCodeTest {
   public static function setUpBeforeClass() {
     self::$client = new math\MathClient(
         getenv('GRPC_TEST_HOST'), []);
   }
+
+  public static function tearDownAfterClass() {
+    self::$client->close();
+  }
 }

src/php/tests/generated_code/GeneratedCodeWithCallbackTest.php

@@ -31,7 +31,7 @@
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  *
  */
-require 'AbstractGeneratedCodeTest.php';
+require_once dirname(__FILE__) . '/AbstractGeneratedCodeTest.php';
 
 class GeneratedCodeWithCallbackTest extends AbstractGeneratedCodeTest {
   public static function setUpBeforeClass() {
@@ -44,4 +44,8 @@ class GeneratedCodeWithCallbackTest extends AbstractGeneratedCodeTest {
                                      return $a_copy;
                                    }]);
   }
+
+  public static function tearDownAfterClass() {
+    self::$client->close();
+  }
 }