diff options
author | J-P Nurmi <jpnurmi@qt.io> | 2017-06-26 03:08:33 -1000 |
---|---|---|
committer | J-P Nurmi <jpnurmi@qt.io> | 2017-06-26 13:21:28 +0000 |
commit | eea527e1d4f8182d13a5fc5597370ac50cdaa5fe (patch) | |
tree | f7e4de75e7f87c196c08771f6bebc0bc8a746cca /tests | |
parent | 5f09ab8b94ab83e04493940cbd6c95e6d363d0db (diff) |
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 <mitch.curtis@qt.io>
Diffstat (limited to 'tests')
-rw-r--r-- | tests/auto/calendar/data/tst_monthgrid.qml | 24 |
1 files changed, 12 insertions, 12 deletions
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() |