Merge branch 'hotfix/5'

Close #5

Author: weierophinney

test/ServerRequestTest.php

@@ -101,7 +101,19 @@ public function testRemovingAttributeReturnsCloneWithoutAttribute($request)
         $this->assertNull($new->getAttribute('foo', null));
     }
 
-    public function testUsesProvidedConstructorArguments()
+    public function provideMethods()
+    {
+        return [
+            'post' => ['POST', 'POST'],
+            'get'  => ['GET', 'GET'],
+            'null' => [null, 'GET'],
+        ];
+    }
+
+    /**
+     * @dataProvider provideMethods
+     */
+    public function testUsesProvidedConstructorArguments($parameterMethod, $methodReturned)
     {
         $server = [
             'foo' => 'bar',
@@ -115,7 +127,6 @@ public function testUsesProvidedConstructorArguments()
         ];
 
         $uri = new Uri('http://example.com');
-        $method = 'POST';
         $headers = [
             'host' => ['example.com'],
         ];
@@ -124,7 +135,7 @@ public function testUsesProvidedConstructorArguments()
             $server,
             $files,
             $uri,
-            $method,
+            $parameterMethod,
             'php://memory',
             $headers
         );
@@ -133,7 +144,7 @@ public function testUsesProvidedConstructorArguments()
         $this->assertEquals($files, $request->getUploadedFiles());
 
         $this->assertSame($uri, $request->getUri());
-        $this->assertEquals($method, $request->getMethod());
+        $this->assertEquals($methodReturned, $request->getMethod());
         $this->assertEquals($headers, $request->getHeaders());
 
         $body = $request->getBody();