mirror/php-flarum
1
0
Fork 0
mirror of https://github.com/flarum/core.git synced 2025-08-05 07:57:46 +02:00
Code Issues Releases Wiki Activity

Do not catch exceptions when testing Console commands (#3813)

Browse Source 
* Do not catch exceptions when testing Console commands

This allows to easily test a case where a command should throw using `$this->expectException()`.

* Rewrite ConsoleTest to use expectException

Now that the ConsoleApplication is set to not catch exceptions.

---------

Co-authored-by: IanM <16573496+imorland@users.noreply.github.com>
This commit is contained in:
Nicolas Peugnet
2023-06-21 09:14:12 +02:00
committed by GitHub
parent 7799c2fcd5
commit 4a966b830f
2 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
framework/core/tests/integration/extenders/ConsoleTest.php
View File

@@ -14,6 +14,7 @@ use Flarum\Extend;
use Flarum\Testing\integration\ConsoleTestCase;
use Illuminate\Console\Scheduling\Event;
use Symfony\Component\Console\Command\Command;
use Symfony\Component\Console\Exception\CommandNotFoundException;
class ConsoleTest extends ConsoleTestCase
{
@@ -26,7 +27,8 @@ class ConsoleTest extends ConsoleTestCase
'command' => 'customTestCommand'
];
$this->assertEquals('Command "customTestCommand" is not defined.', $this->runCommand($input));
$this->expectException(CommandNotFoundException::class);
$this->runCommand($input);
}
/**

1
php-packages/testing/src/integration/ConsoleTestCase.php
View File

@@ -23,6 +23,7 @@ abstract class ConsoleTestCase extends TestCase
if (is_null($this->console)) {
$this->console = new ConsoleApplication('Flarum', Application::VERSION);
$this->console->setAutoExit(false);
$this->console->setCatchExceptions(false);
foreach ($this->app()->getConsoleCommands() as $command) {
$this->console->add($command);