zenstruck / schedule-bundle

src/Schedule/Task.php

Spacing +4 added lines, -4 removed lines

@@ -123,11 +123,11 @@ 
      */
     final public function when(string $description, $callback): self
     {
-        $callback = \is_callable($callback) ? $callback : function () use ($callback) {
+        $callback = \is_callable($callback) ? $callback : function() use ($callback) {
             return (bool) $callback;
         };
 
-        return $this->filter(function (BeforeTaskEvent $event) use ($callback, $description) {
+        return $this->filter(function(BeforeTaskEvent $event) use ($callback, $description) {
             if (!$callback($event)) {
                 throw new SkipTask($description);
             }
@@ -142,11 +142,11 @@ 
      */
     final public function skip(string $description, $callback): self
     {
-        $callback = \is_callable($callback) ? $callback : function () use ($callback) {
+        $callback = \is_callable($callback) ? $callback : function() use ($callback) {
             return (bool) $callback;
         };
 
-        return $this->filter(function (BeforeTaskEvent $event) use ($callback, $description) {
+        return $this->filter(function(BeforeTaskEvent $event) use ($callback, $description) {
             if ($callback($event)) {
                 throw new SkipTask($description);
             }

src/Schedule/Extension/EmailExtension.php

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
             return "{$this->hook}, email output";
         }
 
-        $to = \array_map(function (Address $address) { return $address->toString(); }, $to);
+        $to = \array_map(function(Address $address) { return $address->toString(); }, $to);
         $to = \implode('; ', $to);
 
         return "{$this->hook}, email output to \"{$to}\"";

src/Command/ScheduleListCommand.php

Spacing +1 added lines, -1 removed lines

@@ -147,7 +147,7 @@
 
         $io->comment(\sprintf('<info>%d</info> %s Extension%s:', $count, $type, $count > 1 ? 's' : ''));
         $io->listing(\array_map(
-            function (Extension $extension) {
+            function(Extension $extension) {
                 return \sprintf('%s <comment>(%s)</comment>',
                     \strtr($extension, self::extensionHighlightMap()),
                     \get_class($extension)

src/Schedule.php

Spacing +5 added lines, -5 removed lines

@@ -31,7 +31,7 @@ 
     public function getId(): string
     {
         $tasks = \array_map(
-            function (Task $task) {
+            function(Task $task) {
                 return $task->getId();
             },
             $this->all()
@@ -99,11 +99,11 @@ 
      */
     public function when(string $description, $callback): self
     {
-        $callback = \is_callable($callback) ? $callback : function () use ($callback) {
+        $callback = \is_callable($callback) ? $callback : function() use ($callback) {
             return (bool) $callback;
         };
 
-        return $this->filter(function (BeforeScheduleEvent $event) use ($callback, $description) {
+        return $this->filter(function(BeforeScheduleEvent $event) use ($callback, $description) {
             if (!$callback($event)) {
                 throw new SkipSchedule($description);
             }
@@ -118,11 +118,11 @@ 
      */
     public function skip(string $description, $callback): self
     {
-        $callback = \is_callable($callback) ? $callback : function () use ($callback) {
+        $callback = \is_callable($callback) ? $callback : function() use ($callback) {
             return (bool) $callback;
         };
 
-        return $this->filter(function (BeforeScheduleEvent $event) use ($callback, $description) {
+        return $this->filter(function(BeforeScheduleEvent $event) use ($callback, $description) {
             if ($callback($event)) {
                 throw new SkipSchedule($description);
             }

tests/Functional/LoggingTest.php

Spacing +1 added lines, -1 removed lines

@@ -112,7 +112,7 @@
     {
         $event = $this->createRunnerBuilder()
             ->addTask(new MockTask())
-            ->addExtension(CallbackExtension::scheduleFilter(function () {
+            ->addExtension(CallbackExtension::scheduleFilter(function() {
                 throw new SkipSchedule('the schedule has skipped');
             }))
             ->run()

tests/Functional/SelfHandlingExtensionTest.php

Spacing +10 added lines, -10 removed lines

@@ -77,19 +77,19 @@
             public function buildSchedule(Schedule $schedule): void
             {
                 $schedule
-                    ->filter(function () { $this->calls[] = 'scheduleFilter'; })
-                    ->before(function () { $this->calls[] = 'scheduleBefore'; })
-                    ->after(function () { $this->calls[] = 'scheduleAfter'; })
-                    ->onSuccess(function () { $this->calls[] = 'scheduleSuccess'; })
-                    ->onFailure(function () { $this->calls[] = 'scheduleFailure'; })
+                    ->filter(function() { $this->calls[] = 'scheduleFilter'; })
+                    ->before(function() { $this->calls[] = 'scheduleBefore'; })
+                    ->after(function() { $this->calls[] = 'scheduleAfter'; })
+                    ->onSuccess(function() { $this->calls[] = 'scheduleSuccess'; })
+                    ->onFailure(function() { $this->calls[] = 'scheduleFailure'; })
                 ;
 
                 $schedule->add($this->task)
-                    ->filter(function () { $this->calls[] = 'taskFilter'; })
-                    ->before(function () { $this->calls[] = 'taskBefore'; })
-                    ->after(function () { $this->calls[] = 'taskAfter'; })
-                    ->onSuccess(function () { $this->calls[] = 'taskSuccess'; })
-                    ->onFailure(function () { $this->calls[] = 'taskFailure'; })
+                    ->filter(function() { $this->calls[] = 'taskFilter'; })
+                    ->before(function() { $this->calls[] = 'taskBefore'; })
+                    ->after(function() { $this->calls[] = 'taskAfter'; })
+                    ->onSuccess(function() { $this->calls[] = 'taskSuccess'; })
+                    ->onFailure(function() { $this->calls[] = 'taskFailure'; })
                 ;
             }
         };

tests/ScheduleTest.php

Spacing +16 added lines, -16 removed lines

@@ -28,14 +28,14 @@ 
     {
         $schedule = new Schedule();
 
-        $schedule->add(new CallbackTask(function () {}))->description('task1');
-        $schedule->addCallback(function () {})->description('task2');
+        $schedule->add(new CallbackTask(function() {}))->description('task1');
+        $schedule->addCallback(function() {})->description('task2');
         $schedule->addProcess('php -v')->description('task3');
         $schedule->addCommand('my:command')->description('task4');
 
         $this->assertCount(4, $schedule->all());
         $this->assertSame(['task1', 'task2', 'task3', 'task4'], \array_map(
-            function (Task $task) {
+            function(Task $task) {
                 return $task->getDescription();
             },
             $schedule->all()
@@ -52,7 +52,7 @@ 
         $schedule->addCommand('my:command')->description('task1')->tuesdays();
         $schedule->addCompound()
             ->addCommand('another:command', [], 'task2')
-            ->addCallback(function () {}, 'task3')
+            ->addCallback(function() {}, 'task3')
             ->addProcess('php -v', 'task4')
             ->add((new CommandTask('yet:another:command'))
                 ->description('task5')
@@ -94,7 +94,7 @@ 
     {
         $schedule = new Schedule();
 
-        $schedule->addCallback(function () {})->description('task1');
+        $schedule->addCallback(function() {})->description('task1');
         $notDueTask = $schedule->addProcess('php -v')->description('task2')->sundays();
 
         if ('Sun' === \date('D')) {
@@ -150,7 +150,7 @@ 
     {
         $schedule = new Schedule();
 
-        $schedule->when('callback value', function () { return false; });
+        $schedule->when('callback value', function() { return false; });
 
         $this->expectException(SkipSchedule::class);
         $this->expectExceptionMessage('callback value');
@@ -179,7 +179,7 @@ 
     {
         $schedule = new Schedule();
 
-        $schedule->when('callback value', function () { return true; });
+        $schedule->when('callback value', function() { return true; });
 
         $schedule->getExtensions()[0]->filterSchedule(new BeforeScheduleEvent($schedule));
 
@@ -208,7 +208,7 @@ 
     {
         $schedule = new Schedule();
 
-        $schedule->skip('callback value', function () { return true; });
+        $schedule->skip('callback value', function() { return true; });
 
         $this->expectException(SkipSchedule::class);
         $this->expectExceptionMessage('callback value');
@@ -237,7 +237,7 @@ 
     {
         $schedule = new Schedule();
 
-        $schedule->skip('callback value', function () { return false; });
+        $schedule->skip('callback value', function() { return false; });
 
         $schedule->getExtensions()[0]->filterSchedule(new BeforeScheduleEvent($schedule));
 
@@ -252,12 +252,12 @@ 
         $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->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));
@@ -334,7 +334,7 @@ 
     public function can_add_email_on_failure_extension()
     {
         $schedule = new Schedule();
-        $schedule->emailOnFailure('kevin@example.com', 'my subject', function (Email $email) {
+        $schedule->emailOnFailure('kevin@example.com', 'my subject', function(Email $email) {
             $email->cc('emily@example.com');
         });
 

tests/Schedule/TaskTest.php

Spacing +46 added lines, -46 removed lines

@@ -70,40 +70,40 @@ 
     public static function frequencyProvider(): array
     {
         return [
-            [function () { return self::createTask()->daily(); }, '0 0 * * *'],
-            [function () { return self::createTask()->cron('0 0 * * *')->everyMinute(); }, '* 0 * * *'],
-            [function () { return self::createTask()->everyFiveMinutes(); }, '*/5 * * * *'],
-            [function () { return self::createTask()->everyTenMinutes(); }, '*/10 * * * *'],
-            [function () { return self::createTask()->everyFifteenMinutes(); }, '*/15 * * * *'],
-            [function () { return self::createTask()->everyThirtyMinutes(); }, '0,30 * * * *'],
-            [function () { return self::createTask()->hourly(); }, '0 * * * *'],
-            [function () { return self::createTask()->hourlyAt(6); }, '6 * * * *'],
-            [function () { return self::createTask()->at('3'); }, '0 3 * * *'],
-            [function () { return self::createTask()->at('3:16'); }, '16 3 * * *'],
-            [function () { return self::createTask()->dailyAt('3'); }, '0 3 * * *'],
-            [function () { return self::createTask()->twiceDaily(); }, '0 1,13 * * *'],
-            [function () { return self::createTask()->twiceDaily(2, 14); }, '0 2,14 * * *'],
-            [function () { return self::createTask()->weekdays(); }, '* * * * 1-5'],
-            [function () { return self::createTask()->weekdays()->at(2); }, '0 2 * * 1-5'],
-            [function () { return self::createTask()->weekends(); }, '* * * * 0,6'],
-            [function () { return self::createTask()->mondays(); }, '* * * * 1'],
-            [function () { return self::createTask()->tuesdays(); }, '* * * * 2'],
-            [function () { return self::createTask()->wednesdays(); }, '* * * * 3'],
-            [function () { return self::createTask()->thursdays(); }, '* * * * 4'],
-            [function () { return self::createTask()->fridays(); }, '* * * * 5'],
-            [function () { return self::createTask()->saturdays(); }, '* * * * 6'],
-            [function () { return self::createTask()->sundays(); }, '* * * * 0'],
-            [function () { return self::createTask()->days(1, 2, 3); }, '* * * * 1,2,3'],
-            [function () { return self::createTask()->weekly(); }, '0 0 * * 0'],
-            [function () { return self::createTask()->weekly()->at('3:15'); }, '15 3 * * 0'],
-            [function () { return self::createTask()->monthly(); }, '0 0 1 * *'],
-            [function () { return self::createTask()->monthlyOn(3); }, '0 0 3 * *'],
-            [function () { return self::createTask()->monthlyOn(3, '4:15'); }, '15 4 3 * *'],
-            [function () { return self::createTask()->twiceMonthly(); }, '0 0 1,16 * *'],
-            [function () { return self::createTask()->twiceMonthly(3, 17); }, '0 0 3,17 * *'],
-            [function () { return self::createTask()->twiceMonthly()->at('3:15'); }, '15 3 1,16 * *'],
-            [function () { return self::createTask()->quarterly(); }, '0 0 1 1-12/3 *'],
-            [function () { return self::createTask()->yearly(); }, '0 0 1 1 *'],
+            [function() { return self::createTask()->daily(); }, '0 0 * * *'],
+            [function() { return self::createTask()->cron('0 0 * * *')->everyMinute(); }, '* 0 * * *'],
+            [function() { return self::createTask()->everyFiveMinutes(); }, '*/5 * * * *'],
+            [function() { return self::createTask()->everyTenMinutes(); }, '*/10 * * * *'],
+            [function() { return self::createTask()->everyFifteenMinutes(); }, '*/15 * * * *'],
+            [function() { return self::createTask()->everyThirtyMinutes(); }, '0,30 * * * *'],
+            [function() { return self::createTask()->hourly(); }, '0 * * * *'],
+            [function() { return self::createTask()->hourlyAt(6); }, '6 * * * *'],
+            [function() { return self::createTask()->at('3'); }, '0 3 * * *'],
+            [function() { return self::createTask()->at('3:16'); }, '16 3 * * *'],
+            [function() { return self::createTask()->dailyAt('3'); }, '0 3 * * *'],
+            [function() { return self::createTask()->twiceDaily(); }, '0 1,13 * * *'],
+            [function() { return self::createTask()->twiceDaily(2, 14); }, '0 2,14 * * *'],
+            [function() { return self::createTask()->weekdays(); }, '* * * * 1-5'],
+            [function() { return self::createTask()->weekdays()->at(2); }, '0 2 * * 1-5'],
+            [function() { return self::createTask()->weekends(); }, '* * * * 0,6'],
+            [function() { return self::createTask()->mondays(); }, '* * * * 1'],
+            [function() { return self::createTask()->tuesdays(); }, '* * * * 2'],
+            [function() { return self::createTask()->wednesdays(); }, '* * * * 3'],
+            [function() { return self::createTask()->thursdays(); }, '* * * * 4'],
+            [function() { return self::createTask()->fridays(); }, '* * * * 5'],
+            [function() { return self::createTask()->saturdays(); }, '* * * * 6'],
+            [function() { return self::createTask()->sundays(); }, '* * * * 0'],
+            [function() { return self::createTask()->days(1, 2, 3); }, '* * * * 1,2,3'],
+            [function() { return self::createTask()->weekly(); }, '0 0 * * 0'],
+            [function() { return self::createTask()->weekly()->at('3:15'); }, '15 3 * * 0'],
+            [function() { return self::createTask()->monthly(); }, '0 0 1 * *'],
+            [function() { return self::createTask()->monthlyOn(3); }, '0 0 3 * *'],
+            [function() { return self::createTask()->monthlyOn(3, '4:15'); }, '15 4 3 * *'],
+            [function() { return self::createTask()->twiceMonthly(); }, '0 0 1,16 * *'],
+            [function() { return self::createTask()->twiceMonthly(3, 17); }, '0 0 3,17 * *'],
+            [function() { return self::createTask()->twiceMonthly()->at('3:15'); }, '15 3 1,16 * *'],
+            [function() { return self::createTask()->quarterly(); }, '0 0 1 1-12/3 *'],
+            [function() { return self::createTask()->yearly(); }, '0 0 1 1 *'],
         ];
     }
 
@@ -152,7 +152,7 @@ 
     {
         $task = self::createTask();
 
-        $task->when('callback value', function () { return false; });
+        $task->when('callback value', function() { return false; });
 
         $this->expectException(SkipTask::class);
         $this->expectExceptionMessage('callback value');
@@ -181,7 +181,7 @@ 
     {
         $task = self::createTask();
 
-        $task->when('callback value', function () { return true; });
+        $task->when('callback value', function() { return true; });
 
         $task->getExtensions()[0]->filterTask(new BeforeTaskEvent(new BeforeScheduleEvent(new Schedule()), $task));
 
@@ -210,7 +210,7 @@ 
     {
         $task = self::createTask();
 
-        $task->skip('callback value', function () { return true; });
+        $task->skip('callback value', function() { return true; });
 
         $this->expectException(SkipTask::class);
         $this->expectExceptionMessage('callback value');
@@ -239,7 +239,7 @@ 
     {
         $task = self::createTask();
 
-        $task->skip('callback value', function () { return false; });
+        $task->skip('callback value', function() { return false; });
 
         $task->getExtensions()[0]->filterTask(new BeforeTaskEvent(new BeforeScheduleEvent(new Schedule()), $task));
 
@@ -254,12 +254,12 @@ 
         $task = self::createTask();
         $calls = [];
 
-        $task->filter(function () use (&$calls) { $calls[] = 'filter'; });
-        $task->before(function () use (&$calls) { $calls[] = 'before'; });
-        $task->after(function () use (&$calls) { $calls[] = 'after'; });
-        $task->then(function () use (&$calls) { $calls[] = 'then'; });
-        $task->onSuccess(function () use (&$calls) { $calls[] = 'onSuccess'; });
-        $task->onFailure(function () use (&$calls) { $calls[] = 'onFailure'; });
+        $task->filter(function() use (&$calls) { $calls[] = 'filter'; });
+        $task->before(function() use (&$calls) { $calls[] = 'before'; });
+        $task->after(function() use (&$calls) { $calls[] = 'after'; });
+        $task->then(function() use (&$calls) { $calls[] = 'then'; });
+        $task->onSuccess(function() use (&$calls) { $calls[] = 'onSuccess'; });
+        $task->onFailure(function() use (&$calls) { $calls[] = 'onFailure'; });
 
         $task->getExtensions()[0]->filterTask($event = new BeforeTaskEvent(new BeforeScheduleEvent(new Schedule()), $task));
         $task->getExtensions()[1]->beforeTask($event);
@@ -314,7 +314,7 @@ 
     public function can_add_email_after_extension($method)
     {
         $task = self::createTask();
-        $task->{$method}('kevin@example.com', 'my subject', function (Email $email) {
+        $task->{$method}('kevin@example.com', 'my subject', function(Email $email) {
             $email->cc('emily@example.com');
         });
 
@@ -338,7 +338,7 @@ 
     public function can_add_email_on_failure_extension()
     {
         $task = self::createTask();
-        $task->emailOnFailure('kevin@example.com', 'my subject', function (Email $email) {
+        $task->emailOnFailure('kevin@example.com', 'my subject', function(Email $email) {
             $email->cc('emily@example.com');
         });
 

tests/Schedule/Task/CallbackTaskTest.php

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
      */
     public function can_create_successful_result()
     {
-        $result = (new CallbackTask(function () {}))();
+        $result = (new CallbackTask(function() {}))();
 
         $this->assertTrue($result->isSuccessful());
         $this->assertNull($result->getOutput());
@@ -27,7 +27,7 @@ 
      */
     public function stringifies_output($output, $expectedOutput)
     {
-        $result = (new CallbackTask(function () use ($output) { return $output; }))();
+        $result = (new CallbackTask(function() use ($output) { return $output; }))();
 
         $this->assertTrue($result->isSuccessful());
         $this->assertSame($expectedOutput, $result->getOutput());