Some improvements in tests #1:

tests/unit/bootstrap.php

@@ -21,7 +21,6 @@
  */
 function jexit($message = 0)
 {
-	return;
 }
 
 define('_JEXEC', 1);

tests/unit/stubs/discover1/challenger.php

@@ -1 +1 @@
-<?php /* @codingStandardsIgnoreFile Empty file */ ?>
+<?php /* @codingStandardsIgnoreFile Empty file */

tests/unit/suites/libraries/joomla/database/stubs/nosqldriver.php

@@ -84,7 +84,6 @@ public function connected()
 	 */
 	public function disconnect()
 	{
-		return;
 	}
 
 	/**

tests/unit/suites/libraries/joomla/form/JFormTest.php

@@ -1467,15 +1467,15 @@ public function testLoad()
 			'Line:' . __LINE__ . ' The show_title in the params group has been replaced by show_abstract.'
 		);
 
-		$originalform = new JFormInspector('form1');
-		$originalform->load(JFormDataHelper::$loadDocument);
-		$originalset = $originalform->getXml()->xpath('/form/fields/field');
+		$originalForm = new JFormInspector('form1');
+		$originalForm->load(JFormDataHelper::$loadDocument);
+		$originalSet = $originalForm->getXml()->xpath('/form/fields/field');
 		$set = $form->getXml()->xpath('/form/fields/field');
 
-		for ($i = 0; $i < count($originalset); $i++)
+		for ($i = 0, $iMax = count($originalSet); $i < $iMax; ++$i)
 		{
 			$this->assertThat(
-				(string) ($originalset[$i]->attributes()->name) == (string) ($set[$i]->attributes()->name),
+				(string) $originalSet[$i]->attributes()->name === (string) $set[$i]->attributes()->name,
 				$this->isTrue(),
 				'Line:' . __LINE__ . ' Replace should leave fields in the original order.'
 			);

tests/unit/suites/libraries/joomla/log/loggers/JLogLoggerCallbackTest.php

@@ -56,7 +56,6 @@ public function testConstructor02()
 		// Create a callback function (since php 5.3)
 		$callback = function ($entry)
 		{
-			return;
 		};
 
 		// Setup the basic configuration.

tests/unit/suites/libraries/joomla/session/JSessionTest.php

@@ -81,7 +81,7 @@ protected function tearDown()
 	 *
 	 * @return array
 	 */
-	Public function casesGetInstance()
+	public function casesGetInstance()
 	{
 		return array(
 			'first_instance' => array(

tests/unit/suites/libraries/legacy/error/JErrorTest.php

@@ -165,6 +165,5 @@ public function testSetErrorHandling()
 	 */
 	public function callbackHandler()
 	{
-		return;
 	}
 }

tests/unit/suites/libraries/legacy/model/stubs/name.php

@@ -26,6 +26,5 @@ class NokeywordInName extends JModelLegacy
 	 */
 	public function __construct()
 	{
-		return;
 	}
 }