SemanticMediaWiki / SemanticFormsSelect

tests/phpunit/Unit/SemanticFormsSelectInputTest.php

Spacing +10 added lines, -10 removed lines

@@ -25,17 +25,17 @@ 
 		$isMandatory = false;
 		$isDisabled = false;
 
-		$otherArgs = [ 'template' => 'Foo', 'field' => '',
-		                    'function' => 'Bar', 'is_list' => true ];
+		$otherArgs = ['template' => 'Foo', 'field' => '',
+		                    'function' => 'Bar', 'is_list' => true];
 
-		$parserOutput = $this->getMockBuilder( '\ParserOutput' )
+		$parserOutput = $this->getMockBuilder('\ParserOutput')
 			->disableOriginalConstructor()->getMock();
 
-		$parser = $this->getMockBuilder( '\Parser' )
+		$parser = $this->getMockBuilder('\Parser')
 			->disableOriginalConstructor()->getMock();
 
-		$parser->expects( $this->any() )->method( 'getOutput' )->will(
-				$this->returnValue( $parserOutput )
+		$parser->expects($this->any())->method('getOutput')->will(
+				$this->returnValue($parserOutput)
 			);
 		$this->SFSInput = new SemanticFormsSelectInput(
 			$value, $inputName, $isMandatory, $isDisabled, $otherArgs
@@ -43,7 +43,7 @@ 
 	}
 
 	protected function tearDown(): void {
-		unset( $this->SelectField );
+		unset($this->SelectField);
 		parent::tearDown();
 	}
 
@@ -70,14 +70,14 @@ 
 
 	public function testGetParameters() {
 
-		$this->assertIsArray( $this->SFSInput->getParameters() );
+		$this->assertIsArray($this->SFSInput->getParameters());
 	}
 
 
 	public function testGetResourceModuleNames() {
-		$rsmn = [ 'ext.sf_select.scriptselect' ];
+		$rsmn = ['ext.sf_select.scriptselect'];
 
-		$this->assertEquals( $rsmn, $this->SFSInput->getResourceModuleNames() );
+		$this->assertEquals($rsmn, $this->SFSInput->getResourceModuleNames());
 	}
 
 }

tests/phpunit/Unit/SelectFieldTest.php

Spacing +50 added lines, -50 removed lines

@@ -23,16 +23,16 @@ 
 class SelectFieldTest extends \PHPUnit_Framework_TestCase {
 	private $selectField;
 	
-	private $other_args_query_parametrized = [ 'query' => '((Category:Building Complex))((Part Of Site::@@@@));?Display Title;format~list;sort~Display Title;sep~,;link~none;headers~hide;limit~500' ];
+	private $other_args_query_parametrized = ['query' => '((Category:Building Complex))((Part Of Site::@@@@));?Display Title;format~list;sort~Display Title;sep~,;link~none;headers~hide;limit~500'];
 	private $expected_result_parametrized_setQuery = "[[Category:Building Complex]][[Part Of Site::@@@@]];?Display Title;format=list;sort=Display Title;sep=,;link=none;headers=hide;limit=500";
-	private $other_args_query_unparametrized = [ 'query' => '((Category:Building Complex));?Display Title;format~list;sort~Display Title;sep~,;link~none;headers~hide;limit~500' ];
-	private $other_args_function_parametrized = [ 'function' => '((Category:Building Complex))((Part Of Site::@@@@));?Display Title;format~list;sort~Display Title;sep~,;link~none;headers~hide;limit~500' ];
+	private $other_args_query_unparametrized = ['query' => '((Category:Building Complex));?Display Title;format~list;sort~Display Title;sep~,;link~none;headers~hide;limit~500'];
+	private $other_args_function_parametrized = ['function' => '((Category:Building Complex))((Part Of Site::@@@@));?Display Title;format~list;sort~Display Title;sep~,;link~none;headers~hide;limit~500'];
 	private $expected_result_parametrized_seFunction = '{{#[[Category:Building Complex]][[Part Of Site::@@@@]];?Display Title;format=list;sort=Display Title;sep=,;link=none;headers=hide;limit=500}}';
-	private $other_args_function_unparametrized = [ 'function' => 'ask:((Category:Building Complex));?Display Title;format~list;sort~Display Title;sep~@@;link~none;headers~hide;limit~500' ];
+	private $other_args_function_unparametrized = ['function' => 'ask:((Category:Building Complex));?Display Title;format~list;sort~Display Title;sep~@@;link~none;headers~hide;limit~500'];
 
 	public function testCanConstruct() {
 
-		$this->assertInstanceOf( '\SFS\SelectField', $this->selectField );
+		$this->assertInstanceOf('\SFS\SelectField', $this->selectField);
 	}
 
 	public function testProcessParameters_Query() {
@@ -41,7 +41,7 @@ 
 			"", $this->other_args_query_parametrized
 		);
 		$this->assertTrue(
-			array_key_exists( "query", $this->other_args_query_parametrized )
+			array_key_exists("query", $this->other_args_query_parametrized)
 		);
 	}
 
