From eea527e1d4f8182d13a5fc5597370ac50cdaa5fe Mon Sep 17 00:00:00 2001 From: J-P Nurmi Date: Mon, 26 Jun 2017 03:08:33 -1000 Subject: Fix tst_calendar::MonthGrid::test_locale() When the JS Date constructor is passed a string in the ISO 8601 format (e.g. "1970-01-01"), it is treatead as UTC, not local. Therefore, while comparing the expected dates from the model, we must base the comparison on UTC as well. Otherwise, the actual and expected dates may differ due to the timezone difference. This issue was detected thanks to a new CI setup that had accidentally UTC+8 timezone set. Task-number: QTBUG-61535 Change-Id: I93cfce5a81d48dc859f002dcd6b59ddb0701911c Reviewed-by: Mitch Curtis --- tests/auto/calendar/data/tst_monthgrid.qml | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) (limited to 'tests/auto/calendar') diff --git a/tests/auto/calendar/data/tst_monthgrid.qml b/tests/auto/calendar/data/tst_monthgrid.qml index f07121dc..e524f53e 100644 --- a/tests/auto/calendar/data/tst_monthgrid.qml +++ b/tests/auto/calendar/data/tst_monthgrid.qml @@ -107,13 +107,13 @@ TestCase { for (var i = 0; i < 42; ++i) { var cellDate = new Date(en_GB[i]) - compare(control.contentItem.children[i].date.getFullYear(), cellDate.getFullYear()) - compare(control.contentItem.children[i].date.getMonth(), cellDate.getMonth()) - compare(control.contentItem.children[i].date.getDate(), cellDate.getDate()) - compare(control.contentItem.children[i].day, cellDate.getDate()) + compare(control.contentItem.children[i].date.getFullYear(), cellDate.getUTCFullYear()) + compare(control.contentItem.children[i].date.getMonth(), cellDate.getUTCMonth()) + compare(control.contentItem.children[i].date.getDate(), cellDate.getUTCDate()) + compare(control.contentItem.children[i].day, cellDate.getUTCDate()) compare(control.contentItem.children[i].today, cellDate === new Date()) - compare(control.contentItem.children[i].month, cellDate.getMonth()) - compare(control.contentItem.children[i].year, cellDate.getFullYear()) + compare(control.contentItem.children[i].month, cellDate.getUTCMonth()) + compare(control.contentItem.children[i].year, cellDate.getUTCFullYear()) } // en_US @@ -130,13 +130,13 @@ TestCase { for (var j = 0; j < 42; ++j) { cellDate = new Date(en_US[j]) - compare(control.contentItem.children[j].date.getFullYear(), cellDate.getFullYear()) - compare(control.contentItem.children[j].date.getMonth(), cellDate.getMonth()) - compare(control.contentItem.children[j].date.getDate(), cellDate.getDate()) - compare(control.contentItem.children[j].day, cellDate.getDate()) + compare(control.contentItem.children[j].date.getFullYear(), cellDate.getUTCFullYear()) + compare(control.contentItem.children[j].date.getMonth(), cellDate.getUTCMonth()) + compare(control.contentItem.children[j].date.getDate(), cellDate.getUTCDate()) + compare(control.contentItem.children[j].day, cellDate.getUTCDate()) compare(control.contentItem.children[j].today, cellDate === new Date()) - compare(control.contentItem.children[j].month, cellDate.getMonth()) - compare(control.contentItem.children[j].year, cellDate.getFullYear()) + compare(control.contentItem.children[j].month, cellDate.getUTCMonth()) + compare(control.contentItem.children[j].year, cellDate.getUTCFullYear()) } control.destroy() -- cgit v1.2.3