Fixing tests and removing one that made no sense

src/Controller/Controller.php

@@ -325,7 +325,7 @@ class Controller implements EventListener {
  */
 	public function setRequest(Request $request) {
 		$this->request = $request;
-		$this->plugin = isset($request->params['plugin']) ? Inflector::camelize($request->params['plugin']) : null;
+		$this->plugin = isset($request->params['plugin']) ? $request->params['plugin'] : null;
 		$this->view = isset($request->params['action']) ? $request->params['action'] : null;
 
 		if (isset($request->params['pass'])) {

src/TestSuite/ControllerTestCase.php

@@ -286,9 +286,9 @@ abstract class ControllerTestCase extends TestCase {
 		if ($this->controller === null && $this->autoMock) {
 			$plugin = '';
 			if (!empty($request->params['plugin'])) {
-				$plugin = Inflector::camelize($request->params['plugin']) . '.';
+				$plugin = $request->params['plugin'] . '.';
 			}
-			$controllerName = Inflector::camelize($request->params['controller']);
+			$controllerName = $request->params['controller'];
 			if (!empty($request->params['prefix'])) {
 				$controllerName = Inflector::camelize($request->params['prefix']) . '/' . $controllerName;
 			}

src/View/Helper.php

@@ -319,7 +319,7 @@ class Helper implements EventListener {
 				//@codingStandardsIgnoreEnd
 			}
 			$segments = explode('/', ltrim($filepath, '/'));
-			$plugin = Inflector::camelize($segments[0]);
+			$plugin = $segments[0];
 			if (Plugin::loaded($plugin)) {
 				unset($segments[0]);
 				$pluginPath = Plugin::path($plugin) . 'webroot' . DS . implode(DS, $segments);

src/View/Helper/HtmlHelper.php

@@ -18,7 +18,6 @@ use Cake\Core\App;
 use Cake\Core\Configure;
 use Cake\Error;
 use Cake\Network\Response;
-use Cake\Utility\Inflector;
 use Cake\View\Helper;
 use Cake\View\Helper\StringTemplateTrait;
 use Cake\View\View;

tests/TestCase/TestSuite/ControllerTestCaseTest.php

@@ -52,6 +52,7 @@ class ControllerTestCaseTest extends TestCase {
 		DispatcherFactory::add('Routing');
 		DispatcherFactory::add('ControllerFactory');
 		Router::reload();
+		require CAKE . 'Config/routes.php';
 		TableRegistry::clear();
 	}
 
@@ -228,36 +229,12 @@ class ControllerTestCaseTest extends TestCase {
 	}
 
 /**
- * Tests using loaded routes during tests
- *
- * @return void
- */
-	public function testUseRoutes() {
-		Router::connect('/:controller/:action/*');
-		include APP . '/Config/routes.php';
-
-		$controller = $this->Case->generate('TestsApps');
-		$controller->components()->load('RequestHandler');
-		$result = $this->Case->testAction('/tests_apps/index.json', array('return' => 'contents'));
-		$result = json_decode($result, true);
-		$expected = array('cakephp' => 'cool');
-		$this->assertEquals($expected, $result);
-
-		include APP . '/Config/routes.php';
-		$result = $this->Case->testAction('/some_alias');
-		$this->assertEquals(5, $result);
-	}
-
-/**
  * Tests not using loaded routes during tests
  *
  * @expectedException \Cake\Controller\Error\MissingActionException
  * @return void
  */
 	public function testSkipRoutes() {
-		Router::connect('/:controller/:action/*');
-		include APP . 'Config/routes.php';
-
 		$this->Case->loadRoutes = false;
 		$this->Case->testAction('/tests_apps/missing_action.json', array('return' => 'view'));
 	}