From 5e92034ad878d8c0fe08df98cbcdfa3e27ea9676 Mon Sep 17 00:00:00 2001
From: Val Bancer <valerij.bancer@gmail.com>
Date: Tue, 18 Jul 2017 22:31:28 +0200
Subject: [PATCH] improved code style

---
 lib/Cake/TestSuite/CakeTestCase.php | 62 ++++++++++++++++++++++-------
 1 file changed, 48 insertions(+), 14 deletions(-)

diff --git a/lib/Cake/TestSuite/CakeTestCase.php b/lib/Cake/TestSuite/CakeTestCase.php
index c8d01bf19..24b2838f1 100644
--- a/lib/Cake/TestSuite/CakeTestCase.php
+++ b/lib/Cake/TestSuite/CakeTestCase.php
@@ -737,25 +737,14 @@ abstract class CakeTestCase extends PHPUnit_Framework_TestCase {
  *   to disable the call to the original class' clone constructor.
  * @param bool $callAutoload The seventh (optional) parameter can be used to
  *   disable __autoload() during the generation of the test double class.
- * @param bool $cloneArguments Not supported.
- * @param bool $callOriginalMethods Not supported.
- * @param string $proxyTarget Not supported.
  * @return object
- * @throws InvalidArgumentException When not supported parameters are set.
  * @deprecated Use `getMockBuilder()` or `createMock()` in new unit tests.
  * @see https://phpunit.de/manual/current/en/test-doubles.html
  */
-	public function getMock($originalClassName, $methods = array(),
+	protected function _buildMock($originalClassName, $methods = array(),
 			array $arguments = array(), $mockClassName = '',
 			$callOriginalConstructor = true, $callOriginalClone = true,
-			$callAutoload = true, $cloneArguments = false,
-			$callOriginalMethods = false, $proxyTarget = null) {
-		$phpUnitVersion = PHPUnit_Runner_Version::id();
-		if ($phpUnitVersion < '5.7.0') {
-			return parent::getMock($originalClassName, $methods, $arguments,
-					$mockClassName, $callOriginalConstructor, $callOriginalClone,
-					$callAutoload, $cloneArguments, $callOriginalMethods, $proxyTarget);
-		}
+			$callAutoload = true) {
 		$MockBuilder = $this->getMockBuilder($originalClassName);
 		if (!empty($methods)) {
 			$MockBuilder = $MockBuilder->setMethods($methods);
@@ -775,6 +764,50 @@ abstract class CakeTestCase extends PHPUnit_Framework_TestCase {
 		if ($callAutoload !== true) {
 			$MockBuilder = $MockBuilder->disableAutoload();
 		}
+		return $MockBuilder->getMock();
+	}
+
+/**
+ * Returns a mock object for the specified class.
+ *
+ * @param string $originalClassName The class name of the object to be mocked.
+ * @param array $methods By default, all methods of the given class are replaced
+ *   with a test double that just returns NULL unless a return value is configured
+ *   using will($this->returnValue()), for instance.
+ *   When the second (optional) parameter is provided, only the methods whose names
+ *   are in the array are replaced with a configurable test double. The behavior
+ *   of the other methods is not changed. Providing NULL as the parameter means
+ *   that no methods will be replaced.
+ * @param array $arguments The third (optional) parameter may hold a parameter
+ *   array that is passed to the original class' constructor (which is not replaced
+ *   with a dummy implementation by default).
+ * @param string $mockClassName The fourth (optional) parameter can be used to
+ *   specify a class name for the generated test double class.
+ * @param bool $callOriginalConstructor The fifth (optional) parameter can be
+ *   used to disable the call to the original class' constructor.
+ * @param bool $callOriginalClone The sixth (optional) parameter can be used
+ *   to disable the call to the original class' clone constructor.
+ * @param bool $callAutoload The seventh (optional) parameter can be used to
+ *   disable __autoload() during the generation of the test double class.
+ * @param bool $cloneArguments Not supported.
+ * @param bool $callOriginalMethods Not supported.
+ * @param string $proxyTarget Not supported.
+ * @return object
+ * @throws InvalidArgumentException When not supported parameters are set.
+ * @deprecated Use `getMockBuilder()` or `createMock()` in new unit tests.
+ * @see https://phpunit.de/manual/current/en/test-doubles.html
+ */
+	public function getMock($originalClassName, $methods = array(),
+			array $arguments = array(), $mockClassName = '',
+			$callOriginalConstructor = true, $callOriginalClone = true,
+			$callAutoload = true, $cloneArguments = false,
+			$callOriginalMethods = false, $proxyTarget = null) {
+		$phpUnitVersion = PHPUnit_Runner_Version::id();
+		if (version_compare($phpUnitVersion, '5.7.0', '<')) {
+			return parent::getMock($originalClassName, $methods, $arguments,
+					$mockClassName, $callOriginalConstructor, $callOriginalClone,
+					$callAutoload, $cloneArguments, $callOriginalMethods, $proxyTarget);
+		}
 		if ($cloneArguments) {
 			throw new InvalidArgumentException('$cloneArguments parameter is not supported');
 		}
@@ -784,7 +817,8 @@ abstract class CakeTestCase extends PHPUnit_Framework_TestCase {
 		if ($proxyTarget !== null) {
 			throw new InvalidArgumentException('$proxyTarget parameter is not supported');
 		}
-		return $MockBuilder->getMock();
+		return $this->_buildMock($originalClassName, $methods, $arguments,
+				$mockClassName, $callOriginalConstructor, $callOriginalClone, $callAutoload);
 	}
 
 /**