Skip to content

Commit

Permalink
Fixed ConsoleIntegrationTestCase failing to load user bootstrap files
Browse files Browse the repository at this point in the history
  • Loading branch information
jeremyharris committed May 3, 2018
1 parent 836da74 commit 1cc085d
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 12 deletions.
2 changes: 1 addition & 1 deletion src/TestSuite/ConsoleIntegrationTestCase.php
Original file line number Diff line number Diff line change
Expand Up @@ -253,7 +253,7 @@ protected function _makeRunner()
if ($this->_useCommandRunner) {
$applicationClassName = Configure::read('App.namespace') . '\Application';

return new CommandRunner(new $applicationClassName([CONFIG]));
return new CommandRunner(new $applicationClassName(CONFIG));
}

return new LegacyCommandRunner();
Expand Down
11 changes: 0 additions & 11 deletions tests/test_app/TestApp/Application.php
Original file line number Diff line number Diff line change
Expand Up @@ -19,17 +19,6 @@

class Application extends BaseApplication
{
/**
* Bootstrap hook.
*
* Nerfed as this is for IntegrationTestCase testing.
*
* @return void
*/
public function bootstrap()
{
// Do nothing.
}

public function middleware($middleware)
{
Expand Down
2 changes: 2 additions & 0 deletions tests/test_app/config/bootstrap.php
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
<?php
// Do nothing

0 comments on commit 1cc085d

Please sign in to comment.