X-Git-Url: https://git.cyclocoop.org/?a=blobdiff_plain;f=tests%2Fphpunit%2Fincludes%2FFauxResponseTest.php;h=4a974ba2db6225ab20843caf5a0b484ac8abd75a;hb=2bcbc311b49af248b9f4882ae5da787a8b91a8a5;hp=977c22bc83c94e15732dc25892114973e9fc029f;hpb=8fe0cf970401f2796ca39165a2d0a311b551c47a;p=lhc%2Fweb%2Fwiklou.git

diff --git a/tests/phpunit/includes/FauxResponseTest.php b/tests/phpunit/includes/FauxResponseTest.php
index 977c22bc83..4a974ba2db 100644
--- a/tests/phpunit/includes/FauxResponseTest.php
+++ b/tests/phpunit/includes/FauxResponseTest.php
@@ -23,52 +23,96 @@
  */
 
 class FauxResponseTest extends MediaWikiTestCase {
-	var $response;
+	/** @var FauxResponse */
+	protected $response;
 
 	protected function setUp() {
 		parent::setUp();
 		$this->response = new FauxResponse;
 	}
 
-	function testCookie() {
+	/**
+	 * @covers FauxResponse::getcookie
+	 * @covers FauxResponse::setcookie
+	 */
+	public function testCookie() {
 		$this->assertEquals( null, $this->response->getcookie( 'key' ), 'Non-existing cookie' );
 		$this->response->setcookie( 'key', 'val' );
 		$this->assertEquals( 'val', $this->response->getcookie( 'key' ), 'Existing cookie' );
 	}
 
-	function testHeader() {
+	/**
+	 * @covers FauxResponse::getheader
+	 * @covers FauxResponse::header
+	 */
+	public function testHeader() {
 		$this->assertEquals( null, $this->response->getheader( 'Location' ), 'Non-existing header' );
 
 		$this->response->header( 'Location: http://localhost/' );
-		$this->assertEquals( 'http://localhost/', $this->response->getheader( 'Location' ), 'Set header' );
+		$this->assertEquals(
+			'http://localhost/',
+			$this->response->getheader( 'Location' ),
+			'Set header'
+		);
 
 		$this->response->header( 'Location: http://127.0.0.1/' );
-		$this->assertEquals( 'http://127.0.0.1/', $this->response->getheader( 'Location' ), 'Same header' );
+		$this->assertEquals(
+			'http://127.0.0.1/',
+			$this->response->getheader( 'Location' ),
+			'Same header'
+		);
 
 		$this->response->header( 'Location: http://127.0.0.2/', false );
-		$this->assertEquals( 'http://127.0.0.1/', $this->response->getheader( 'Location' ), 'Same header with override disabled' );
+		$this->assertEquals(
+			'http://127.0.0.1/',
+			$this->response->getheader( 'Location' ),
+			'Same header with override disabled'
+		);
 
 		$this->response->header( 'Location: http://localhost/' );
-		$this->assertEquals( 'http://localhost/', $this->response->getheader( 'LOCATION' ), 'Get header case insensitive' );
+		$this->assertEquals(
+			'http://localhost/',
+			$this->response->getheader( 'LOCATION' ),
+			'Get header case insensitive'
+		);
 	}
 
-	function testResponseCode() {
+	/**
+	 * @covers FauxResponse::getStatusCode
+	 */
+	public function testResponseCode() {
 		$this->response->header( 'HTTP/1.1 200' );
 		$this->assertEquals( 200, $this->response->getStatusCode(), 'Header with no message' );
 
 		$this->response->header( 'HTTP/1.x 201' );
-		$this->assertEquals( 201, $this->response->getStatusCode(), 'Header with no message and protocol 1.x' );
+		$this->assertEquals(
+			201,
+			$this->response->getStatusCode(),
+			'Header with no message and protocol 1.x'
+		);
 
 		$this->response->header( 'HTTP/1.1 202 OK' );
 		$this->assertEquals( 202, $this->response->getStatusCode(), 'Normal header' );
 
 		$this->response->header( 'HTTP/1.x 203 OK' );
-		$this->assertEquals( 203, $this->response->getStatusCode(), 'Normal header with no message and protocol 1.x' );
+		$this->assertEquals(
+			203,
+			$this->response->getStatusCode(),
+			'Normal header with no message and protocol 1.x'
+		);
 
 		$this->response->header( 'HTTP/1.x 204 OK', false, 205 );
-		$this->assertEquals( 205, $this->response->getStatusCode(), 'Third parameter overrides the HTTP/... header' );
+		$this->assertEquals(
+			205,
+			$this->response->getStatusCode(),
+			'Third parameter overrides the HTTP/... header'
+		);
 
 		$this->response->header( 'Location: http://localhost/', false, 206 );
-		$this->assertEquals( 206, $this->response->getStatusCode(), 'Third parameter with another header' );
+		$this->assertEquals(
+			206,
+			$this->response->getStatusCode(),
+			'Third parameter with another header'
+		);
 	}
 }