ScheduleTest::can_add_tasks() - Code Metrics - Inspection of "add Schedule::timezone(), remove need for Schedule..." - zenstruck/schedule-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Passed
Push — master ( 68072d...6506ec )

by Kevin
created 2020-01-13 02:30 UTC
ScheduleTest::can_add_tasks() A

↳ Parent: ScheduleTest
Complexity

Conditions	1
Paths	1
Size

Total Lines	16
Code Lines	11
Duplication

Lines	0
Ratio	0 %
Importance

Changes	1
Bugs	0	Features	0
Metric	Value
eloc	11
c	1
b	0
f	0
dl	0
loc	16
rs	9.9
cc	1
nc	1
nop	0
<?php

namespace Zenstruck\ScheduleBundle\Tests;

use PHPUnit\Framework\TestCase;
filter:
    dependency_paths: ["lib/*"]
use Symfony\Component\Lock\LockFactory;
use Symfony\Component\Lock\Store\FlockStore;
use Symfony\Component\Mime\Email;
use Symfony\Component\Process\Process;
use Symfony\Contracts\HttpClient\HttpClientInterface;
use Zenstruck\ScheduleBundle\Event\AfterScheduleEvent;
use Zenstruck\ScheduleBundle\Event\BeforeScheduleEvent;
use Zenstruck\ScheduleBundle\Schedule;
use Zenstruck\ScheduleBundle\Schedule\Exception\SkipSchedule;
use Zenstruck\ScheduleBundle\Schedule\Extension;
use Zenstruck\ScheduleBundle\Schedule\Task;
use Zenstruck\ScheduleBundle\Schedule\Task\CallbackTask;
use Zenstruck\ScheduleBundle\Schedule\Task\CommandTask;

/**
 * @author Kevin Bond <[email protected]>
 */
class ScheduleTest extends TestCase
{
    /**
     * @test
     */
    public function can_add_tasks()
    {
        $schedule = new Schedule();

        $schedule->add(new CallbackTask(function () {}))->description('task1');
        $schedule->addCallback(function () {})->description('task2');
        $schedule->addProcess('php -v')->description('task3');
        $schedule->addProcess(new Process(['php -v']))->description('task4');
        $schedule->addCommand('my:command')->description('task5');

        $this->assertCount(5, $schedule->all());
        $this->assertSame(['task1', 'task2', 'task3', 'task4', 'task5'], \array_map(
            function (Task $task) {
                return $task->getDescription();
            },
            $schedule->all()
        ));
    }

    /**
     * @test
     */
    public function can_add_compound_tasks()
    {
        $schedule = new Schedule();

        $schedule->addCommand('my:command')->description('task1')->tuesdays();
        $schedule->addCompound()
            ->addCommand('another:command', [], 'task2')
            ->addCallback(function () {}, 'task3')
            ->addProcess('php -v', 'task4')
            ->addProcess(new Process(['php -v']), 'task5')
            ->add((new CommandTask('yet:another:command'))
                ->description('task6')
                ->sundays()
                ->timezone('America/Los_Angeles')
            )
            ->timezone('UTC')
            ->mondays()
            ->onSingleServer()
        ;

        $this->assertCount(6, $schedule->all());
        $this->assertSame('task1', $schedule->all()[0]->getDescription());
        $this->assertSame('* * * * 2', $schedule->all()[0]->getExpression());
        $this->assertNull($schedule->all()[0]->getTimezone());
        $this->assertCount(0, $schedule->all()[0]->getExtensions());
        $this->assertSame('task2', $schedule->all()[1]->getDescription());
        $this->assertSame('* * * * 1', $schedule->all()[1]->getExpression());
        $this->assertSame('UTC', $schedule->all()[1]->getTimezone()->getName());
        $this->assertCount(1, $schedule->all()[1]->getExtensions());
        $this->assertSame('task3', $schedule->all()[2]->getDescription());
        $this->assertSame('* * * * 1', $schedule->all()[2]->getExpression());
        $this->assertSame('UTC', $schedule->all()[2]->getTimezone()->getName());
        $this->assertCount(1, $schedule->all()[2]->getExtensions());
        $this->assertSame('task4', $schedule->all()[3]->getDescription());
        $this->assertSame('* * * * 1', $schedule->all()[3]->getExpression());
        $this->assertSame('UTC', $schedule->all()[3]->getTimezone()->getName());
        $this->assertCount(1, $schedule->all()[3]->getExtensions());
        $this->assertSame('task5', $schedule->all()[4]->getDescription());
        $this->assertSame('* * * * 1', $schedule->all()[4]->getExpression());
        $this->assertSame('UTC', $schedule->all()[4]->getTimezone()->getName());
        $this->assertCount(1, $schedule->all()[4]->getExtensions());
        $this->assertSame('task6', $schedule->all()[5]->getDescription());
        $this->assertSame('* * * * 1', $schedule->all()[5]->getExpression());
        $this->assertSame('UTC', $schedule->all()[5]->getTimezone()->getName());
        $this->assertCount(1, $schedule->all()[5]->getExtensions());
    }

