fix cs

Author: michalsn

.php-cs-fixer.dist.php

@@ -43,5 +43,5 @@
 return Factory::create(new CodeIgniter4(), $overrides, $options)->forLibrary(
     'CodeIgniter Tasks',
     'CodeIgniter Foundation',
-    '[email protected]'
+    '[email protected]',
 );

src/Commands/Publish.php

@@ -48,7 +48,7 @@ public function run(array $params)
                     'namespace CodeIgniter\\Tasks\\Config' => 'namespace Config',
                     'use CodeIgniter\\Config\\BaseConfig'  => 'use CodeIgniter\\Tasks\\Config\\Tasks as BaseTasks',
                     'class Tasks extends BaseConfig'       => 'class Tasks extends BaseTasks',
-                ]
+                ],
             );
         }
 

src/CronExpression.php

@@ -145,7 +145,7 @@ private function checkTime(string $time, string $format): bool
         assert(ctype_digit($currentTime));
 
         // Handle repeating times (i.e. /5 or */5 for every 5 minutes)
-        if (strpos($time, '/') !== false) {
+        if (str_contains($time, '/')) {
             $period = substr($time, strpos($time, '/') + 1) ?: '';
 
             if ($period === '' || ! ctype_digit($period)) {
@@ -156,7 +156,7 @@ private function checkTime(string $time, string $format): bool
         }
 
         // Handle ranges (1-5)
-        if (strpos($time, '-') !== false) {
+        if (str_contains($time, '-')) {
             $items         = [];
             [$start, $end] = explode('-', $time);
 

src/RunResolver.php

@@ -82,19 +82,19 @@ public function nextRun(string $expression, Time $next): Time
                     $satisfied = true;
                 }
                 // If the value is a list
-                elseif (strpos($value, ',') !== false) {
+                elseif (str_contains($value, ',')) {
                     if ($this->isInList($nextValue, $value)) {
                         $satisfied = true;
                     }
                 }
                 // If the value is a range
-                elseif (strpos($value, '-') !== false) {
+                elseif (str_contains($value, '-')) {
                     if ($this->isInRange($nextValue, $value)) {
                         $satisfied = true;
                     }
                 }
                 // If the value is an increment
-                elseif (strpos($value, '/') !== false) {
+                elseif (str_contains($value, '/')) {
                     if ($this->isInIncrement($nextValue, $value)) {
                         $satisfied = true;
                     }

tests/unit/CronExpressionTest.php

@@ -15,6 +15,7 @@
 use CodeIgniter\Tasks\CronExpression;
 use CodeIgniter\Tasks\Exceptions\TasksException;
 use CodeIgniter\Test\CIUnitTestCase as TestCase;
+use PHPUnit\Framework\Attributes\DataProvider;
 
 /**
  * @internal
@@ -140,11 +141,10 @@ public function testHoursAndMins()
     }
 
     /**
-     * @dataProvider provideEveryHour
-     *
      * @param mixed $hourTrue
      * @param mixed $hourFalse
      */
+    #[DataProvider('provideEveryHour')]
     public function testEveryHour($hourTrue, $hourFalse)
     {
         $this->cron->testTime($hourTrue);
@@ -234,9 +234,7 @@ public static function provideNextRun(): iterable
         ];
     }
 
-    /**
-     * @dataProvider provideNextRun
-     */
+    #[DataProvider('provideNextRun')]
     public function testNextRun(string $exp, string $expected)
     {
         $this->cron->testTime('October 5, 2020 8:00 pm');

tests/unit/TaskLogTest.php

@@ -15,6 +15,7 @@
 use CodeIgniter\Tasks\Task;
 use CodeIgniter\Tasks\TaskLog;
 use CodeIgniter\Test\CIUnitTestCase as TestCase;
+use PHPUnit\Framework\Attributes\DataProvider;
 
 /**
  * @internal
@@ -46,12 +47,11 @@ public static function provideDuration(): iterable
     }
 
     /**
-     * @dataProvider provideDuration
-     *
      * @param array|bool|int|string|null $output
      *
      * @throws Exception
      */
+    #[DataProvider('provideDuration')]
     public function testDuration(string $start, string $end, string $expected, $output)
     {
         $start = new Time($start);