Update deps (#17)

- Change PHP constraint
- Update PHPUnit
- Delegate openssl check to Composer
- make `FileGetContents::getTlsStreamContextDefaults()` private

Regarding `FileGetContents::getTlsStreamContextDefaults()` the reason is it's more of an `init()` method in the first place and can easily be overridden in the constructor instead of overriding the method itself.

composer.json

@@ -14,12 +14,13 @@
     ],
 
     "require": {
-        "php": ">=5.3",
+        "php": "^5.3 || ^7.0",
+        "ext-openssl": "*",
         "composer/ca-bundle": "^1.0"
     },
     "require-dev": {
         "bamarni/composer-bin-plugin": "^1.1",
-        "phpunit/phpunit": "~4.0"
+        "phpunit/phpunit": "^4.0 || ^5.0 || ^6.0"
     },
 
     "autoload": {

src/Humbug/FileGetContents.php

@@ -12,21 +12,21 @@
 namespace Humbug;
 
 use Composer\CaBundle\CaBundle;
+use RuntimeException;
 
 /**
  * This is largely extracted from the Composer Installer where originally implemented.
  */
 class FileGetContents
 {
-    protected $options = array('http' => array());
-
     protected static $lastResponseHeaders;
 
     protected static $nextRequestHeaders;
 
+    protected $options = array('http' => array());
+
     public function __construct()
     {
-        $this->checkConfig();
         $options = $this->getTlsStreamContextDefaults(null);
         $this->options = array_replace_recursive($this->options, $options);
     }
@@ -93,15 +93,6 @@ public static function setHttpHeaders($context)
         return $context;
     }
 
-    protected function checkConfig()
-    {
-        if (!extension_loaded('openssl')) {
-            throw new \RuntimeException(
-                'The openssl extension is not loaded but is required for secure HTTPS connections'
-            );
-        }
-    }
-
     protected function getStreamContext($url)
     {
         $host = parse_url($url, PHP_URL_HOST);
@@ -113,6 +104,13 @@ protected function getStreamContext($url)
         return $this->getMergedStreamContext($url);
     }
 
+    /**
+     * @param string $cafile
+     *
+     * @return array
+     *
+     * @private since 1.1.0
+     */
     protected function getTlsStreamContextDefaults($cafile)
     {
         $ciphers = implode(':', array(
@@ -177,7 +175,7 @@ protected function getTlsStreamContextDefaults($cafile)
         } elseif ($cafile) {
             $options['ssl']['cafile'] = $cafile;
         } else {
-            throw new \RuntimeException('A valid cafile could not be located locally.');
+            throw new RuntimeException('A valid cafile could not be located locally.');
         }
 
         if (version_compare(PHP_VERSION, '5.4.13') >= 0) {
@@ -188,7 +186,7 @@ protected function getTlsStreamContextDefaults($cafile)
     }
 
     /**
-     * function copied from Composer\Util\StreamContextFactory::getContext
+     * Function copied from Composer\Util\StreamContextFactory::getContext
      *
      * This function is part of Composer.
      *
@@ -197,7 +195,7 @@ protected function getTlsStreamContextDefaults($cafile)
      *
      * @param string $url URL the context is to be used for
      *
-     * @throws \\RuntimeException if https proxy required and OpenSSL uninstalled
+     * @throws \RuntimeException If https proxy required and OpenSSL uninstalled
      *
      * @return resource Default context
      */
@@ -227,7 +225,7 @@ protected function getMergedStreamContext($url)
             $proxyURL = str_replace(array('http://', 'https://'), array('tcp://', 'ssl://'), $proxyURL);
 
             if (0 === strpos($proxyURL, 'ssl:') && !extension_loaded('openssl')) {
-                throw new \RuntimeException('You must enable the openssl extension to use a proxy over https');
+                throw new RuntimeException('You must enable the openssl extension to use a proxy over https');
             }
 
             $options['http'] = array(
@@ -278,7 +276,7 @@ public static function getSystemCaRootBundlePath()
      */
     protected static function validateCaFile($contents)
     {
-        // assume the CA is valid if php is vunerable to
+        // assume the CA is valid if php is vulnerable to
         // https://www.sektioneins.de/advisories/advisory-012013-php-openssl_x509_parse-memory-corruption-vulnerability.html
         if (!CaBundle::isOpensslParseSafe()) {
             return !empty($contents);

tests/Humbug/Test/FunctionTest.php

@@ -11,10 +11,16 @@
 
 namespace Humbug\Test;
 
+use PHPUnit\Framework\TestCase;
+
+if (!class_exists('\PHPUnit\Framework\TestCase', true)) {
+    class_alias('\PHPUnit_Framework_TestCase', 'TestCase');
+}
+
 /**
  * @coversNothing
  */
-class FunctionTest extends \PHPUnit_Framework_TestCase
+class FunctionTest extends TestCase
 {
     private static $result;