Port must be an int, and other small improvements

Author: snapshotpl

src/Uri.php

@@ -111,16 +111,18 @@ class Uri implements UriInterface
      */
     public function __construct($uri = '')
     {
+        if ('' === $uri) {
+            return;
+        }
+
         if (! is_string($uri)) {
             throw new InvalidArgumentException(sprintf(
                 'URI passed to constructor must be a string; received "%s"',
                 (is_object($uri) ? get_class($uri) : gettype($uri))
             ));
         }
 
-        if ('' !== $uri) {
-            $this->parseUri($uri);
-        }
+        $this->parseUri($uri);
     }
 
     /**
@@ -282,7 +284,7 @@ public function withUserInfo($user, $password = null)
         }
         if (null !== $password && ! is_string($password)) {
             throw new InvalidArgumentException(sprintf(
-                '%s expects a string password argument; received %s',
+                '%s expects a string or null password argument; received %s',
                 __METHOD__,
                 (is_object($password) ? get_class($password) : gettype($password))
             ));
@@ -333,17 +335,13 @@ public function withHost($host)
      */
     public function withPort($port)
     {
-        if (! is_numeric($port) && $port !== null) {
+        if (! is_int($port) && $port !== null) {
             throw new InvalidArgumentException(sprintf(
-                'Invalid port "%s" specified; must be an integer, an integer string, or null',
+                'Invalid port "%s" specified; must be an integer, or null',
                 (is_object($port) ? get_class($port) : gettype($port))
             ));
         }
 
-        if ($port !== null) {
-            $port = (int) $port;
-        }
-
         if ($port === $this->port) {
             // Do nothing if no change was made.
             return $this;
@@ -559,7 +557,7 @@ private function filterScheme($scheme)
             return '';
         }
 
-        if (! array_key_exists($scheme, $this->allowedSchemes)) {
+        if (! isset($this->allowedSchemes[$scheme])) {
             throw new InvalidArgumentException(sprintf(
                 'Unsupported scheme "%s"; must be any empty string or in the set (%s)',
                 $scheme,
@@ -655,7 +653,7 @@ private function filterQuery($query)
     private function splitQueryValue($value)
     {
         $data = explode('=', $value, 2);
-        if (1 === count($data)) {
+        if (! isset($data[1])) {
             $data[] = null;
         }
         return $data;

test/UriTest.php

@@ -137,7 +137,6 @@ public function validPorts()
         return [
             'null'         => [ null ],
             'int'          => [ 3000 ],
-            'string'       => [ "3000" ],
         ];
     }
 
@@ -159,22 +158,23 @@ public function testWithPortReturnsNewInstanceWithProvidedPort($port)
     public function testWithPortReturnsSameInstanceWithProvidedPortIsSameAsBefore()
     {
         $uri = new Uri('https://user:[email protected]:3001/foo?bar=baz#quz');
-        $new = $uri->withPort('3001');
+        $new = $uri->withPort(3001);
         $this->assertSame($uri, $new);
-        $this->assertEquals('3001', $new->getPort());
+        $this->assertEquals(3001, $new->getPort());
     }
 
     public function invalidPorts()
     {
         return [
-            'true'      => [ true ],
-            'false'     => [ false ],
-            'string'    => [ 'string' ],
-            'array'     => [ [ 3000 ] ],
-            'object'    => [ (object) [ 3000 ] ],
-            'zero'      => [ 0 ],
-            'too-small' => [ -1 ],
-            'too-big'   => [ 65536 ],
+            'true'       => [ true ],
+            'false'      => [ false ],
+            'string'     => [ 'string' ],
+            'string-int' => [ '3000' ],
+            'array'      => [ [ 3000 ] ],
+            'object'     => [ (object) [ 3000 ] ],
+            'zero'       => [ 0 ],
+            'too-small'  => [ -1 ],
+            'too-big'    => [ 65536 ],
         ];
     }