    /**
     * @test
     */
    public function can_get_due_tasks()
    {
        $schedule = new Schedule();

        $schedule->addCallback(function () {})->description('task1');
        $notDueTask = $schedule->addProcess('php -v')->description('task2')->sundays();

        if ('Sun' === \date('D')) {
            $notDueTask->mondays();
        }

        $this->assertCount(2, $schedule->all());
        $this->assertCount(1, $schedule->due());
        $this->assertCount(1, $schedule->due(), 'Due tasks are cached');
        $this->assertSame('task1', $schedule->due()[0]->getDescription());
    }

    /**
     * @test
     */
    public function has_unique_id_based_on_tasks()
    {
        $schedule1 = new Schedule();
        $schedule1->addCommand('my:command');
        $schedule2 = new Schedule();
        $schedule2->addCommand('my:command');
        $schedule3 = new Schedule();
        $schedule3->addCommand('another:command');
        $schedule4 = new Schedule();
        $schedule4->addCommand('my:command');
        $schedule4->addCommand('another:command');

        $this->assertSame((new Schedule())->getId(), (new Schedule())->getId());
        $this->assertSame($schedule1->getId(), $schedule2->getId());
        $this->assertNotSame($schedule2->getId(), $schedule3->getId());
        $this->assertNotSame($schedule2->getId(), $schedule4->getId());
    }

    /**
     * @test
     */
    public function false_when_filter_skips_schedule()
    {
        $schedule = new Schedule();

        $schedule->when('boolean value', false);

        $this->expectException(SkipSchedule::class);
        $this->expectExceptionMessage('boolean value');

        $schedule->getExtensions()[0]->filterSchedule(new BeforeScheduleEvent($schedule));
                                       filterSchedule(new BeforeScheduleEvent($schedule));
    }

    /**
     * @test
     */
    public function callback_returning_false_when_filter_skips_schedule()
    {
        $schedule = new Schedule();

        $schedule->when('callback value', function () { return false; });

        $this->expectException(SkipSchedule::class);
        $this->expectExceptionMessage('callback value');

        $schedule->getExtensions()[0]->filterSchedule(new BeforeScheduleEvent($schedule));
    }

    /**
     * @test
     */
    public function true_when_filter_allows_schedule_to_run()
    {
        $schedule = new Schedule();

        $schedule->when('boolean value', true);

        $schedule->getExtensions()[0]->filterSchedule(new BeforeScheduleEvent($schedule));

        $this->assertTrue(true);
    }

    /**
     * @test
     */
    public function callback_returning_true_when_filter_allows_schedule_to_run()
    {
        $schedule = new Schedule();

        $schedule->when('callback value', function () { return true; });

        $schedule->getExtensions()[0]->filterSchedule(new BeforeScheduleEvent($schedule));

        $this->assertTrue(true);
    }

