[ticket/12602] Cleanup tests

PHPBB3-12602
This commit is contained in:
Tristan Darricau 2014-06-02 16:21:25 +02:00
parent 85e8d1c2ef
commit c3ccef10d1
4 changed files with 9 additions and 50 deletions

View file

@ -7,7 +7,7 @@
* *
*/ */
require_once dirname(__FILE__) . '/tasks/simple.php'; require_once dirname(__FILE__) . '/tasks/simple_ready.php';
require_once dirname(__FILE__) . '/tasks/simple_not_ready.php'; require_once dirname(__FILE__) . '/tasks/simple_not_ready.php';
use Symfony\Component\Console\Application; use Symfony\Component\Console\Application;
use Symfony\Component\Console\Tester\CommandTester; use Symfony\Component\Console\Tester\CommandTester;
@ -41,13 +41,13 @@ class phpbb_console_command_cron_list_test extends phpbb_test_case
public function test_only_ready() public function test_only_ready()
{ {
$tasks = array( $tasks = array(
new phpbb_cron_task_simple(), new phpbb_cron_task_simple_ready(),
new phpbb_cron_task_simple() new phpbb_cron_task_simple_ready()
); );
$this->get_cron_manager($tasks); $this->get_cron_manager($tasks);
$command_tester = $this->get_command_tester(); $command_tester = $this->get_command_tester();
$command_tester->execute(array('command' => $this->command_name, '--no-ansi' => true)); $command_tester->execute(array('command' => $this->command_name, '--no-ansi' => true));
$this->assertContains('TASKS_READYcommand1command2', preg_replace('/\s+/', '', $command_tester->getDisplay())); $this->assertContains('TASKS_READY command1 command2', preg_replace('/\s+/', ' ', trim($command_tester->getDisplay())));
} }
public function test_only_not_ready() public function test_only_not_ready()
@ -59,21 +59,21 @@ class phpbb_console_command_cron_list_test extends phpbb_test_case
$this->get_cron_manager($tasks); $this->get_cron_manager($tasks);
$command_tester = $this->get_command_tester(); $command_tester = $this->get_command_tester();
$command_tester->execute(array('command' => $this->command_name, '--no-ansi' => true)); $command_tester->execute(array('command' => $this->command_name, '--no-ansi' => true));
$this->assertContains('TASKS_NOT_READYcommand1command2', preg_replace('/\s+/', '', $command_tester->getDisplay())); $this->assertContains('TASKS_NOT_READY command1 command2', preg_replace('/\s+/', ' ', trim($command_tester->getDisplay())));
} }
public function test_both_ready() public function test_both_ready()
{ {
$tasks = array( $tasks = array(
new phpbb_cron_task_simple(), new phpbb_cron_task_simple_ready(),
new phpbb_cron_task_simple(), new phpbb_cron_task_simple_ready(),
new phpbb_cron_task_simple_not_ready(), new phpbb_cron_task_simple_not_ready(),
new phpbb_cron_task_simple_not_ready() new phpbb_cron_task_simple_not_ready()
); );
$this->get_cron_manager($tasks); $this->get_cron_manager($tasks);
$command_tester = $this->get_command_tester(); $command_tester = $this->get_command_tester();
$command_tester->execute(array('command' => $this->command_name, '--no-ansi' => true)); $command_tester->execute(array('command' => $this->command_name, '--no-ansi' => true));
$this->assertSame('TASKS_READYcommand1command2TASKS_NOT_READYcommand3command4', preg_replace('/\s+/', '', $command_tester->getDisplay())); $this->assertSame('TASKS_READY command1 command2 TASKS_NOT_READY command3 command4', preg_replace('/\s+/', ' ', trim($command_tester->getDisplay())));
} }
public function get_cron_manager(array $tasks) public function get_cron_manager(array $tasks)

View file

@ -1,18 +0,0 @@
<?php
class phpbb_cron_task_core_simple_not_runnable extends \phpbb\cron\task\base
{
public function get_name()
{
return get_class($this);
}
public function run()
{
}
public function is_runnable()
{
return false;
}
}

View file

@ -1,12 +1,7 @@
<?php <?php
class phpbb_cron_task_core_simple_ready extends \phpbb\cron\task\base class phpbb_cron_task_simple_ready extends \phpbb\cron\task\base
{ {
public function get_name()
{
return get_class($this);
}
public function run() public function run()
{ {
} }

View file

@ -1,18 +0,0 @@
<?php
class phpbb_cron_task_core_simple_should_not_run extends \phpbb\cron\task\base
{
public function get_name()
{
return get_class($this);
}
public function run()
{
}
public function should_run()
{
return false;
}
}