Added support for value 7 as sunday

- Added the weekday abbreviation 7 which defaults to 0
mintware-de · Nov 10, 2022 · c430e23 · c430e23
1 parent 6581886
commit c430e23
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -3,6 +3,7 @@
 # Next version
 Features
 - Added support for month and weekday abbreviations [PR #4](https://github.com/mintware-de/native-cron/pull/4)
+- Added support for value 7 as sunday [PR #5](https://github.com/mintware-de/native-cron/pull/5)
 
 # v1.1.2
 Fixes:

diff --git a/src/Content/DateTimeDefinition.php b/src/Content/DateTimeDefinition.php
@@ -33,6 +33,8 @@ class DateTimeDefinition
         'thu' => 4,
         'fri' => 5,
         'sat' => 6,
+        // Alternative value for sunday
+        '7' => 0,
     ];
 
     /** @var DateTimeField[] */

diff --git a/tests/Content/CronjobLineTest.php b/tests/Content/CronjobLineTest.php
@@ -153,6 +153,13 @@ public function testWeekdayAbbreviations(): void
         self::assertEquals(5, $cronjobLine->getDateTimeDefinition()->getWeekdays()[0]->getValueFrom());
     }
 
+    public function testDaySevenShouldBeSunday(): void
+    {
+        $line = '* * * * 7 www-data command';
+        $cronjobLine = new CronJobLine($line, true);
+        self::assertEquals(0, $cronjobLine->getDateTimeDefinition()->getWeekdays()[0]->getValueFrom());
+    }
+
     private function checkEmptyValues(CronJobLine $cronjobLine): void
     {
         $dateTimeDefinition = $cronjobLine->getDateTimeDefinition();