Skip to content

Commit 00825ef

Browse files
committed
MC-13732: Mainline test failure Magento\Reports\Test\TestCase\SalesOrderReportEntityTest
1 parent 8e95f01 commit 00825ef

File tree

1 file changed

+9
-12
lines changed

1 file changed

+9
-12
lines changed

dev/tests/functional/tests/app/Magento/Reports/Test/Constraint/AssertReportStatisticsNoticeMessage.php

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -7,8 +7,6 @@
77
namespace Magento\Reports\Test\Constraint;
88

99
use Magento\Reports\Test\Page\Adminhtml\SalesReport;
10-
use Magento\Sales\Test\Fixture\OrderInjectable;
11-
use DateTime;
1210

1311
/**
1412
* Assert that message in Sales Reports Pages displays correct date/time.
@@ -27,23 +25,22 @@ class AssertReportStatisticsNoticeMessage extends AbstractAssertSalesReportResul
2725
*
2826
* @param array $salesReport
2927
* @param SalesReport $salesReportPage
30-
* @param DateTime $currentDate
3128
* @return void
3229
*/
3330
public function processAssert(
3431
array $salesReport,
35-
SalesReport $salesReportPage,
36-
DateTime $currentDate
32+
SalesReport $salesReportPage
3733
) {
34+
$timezone = new \DateTimeZone($_ENV['magento_timezone']);
35+
$initialDate = new \DateTime('now', $timezone);
3836
$this->salesReportPage = $salesReportPage;
3937
$this->searchInSalesReportGrid($salesReport);
40-
$date = $this->getLastUpdatedDate();
41-
$currentDate->setTimezone(new \DateTimeZone($_ENV['magento_timezone']));
42-
$currentDateTime = $currentDate->format('M j, Y, g');
43-
$displayedDateTime = date('M j, Y, g', strtotime($date));
44-
\PHPUnit\Framework\Assert::assertEquals(
45-
$currentDateTime,
46-
$displayedDateTime,
38+
$displayedDate = new \DateTime($this->getLastUpdatedDate(), $timezone);
39+
$currentDate = new \DateTime('now', $timezone);
40+
41+
\PHPUnit\Framework\Assert::assertTrue(
42+
$displayedDate->getTimestamp() > $initialDate->getTimestamp()
43+
&& $displayedDate->getTimestamp() < $currentDate->getTimestamp(),
4744
"Message in Sales Reports Page is displayed in an incorrect timezone."
4845
);
4946
}

0 commit comments

Comments
 (0)