SemanticMediaWiki / SemanticFormsSelect

tests/phpunit/Unit/OutputTest.php

Spacing +6 added lines, -6 removed lines

@@ -24,24 +24,24 @@
 	}
 
 	protected function tearDown(): void {
-		unset( $this->data );
+		unset($this->data);
 		parent::tearDown();
 	}
 
 	public function testCanConstruct() {
-		$this->assertInstanceOf( '\SFS\Output', new Output() );
+		$this->assertInstanceOf('\SFS\Output', new Output());
 	}
 
 	public function testAddToHeadItem() {
-		$ret = Output::addToHeadItem( $this->data );
+		$ret = Output::addToHeadItem($this->data);
 
-		$this->assertArrayHasKey( 'Foo', $ret );
-		$this->assertArrayHasKey( 'Spam', $ret );
+		$this->assertArrayHasKey('Foo', $ret);
+		$this->assertArrayHasKey('Spam', $ret);
 	}
 
 	public function testCommitToParserOutput() {
 		global $wgOut;
-		$expected_result = '[' . json_encode( $this->data ) . ']';
+		$expected_result = '['.json_encode($this->data).']';
 		Output::commitToParserOutput();
 		$this->assertEquals(
 			$expected_result, $wgOut->getJsConfigVars()['sf_select']

tests/phpunit/Unit/ApiSemanticFormsSelectTest.php

Indentation +2 added lines, -2 removed lines

@@ -25,7 +25,7 @@ 
 	protected function setUp(): void {
 		parent::setUp();
 		$parameters = [ 'action' => 'sformsselect', 'approach' => 'smw',
-		                     'query'  => 'abc', 'sep' => ',' ];
+							 'query'  => 'abc', 'sep' => ',' ];
 
 		$this->ApiMain = new ApiMain(
 			$this->newRequestContext( $parameters ), true
@@ -69,7 +69,7 @@ 
 
 	public function testGetParamDescription() {
 		$tdata = [ 'approach' => 'The actual approach: function or smw',
-		                'query'    => 'The query of the former' ];
+						'query'    => 'The query of the former' ];
 		$this->assertEquals( $this->ApiSFS->getParamDescription(), $tdata );
 	}
 

Spacing +16 added lines, -16 removed lines

@@ -24,11 +24,11 @@ 
 
 	protected function setUp(): void {
 		parent::setUp();
-		$parameters = [ 'action' => 'sformsselect', 'approach' => 'smw',
-		                     'query'  => 'abc', 'sep' => ',' ];
+		$parameters = ['action' => 'sformsselect', 'approach' => 'smw',
+		                     'query'  => 'abc', 'sep' => ','];
 
 		$this->ApiMain = new ApiMain(
-			$this->newRequestContext( $parameters ), true
+			$this->newRequestContext($parameters), true
 		);
 		$this->ApiSFS = new ApiSemanticFormsSelect(
 			$this->ApiMain, 'sformsselect'
@@ -36,15 +36,15 @@ 
 	}
 
 	protected function tearDown(): void {
-		unset( $this->ApiSFS );
-		unset( $this->ApiMain );
+		unset($this->ApiSFS);
+		unset($this->ApiMain);
 		parent::tearDown();
 	}
 
 
 	public function testCanConstruct() {
 
-		$apiMain = new ApiMain( $this->newRequestContext( [] ), true );
+		$apiMain = new ApiMain($this->newRequestContext([]), true);
 
 		$instance = new ApiSemanticFormsSelect(
 			$apiMain, 'sformsselect'
@@ -63,30 +63,30 @@ 
 	}
 
 	public function testGetDescription() {
-		$tdata = [ 'API for providing SemanticFormsSelect values' ];
-		$this->assertEquals( $this->ApiSFS->getDescription(), $tdata );
+		$tdata = ['API for providing SemanticFormsSelect values'];
+		$this->assertEquals($this->ApiSFS->getDescription(), $tdata);
 	}
 
 	public function testGetParamDescription() {
-		$tdata = [ 'approach' => 'The actual approach: function or smw',
-		                'query'    => 'The query of the former' ];
-		$this->assertEquals( $this->ApiSFS->getParamDescription(), $tdata );
+		$tdata = ['approach' => 'The actual approach: function or smw',
+		                'query'    => 'The query of the former'];
+		$this->assertEquals($this->ApiSFS->getParamDescription(), $tdata);
 	}
 
 	public function testGetVersion() {
 		$tdata = 'SFS\ApiSemanticFormsSelect: 1.1';
-		$this->assertEquals( $this->ApiSFS->getVersion(), $tdata );
+		$this->assertEquals($this->ApiSFS->getVersion(), $tdata);
 	}
 
 
-	private function newRequestContext( $request = [] ) {
+	private function newRequestContext($request = []) {
 
 		$context = new RequestContext();
 
-		if ( $request instanceof WebRequest ) {
-			$context->setRequest( $request );
+		if ($request instanceof WebRequest) {
+			$context->setRequest($request);
 		} else {
-			$context->setRequest( new FauxRequest( $request, true ) );
+			$context->setRequest(new FauxRequest($request, true));
 		}
 
 		return $context;