    /**
     * @test
     */
    public function true_skip_filter_skips_schedule()
    {
        $schedule = new Schedule();

        $schedule->skip('boolean value', true);

        $this->expectException(SkipSchedule::class);
        $this->expectExceptionMessage('boolean value');

        $schedule->getExtensions()[0]->filterSchedule(new BeforeScheduleEvent($schedule));
    }

    /**
     * @test
     */
    public function callback_returning_true_skip_filter_skips_schedule()
    {
        $schedule = new Schedule();

        $schedule->skip('callback value', function () { return true; });

        $this->expectException(SkipSchedule::class);
        $this->expectExceptionMessage('callback value');

        $schedule->getExtensions()[0]->filterSchedule(new BeforeScheduleEvent($schedule));
    }

    /**
     * @test
     */
    public function false_skip_filter_allows_schedule_to_run()
    {
        $schedule = new Schedule();

        $schedule->skip('boolean value', false);

        $schedule->getExtensions()[0]->filterSchedule(new BeforeScheduleEvent($schedule));

        $this->assertTrue(true);
    }

    /**
     * @test
     */
    public function callback_returning_false_skip_filter_allows_schedule_to_run()
    {
        $schedule = new Schedule();

        $schedule->skip('callback value', function () { return false; });

        $schedule->getExtensions()[0]->filterSchedule(new BeforeScheduleEvent($schedule));

        $this->assertTrue(true);
    }

    /**
     * @test
     */
    public function can_add_callback_extensions()
    {
        $schedule = new Schedule();
        $calls = [];

        $schedule->filter(function () use (&$calls) { $calls[] = 'filter'; });
        $schedule->before(function () use (&$calls) { $calls[] = 'before'; });
        $schedule->after(function () use (&$calls) { $calls[] = 'after'; });
        $schedule->then(function () use (&$calls) { $calls[] = 'then'; });
        $schedule->onSuccess(function () use (&$calls) { $calls[] = 'onSuccess'; });
        $schedule->onFailure(function () use (&$calls) { $calls[] = 'onFailure'; });

        $schedule->getExtensions()[0]->filterSchedule($event = new BeforeScheduleEvent($schedule));
        $schedule->getExtensions()[1]->beforeSchedule(new BeforeScheduleEvent($schedule));
                                       beforeSchedule(new BeforeScheduleEvent($schedule));
        $schedule->getExtensions()[2]->afterSchedule(new AfterScheduleEvent($event, []));
                                       afterSchedule(new AfterScheduleEvent($event, []));
        $schedule->getExtensions()[3]->afterSchedule(new AfterScheduleEvent($event, []));
        $schedule->getExtensions()[4]->onScheduleSuccess(new AfterScheduleEvent($event, []));
                                       onScheduleSuccess(new AfterScheduleEvent($event, []));
        $schedule->getExtensions()[5]->onScheduleFailure(new AfterScheduleEvent($event, []));
                                       onScheduleFailure(new AfterScheduleEvent($event, []));

        $this->assertSame([
            'filter',
            'before',
            'after',
            'then',
            'onSuccess',
            'onFailure',
        ], $calls);
    }

    /**
     * @test
     */
    public function can_add_single_server_extension()
    {
        $schedule1 = new Schedule();
        $schedule1->addCommand('my:command');
        $schedule1->onSingleServer();

        $schedule2 = new Schedule();
        $schedule2->addCommand('my:command');
        $schedule2->onSingleServer();

        $lockFactory = new LockFactory(new FlockStore());

        $schedule1->getExtensions()[0]->aquireScheduleLock($lockFactory, $schedule1, \time());
                                        aquireScheduleLock($lockFactory, $schedule1, \time());

        $this->expectException(SkipSchedule::class);
        $this->expectExceptionMessage('Schedule running on another server.');

        $schedule2->getExtensions()[0]->aquireScheduleLock($lockFactory, $schedule2, \time());
    }

