Fix issue where date ranges might include too many subperiods (#18995)

* Fix issue where date ranges might include too many subperiods when including a magic keyword and a timezone

* more tests

1 files changed, 42 insertions, 0 deletions

diff --git a/tests/PHPUnit/Unit/Period/RangeTest.php b/tests/PHPUnit/Unit/Period/RangeTest.php
index bbb0873b87..f4eaf7366f 100644
--- a/tests/PHPUnit/Unit/Period/RangeTest.php
+++ b/tests/PHPUnit/Unit/Period/RangeTest.php
@@ -20,6 +20,12 @@ use Piwik\Period\Year;
  */
 class RangeTest extends BasePeriodTest
 {
+    public function setUp(): void
+    {
+        parent::setUp();
+        Date::$now = null;
+    }
+
     /**
      * @dataProvider getDateXPeriodsAgoProvider
      */
@@ -183,6 +189,42 @@ class RangeTest extends BasePeriodTest
     }
 
     // test range date1,date2
+    public function testRangeComma4_EndDateIncludesTodayWithTimezone()
+    {
+        Date::$now = strtotime('2020-08-01 03:00:00');
+        $range = new Range('day', '2008-01-01,today', 'Europe/Berlin');
+        $subPeriods = $range->getSubperiods();
+        $this->assertEquals('2008-01-01', $subPeriods[0]->toString());
+        $this->assertEquals('2008-01-02', $subPeriods[1]->toString());
+        $this->assertEquals('2008-01-03', $subPeriods[2]->toString());
+        $this->assertEquals('2020-08-01', end($subPeriods)->toString());
+    }
+
+    // test range date1,date2
+    public function testRangeComma5_EndDateIncludesTodayWithTimezoneAfterCurrentUTCDate()
+    {
+        Date::$now = strtotime('2020-08-01 03:00:00');
+        $range = new Range('day', '2008-01-01,today', 'Pacific/Auckland');
+        $subPeriods = $range->getSubperiods();
+        $this->assertEquals('2008-01-01', $subPeriods[0]->toString());
+        $this->assertEquals('2008-01-02', $subPeriods[1]->toString());
+        $this->assertEquals('2008-01-03', $subPeriods[2]->toString());
+        $this->assertEquals('2020-08-01', end($subPeriods)->toString());
+    }
+
+    // test range date1,date2
+    public function testRangeComma6_EndDateIncludesTodayWithTimezoneBeforeCurrentUTCDate()
+    {
+        Date::$now = strtotime('2020-08-01 03:00:00');
+        $range = new Range('day', '2008-01-01,today', 'America/New_York');
+        $subPeriods = $range->getSubperiods();
+        $this->assertEquals('2008-01-01', $subPeriods[0]->toString());
+        $this->assertEquals('2008-01-02', $subPeriods[1]->toString());
+        $this->assertEquals('2008-01-03', $subPeriods[2]->toString());
+        $this->assertEquals('2020-07-31', end($subPeriods)->toString());
+    }
+
+    // test range date1,date2
     public function testRangeWeekcomma1()
     {
         $range = new Range('week', '2007-12-22,2008-01-03');