Merge pull request #54 from clue-labs/php

First class support for PHP7 and HHVM

Author: cboden

.travis.yml

@@ -6,17 +6,11 @@ php:
   - 5.6
   - 7
   - hhvm
-  - hhvm-nightly
 
-matrix:
-  allow_failures:
-    - php: 7
-    - php: hhvm
-    - php: hhvm-nightly
-  fast_finish: true
+sudo: false
 
-before_script:
-  - composer install --dev --prefer-source
+install:
+  - composer install --prefer-source --no-interaction
 
 script:
   - phpunit --coverage-text

src/SecureConnector.php

@@ -4,6 +4,7 @@
 
 use React\EventLoop\LoopInterface;
 use React\Stream\Stream;
+use React\Promise;
 
 class SecureConnector implements ConnectorInterface
 {
@@ -18,6 +19,10 @@ public function __construct(ConnectorInterface $connector, LoopInterface $loop)
 
     public function create($host, $port)
     {
+        if (!function_exists('stream_socket_enable_crypto')) {
+            return Promise\reject(new \BadMethodCallException('Encryption not supported on your platform (HHVM < 3.8?)'));
+        }
+
         return $this->connector->create($host, $port)->then(function (Stream $stream) use ($host) {
             // (unencrypted) TCP/IP connection succeeded
 

tests/IntegrationTest.php

@@ -42,6 +42,10 @@ public function gettingStuffFromGoogleShouldWork()
     /** @test */
     public function gettingEncryptedStuffFromGoogleShouldWork()
     {
+        if (defined('HHVM_VERSION')) {
+            $this->markTestSkipped('Not supported on HHVM');
+        }
+
         $loop = new StreamSelectLoop();
 
         $factory = new Factory();