Added some more tests with serverRequest (#9)

* Added some more tests with serverRequest

* style fix

* cs

Author: Nyholm

phpunit.xml.dist

@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 
 <phpunit backupGlobals="false"
-  backupStaticAttributes="false"
+  backupStaticAttributes="true"
   colors="true"
   convertErrorsToExceptions="true"
   convertNoticesToExceptions="true"

src/ServerRequestIntegrationTest.php

@@ -6,8 +6,6 @@
 use Psr\Http\Message\ServerRequestInterface;
 
 /**
- * TODO Write me.
- *
  * @author Tobias Nyholm <[email protected]>
  */
 abstract class ServerRequestIntegrationTest extends BaseTest
@@ -32,14 +30,72 @@ protected function setUp()
         $this->serverRequest = $this->createSubject();
     }
 
-    public function testNothing()
+    public function testGetServerParams()
     {
         if (isset($this->skippedTests[__FUNCTION__])) {
             $this->markTestSkipped($this->skippedTests[__FUNCTION__]);
+        }
+
+        $this->assertEquals($_SERVER, $this->serverRequest->getServerParams());
+    }
+
+    public function testGetCookieParams()
+    {
+        if (isset($this->skippedTests[__FUNCTION__])) {
+            $this->markTestSkipped($this->skippedTests[__FUNCTION__]);
+        }
+
+        $this->assertEquals($_COOKIE, $this->serverRequest->getCookieParams());
+    }
+
+    public function testWithCookieParams()
+    {
+        if (isset($this->skippedTests[__FUNCTION__])) {
+            $this->markTestSkipped($this->skippedTests[__FUNCTION__]);
+        }
+
+        $orgCookie = $_COOKIE;
+        $new = $this->serverRequest->withCookieParams(['foo' => 'bar']);
+
+        $this->assertEquals($orgCookie, $this->serverRequest->getCookieParams(), 'Super global $_COOKIE MUST NOT change.');
+        $this->assertNotEquals($orgCookie, $new->getCookieParams());
 
-            return;
+        $this->assertArrayHasKey('foo', $new->getCookieParams());
+    }
+
+    public function testGetQueryParams()
+    {
+        if (isset($this->skippedTests[__FUNCTION__])) {
+            $this->markTestSkipped($this->skippedTests[__FUNCTION__]);
+        }
+
+        // TODO write me
+    }
+
+    public function testGetUploadedFiles()
+    {
+        if (isset($this->skippedTests[__FUNCTION__])) {
+            $this->markTestSkipped($this->skippedTests[__FUNCTION__]);
+        }
+
+        // TODO write me
+    }
+
+    public function testGetParsedBody()
+    {
+        if (isset($this->skippedTests[__FUNCTION__])) {
+            $this->markTestSkipped($this->skippedTests[__FUNCTION__]);
+        }
+
+        // TODO write me
+    }
+
+    public function testGetAttributes()
+    {
+        if (isset($this->skippedTests[__FUNCTION__])) {
+            $this->markTestSkipped($this->skippedTests[__FUNCTION__]);
         }
 
-        $this->assertTrue(true);
+        // TODO write me
     }
 }

tests/Guzzle/ServerRequestTest.php

@@ -9,6 +9,6 @@ class ServerRequestTest extends ServerRequestIntegrationTest
 {
     public function createSubject()
     {
-        return new ServerRequest('GET', '/');
+        return new ServerRequest('GET', '/', [], null, '1.1', $_SERVER);
     }
 }

tests/Slim/ServerRequestTest.php

@@ -12,6 +12,6 @@ class ServerRequestTest extends ServerRequestIntegrationTest
 {
     public function createSubject()
     {
-        return new Request('GET', new Uri('http', 'foo.com'), new Headers([]), [], [], new Body(fopen('php://temp', 'r+')));
+        return new Request('GET', new Uri('http', 'foo.com'), new Headers([]), $_COOKIE, $_SERVER, new Body(fopen('php://temp', 'r+')));
     }
 }

tests/Zend/ServerRequestTest.php

@@ -9,6 +9,6 @@ class ServerRequestTest extends ServerRequestIntegrationTest
 {
     public function createSubject()
     {
-        return new ServerRequest();
+        return new ServerRequest($_SERVER);
     }
 }