diff options
author | Edward Welbourne <edward.welbourne@qt.io> | 2021-01-08 13:48:51 +0100 |
---|---|---|
committer | Qt Cherry-pick Bot <cherrypick_bot@qt-project.org> | 2021-02-02 11:56:16 +0000 |
commit | 8b1194edea390f5608526a7f15cc562a3f17656b (patch) | |
tree | c77d8c1ceb81bf905efb623b5cea480ac5b8a354 /tests/auto/corelib/time/qtimezone/BLACKLIST | |
parent | f1efd15d430f8d00cf1839b600a9e4b5771b8e38 (diff) |
Correct string comparison in Android's IANA ID matching code
It used QString.compare() and assumed it was returning a bool true on
equality, when it actually returns an int that compares to 0 as the
given strings compare. So it should use compare() == 0.
This fixes several of QTimeZone's blacklisted tests on Android and a
crasher, which we dodged with a QSKIP. Added an id-comparison to a
test. Gave two local variables more informative names, made an early
return into a QSKIP so it explains itself.
Fixes: QTBUG-89905
Fixes: QTBUG-69122
Fixes: QTBUG-69132
Fixes: QTBUG-87435
Change-Id: Icf18ed5a810143d6e65d36e34a70e82faac10b8e
Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Reviewed-by: Andrei Golubev <andrei.golubev@qt.io>
(cherry picked from commit 6ee13db700eecd8dfed54a9ec2d1081b39511562)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
Diffstat (limited to 'tests/auto/corelib/time/qtimezone/BLACKLIST')
-rw-r--r-- | tests/auto/corelib/time/qtimezone/BLACKLIST | 15 |
1 files changed, 0 insertions, 15 deletions
diff --git a/tests/auto/corelib/time/qtimezone/BLACKLIST b/tests/auto/corelib/time/qtimezone/BLACKLIST index ddc9a107c0..1f4811d4b8 100644 --- a/tests/auto/corelib/time/qtimezone/BLACKLIST +++ b/tests/auto/corelib/time/qtimezone/BLACKLIST @@ -1,7 +1,3 @@ -# QTBUG-69122 -[dataStreamTest] -android - # QTBUG-69129 [specificTransition] android @@ -173,14 +169,3 @@ android android [transitionEachZone:America/Danmarkshavn@1970] android -# QTBUG-87435 -[createTest] -android -[systemZone] -android -[utcOffsetId] -android -[stressTest] -android -[serialize] -android |