@@ -57,7 +57,7 @@ 
 
 	public function testParametrized_setQuery() {
 
-		$this->selectField->setQuery( $this->other_args_query_parametrized );
+		$this->selectField->setQuery($this->other_args_query_parametrized);
 
 		$this->assertEquals(
 			$this->expected_result_parametrized_setQuery,
@@ -76,16 +76,16 @@ 
 			$was_replaced
 		);
 
-		$this->assertTrue( count( $was_remove[0] ) == 0 );
-		$this->assertTrue( count( $was_replaced[0] ) > 0 );
+		$this->assertTrue(count($was_remove[0]) == 0);
+		$this->assertTrue(count($was_replaced[0]) > 0);
 	}
 
 	public function testUnparametrized_setQuery() {
 
-		$this->selectField->setQuery( $this->other_args_query_unparametrized );
+		$this->selectField->setQuery($this->other_args_query_unparametrized);
 
-		$this->assertTrue( $this->selectField->getValues() !== null );
-		$this->assertTrue( $this->selectField->hasStaticValues() );
+		$this->assertTrue($this->selectField->getValues() !== null);
+		$this->assertTrue($this->selectField->hasStaticValues());
 	}
 
 	public function testParametrized_setFunction() {
@@ -107,26 +107,26 @@ 
 			$this->other_args_function_unparametrized
 		);
 