    /**
     * @test
     */
    public function can_add_ping_extensions()
    {
        $schedule = new Schedule();

        $schedule->pingBefore('http://before.com');
        $schedule->pingAfter('http://after.com', 'POST');
        $schedule->thenPing('http://then.com');
        $schedule->pingOnSuccess('http://success.com');
        $schedule->pingOnFailure('http://failure.com');

        $client = $this->createMock(HttpClientInterface::class);
        $client->expects($this->exactly(5))->method('request')->withConsecutive(
            [$this->equalTo('GET'), $this->equalTo('http://before.com'), $this->isType('array')],
            [$this->equalTo('POST'), $this->equalTo('http://after.com'), $this->isType('array')],
            [$this->equalTo('GET'), $this->equalTo('http://then.com'), $this->isType('array')],
            [$this->equalTo('GET'), $this->equalTo('http://success.com'), $this->isType('array')],
            [$this->equalTo('GET'), $this->equalTo('http://failure.com'), $this->isType('array')]
        );

        $schedule->getExtensions()[0]->setHttpClient($client)->beforeSchedule($event = new BeforeScheduleEvent($schedule));
                                       setHttpClient($client)->beforeSchedule($event = new BeforeScheduleEvent($schedule));
        $schedule->getExtensions()[1]->setHttpClient($client)->afterSchedule(new AfterScheduleEvent($event, []));
        $schedule->getExtensions()[2]->setHttpClient($client)->afterSchedule(new AfterScheduleEvent($event, []));
        $schedule->getExtensions()[3]->setHttpClient($client)->onScheduleSuccess(new AfterScheduleEvent($event, []));
        $schedule->getExtensions()[4]->setHttpClient($client)->onScheduleFailure(new AfterScheduleEvent($event, []));
    }

    /**
     * @test
     */
    public function can_add_email_on_failure_extension()
    {
        $schedule = new Schedule();
        $schedule->emailOnFailure('[email protected]', 'my subject', function (Email $email) {
            $email->cc('[email protected]');
        });

        $this->assertTrue($schedule->getExtensions()[0]->isHook(Extension::SCHEDULE_FAILURE));
        $this->assertTrue($schedule->getExtensions()[0]->/** @scrutinizer ignore-call */ isHook(Extension::SCHEDULE_FAILURE));
        $this->assertSame('[email protected]', $schedule->getExtensions()[0]->getEmail()->getTo()[0]->toString());
        $this->assertSame('[email protected]', $schedule->getExtensions()[0]->/** @scrutinizer ignore-call */ getEmail()->getTo()[0]->toString());
        $this->assertSame('[email protected]', $schedule->getExtensions()[0]->getEmail()->getCc()[0]->toString());
        $this->assertSame('my subject', $schedule->getExtensions()[0]->getEmail()->getSubject());
    }

    /**
     * @test
     */
    public function can_add_environment_extension()
    {
        $schedule = new Schedule();
        $schedule->environments('prod', 'stage');

        $this->assertSame(['prod', 'stage'], $schedule->getExtensions()[0]->getRunEnvironments());
        $this->assertSame(['prod', 'stage'], $schedule->getExtensions()[0]->/** @scrutinizer ignore-call */ getRunEnvironments());
    }

    /**
     * @test
     */
    public function can_set_timezone()
    {
        $schedule = new Schedule();

        $this->assertNull($schedule->getTimezone());

        $schedule->timezone('UTC');

        $this->assertSame('UTC', $schedule->getTimezone()->getName());

        $schedule->timezone(new \DateTimeZone('America/Los_Angeles'));

        $this->assertSame('America/Los_Angeles', $schedule->getTimezone()->getName());
    }
}

zenstruck / schedule-bundle

Push — master ( 68072d...6506ec )

ScheduleTest::can_add_tasks() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like