-		$this->assertTrue( $this->selectField->hasStaticValues() );
+		$this->assertTrue($this->selectField->hasStaticValues());
 	}
 
 	public function testSetSelectIsMultiple_keyExistTrue() {
-		$other_args = [ "part_of_multiple" => "bla bla bla" ];
-		$this->selectField->setSelectIsMultiple( $other_args );
-		$this->assertTrue( $this->selectField->getData()["selectismultiple"] );
+		$other_args = ["part_of_multiple" => "bla bla bla"];
+		$this->selectField->setSelectIsMultiple($other_args);
+		$this->assertTrue($this->selectField->getData()["selectismultiple"]);
 	}
 
 	public function testSetSelectIsMultiple_keyExistFalse() {
 
-		$other_args = [ "Not_part_of_multiple" => "blas blas blas" ];
-		$this->selectField->setSelectIsMultiple( $other_args );
-		$this->assertFalse( $this->selectField->getData()["selectismultiple"] );
+		$other_args = ["Not_part_of_multiple" => "blas blas blas"];
+		$this->selectField->setSelectIsMultiple($other_args);
+		$this->assertFalse($this->selectField->getData()["selectismultiple"]);
 	}
 
 	public function testSetSelectTemplate_correctData() {
 		$input_name = "{{#[[Category:Building Complex]][[Part Of Site::@@@@]]";
 		$result = "{{#";
-		$this->selectField->setSelectTemplate( $input_name );
+		$this->selectField->setSelectTemplate($input_name);
 
 		$this->assertEquals(
 			$this->selectField->getData()['selecttemplate'], $result
@@ -136,7 +136,7 @@ 
 	public function testSetSelectTemplate_wrongData() {
 		$input_name = "Category:Building Complex";
 		$result = "";
-		$this->selectField->setSelectTemplate( $input_name );
+		$this->selectField->setSelectTemplate($input_name);
 
 		$this->assertEquals(
 			$this->selectField->getData()['selecttemplate'], $result
@@ -147,7 +147,7 @@ 
 		$input_name = "{{#[[Category:Building Complex]][[Part Of Site::@@@@]]";
 		$result = "Part Of Site::@@@@]";
 
-		$this->selectField->setSelectField( $input_name );
+		$this->selectField->setSelectField($input_name);
 
 		$this->assertEquals(
 			$this->selectField->getData()['selectfield'], $result
@@ -157,7 +157,7 @@ 
 	public function testSetSelectField_wrongData() {
 		$input_name = "Category:Building Complex";
 		$result = "";
-		$this->selectField->setSelectField( $input_name );
+		$this->selectField->setSelectField($input_name);
 
 		$this->assertNotEquals(
 			$this->selectField->getData()['selectfield'], $result
@@ -166,10 +166,10 @@ 
 
 	public function testSetValueTemplate_containsMselectTemplate() {
 		$input_name = "{{#[[Category:Building Complex]][[Part Of Site::@@@@]]";
-		$other_args = [ "sametemplate" => "test values" ];
+		$other_args = ["sametemplate" => "test values"];
 		$result = "{{#";
-		$this->selectField->setSelectTemplate( $input_name );
-		$this->selectField->setValueTemplate( $other_args );
+		$this->selectField->setSelectTemplate($input_name);
+		$this->selectField->setValueTemplate($other_args);
 
 		$this->assertEquals(
 			$this->selectField->getData()["valuetemplate"], $result
@@ -178,9 +178,9 @@ 
 
 	public function testSetValueTemplate_containsOtherArgsTemplate() {
 
-		$other_args = [ "template" => "test values" ];
+		$other_args = ["template" => "test values"];
 
-		$this->selectField->setValueTemplate( $other_args );
+		$this->selectField->setValueTemplate($other_args);
 
 		$this->assertEquals(
 			$this->selectField->getData()["valuetemplate"],
@@ -189,9 +189,9 @@ 
 	}
 
 	public function testSetValueField() {
-		$other_args = [ "field" => "test values Field" ];
+		$other_args = ["field" => "test values Field"];
 
-		$this->selectField->setValueField( $other_args );
+		$this->selectField->setValueField($other_args);
 
 		$this->assertEquals(
 			$this->selectField->getData()["valuefield"], $other_args["field"]
@@ -199,35 +199,35 @@ 
 	}
 
 	public function testSetSelectRemove_keyExistTrue() {
-		$other_args = [ 'rmdiv' => "Test data" ];
-		$this->selectField->setSelectRemove( $other_args );
-		$this->assertTrue( $this->selectField->getData()["selectrm"] );
+		$other_args = ['rmdiv' => "Test data"];
+		$this->selectField->setSelectRemove($other_args);
+		$this->assertTrue($this->selectField->getData()["selectrm"]);
 	}
 
 	public function testSetSelectRemove_keyExistFalse() {
 
-		$other_args = [ "no_rmdiv" => "test data" ];
-		$this->selectField->setSelectRemove( $other_args );
-		$this->assertFalse( $this->selectField->getData()["selectrm"] );
+		$other_args = ["no_rmdiv" => "test data"];
+		$this->selectField->setSelectRemove($other_args);
+		$this->assertFalse($this->selectField->getData()["selectrm"]);
 	}
 
 	public function testSetLabel_keyExistTrue() {
-		$other_args = [ 'label' => "Test data" ];
-		$this->selectField->setLabel( $other_args );
-		$this->assertTrue( $this->selectField->getData()["label"] );
+		$other_args = ['label' => "Test data"];
+		$this->selectField->setLabel($other_args);
+		$this->assertTrue($this->selectField->getData()["label"]);
 	}
 
 	public function testSetLabel_keyExistFalse() {
 
-		$other_args = [ "no_label" => "test data" ];
-		$this->selectField->setLabel( $other_args );
-		$this->assertArrayHasKey( "label", $this->selectField->getData() );
-		$this->assertFalse( $this->selectField->getData()["label"] );
+		$other_args = ["no_label" => "test data"];
+		$this->selectField->setLabel($other_args);
+		$this->assertArrayHasKey("label", $this->selectField->getData());
+		$this->assertFalse($this->selectField->getData()["label"]);
 	}
 
 	public function testSetDelimiter_keyExistTrue() {
-		$other_args = [ "delimiter" => ":" ];
-		$this->selectField->setDelimiter( $other_args );
+		$other_args = ["delimiter" => ":"];
+		$this->selectField->setDelimiter($other_args);
 		$this->assertEquals(
 			$this->selectField->getDelimiter(), $other_args["delimiter"]
 		);
@@ -238,8 +238,8 @@ 
 
 	public function testSetWgPageFormsListSeparator_keyExistTrue() {
 
-		$g_args = [ "delimiter" => ";" ];
-		$this->selectField->setDelimiter( $g_args );
+		$g_args = ["delimiter" => ";"];
+		$this->selectField->setDelimiter($g_args);
 		$this->assertEquals(
 			$this->selectField->getDelimiter(), $g_args["delimiter"]
 		);
@@ -252,15 +252,15 @@ 
 		parent::setUp();
 		$parser = MediaWikiServices::getInstance()->getParser();
 		$parser->setOutputType(Parser::OT_HTML);
-		$parser->setTitle( Title::newFromText( 'NO TITLE' ) );
+		$parser->setTitle(Title::newFromText('NO TITLE'));
 		$parser->mOptions = new ParserOptions();
 		$parser->resetOutput();
 		$parser->clearState();
-		$this->selectField = new SelectField( $parser );
+		$this->selectField = new SelectField($parser);
 	}
 
 	protected function tearDown(): void {
-		unset( $this->selectField );
+		unset($this->selectField);
 		parent::tearDown();
 	}
 

tests/phpunit/Unit/ApiSemanticFormsSelectRequestProcessorTest.php

Indentation +2 added lines, -2 removed lines

@@ -65,7 +65,7 @@ 
 	) {
 
 		$parameters = [ 'approach' => 'smw', 'query' => 'foo, baa, gaah',
-		                     'sep'      => ',' ];
+							 'sep'      => ',' ];
 
 		$this->assertIsObject(
 			$this->ApiSFSRP->getJsonDecodedResultValuesForRequestParameters(
@@ -88,7 +88,7 @@ 
 	public function testSetDebugFlag_doProcessQueryFor() {
 		$this->ApiSFSRP->setDebugFlag( true );
 		$parameters = [ 'approach' => 'smw', 'query' => 'my Query,query2',
-		                     'sep'      => ',' ];
+							 'sep'      => ',' ];
 
 		$this->assertIsObject(
 			$this->ApiSFSRP->getJsonDecodedResultValuesForRequestParameters(

Spacing +20 added lines, -20 removed lines

@@ -24,13 +24,13 @@ 
 
 	protected function setUp(): void {
 		parent::setUp();
-		$parser = $this->getMockBuilder( '\Parser' )
+		$parser = $this->getMockBuilder('\Parser')
 			->disableOriginalConstructor()->getMock();
-		$this->ApiSFSRP = new ApiSemanticFormsSelectRequestProcessor( $parser );
+		$this->ApiSFSRP = new ApiSemanticFormsSelectRequestProcessor($parser);
 	}
 
 	protected function tearDown(): void {
-		unset( $this->ApiSFSRP );
+		unset($this->ApiSFSRP);
 		parent::tearDown();
 	}
 
@@ -44,7 +44,7 @@ 
 
 		$parameters = [];
 
-		$this->expectException( 'InvalidArgumentException' );
+		$this->expectException('InvalidArgumentException');
 		$this->ApiSFSRP->getJsonDecodedResultValuesForRequestParameters(
 			$parameters
 		);
@@ -52,7 +52,7 @@ 
 
 	public function testJsonResultValuesFromRequestParameters() {
 
-		$parameters = [ 'query' => 'foo', 'sep' => ',' ];
+		$parameters = ['query' => 'foo', 'sep' => ','];
 
 		$this->assertIsObject(
 			$this->ApiSFSRP->getJsonDecodedResultValuesForRequestParameters(
@@ -64,8 +64,8 @@ 
 	public function testJsonResultValuesFromRequestParameters_doProcessQueryFor(
 	) {
 
-		$parameters = [ 'approach' => 'smw', 'query' => 'foo, baa, gaah',
-		                     'sep'      => ',' ];
+		$parameters = ['approach' => 'smw', 'query' => 'foo, baa, gaah',
+		                     'sep'      => ','];
 
 		$this->assertIsObject(
 			$this->ApiSFSRP->getJsonDecodedResultValuesForRequestParameters(
@@ -75,8 +75,8 @@ 
 	}
 
 	public function testSetDebugFlag() {
-		$this->ApiSFSRP->setDebugFlag( true );
-		$parameters = [ 'query' => 'foo , function', 'sep' => ',' ];
+		$this->ApiSFSRP->setDebugFlag(true);
+		$parameters = ['query' => 'foo , function', 'sep' => ','];
 
 		$this->assertIsObject(
 			$this->ApiSFSRP->getJsonDecodedResultValuesForRequestParameters(
@@ -86,9 +86,9 @@ 
 	}
 
 	public function testSetDebugFlag_doProcessQueryFor() {
-		$this->ApiSFSRP->setDebugFlag( true );
-		$parameters = [ 'approach' => 'smw', 'query' => 'my Query,query2',
-		                     'sep'      => ',' ];
+		$this->ApiSFSRP->setDebugFlag(true);
+		$parameters = ['approach' => 'smw', 'query' => 'my Query,query2',
+		                     'sep'      => ','];
 
 		$this->assertIsObject(
 			$this->ApiSFSRP->getJsonDecodedResultValuesForRequestParameters(
@@ -100,11 +100,11 @@ 
 	public function testGetFormattedValuesFrom() {
 		$sep = ",";
 		$values = "my Query,query2";
-		$result = [ "", "my Query", "query2" ];
+		$result = ["", "my Query", "query2"];
 		$formattedValues = $this->invokeMethod(
-			$this->ApiSFSRP, 'getFormattedValuesFrom', [ $sep, $values ]
+			$this->ApiSFSRP, 'getFormattedValuesFrom', [$sep, $values]
 		);
-		$this->assertEquals( $result, $formattedValues );
+		$this->assertEquals($result, $formattedValues);
 	}
 
 	/**
@@ -116,14 +116,14 @@ 
 	 *
 	 * @return mixed Method return.
 	 */
-	public function invokeMethod( &$object, $methodName,
+	public function invokeMethod(&$object, $methodName,
 		array $parameters = []
 	) {
-		$reflection = new \ReflectionClass( get_class( $object ) );
-		$method = $reflection->getMethod( $methodName );
-		$method->setAccessible( true );
+		$reflection = new \ReflectionClass(get_class($object));
+		$method = $reflection->getMethod($methodName);
+		$method->setAccessible(true);
 
-		return $method->invokeArgs( $object, $parameters );
+		return $method->invokeArgs($object, $parameters);
 	}