diff options
author | Kirill Burtsev <kirill.burtsev@qt.io> | 2019-11-20 18:46:25 +0100 |
---|---|---|
committer | Kirill Burtsev <kirill.burtsev@qt.io> | 2019-11-28 14:08:16 +0100 |
commit | 7e653262ec32a3cfa8280df9e1e7d214d333d2f8 (patch) | |
tree | 9db4ca64350a0700ee5b5465c83bfaff3918c355 | |
parent | 9d2521084dbec9403ab68ea5e95b8a77313af11f (diff) |
Favicon touchIconWithSameURL test: fix waiting on wrong spy
Amends fc0dbde734 timeouts adjust for slower CI
Change-Id: I77bacfb973cfe8e2c259a31d58f439ffcae7b87f
Reviewed-by: Michal Klocek <michal.klocek@qt.io>
-rw-r--r-- | tests/auto/widgets/faviconmanager/tst_faviconmanager.cpp | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/auto/widgets/faviconmanager/tst_faviconmanager.cpp b/tests/auto/widgets/faviconmanager/tst_faviconmanager.cpp index 1469ddb15..46038cdc6 100644 --- a/tests/auto/widgets/faviconmanager/tst_faviconmanager.cpp +++ b/tests/auto/widgets/faviconmanager/tst_faviconmanager.cpp @@ -522,10 +522,10 @@ void tst_FaviconManager::touchIconWithSameURL() "<link rel='icon' type='image/png' href='" + icon + "'/>" "<link rel='apple-touch-icon' type='image/png' href='" + icon + "'/>" "</html>"); - QTRY_COMPARE(loadFinishedSpy.count(), 1); + QTRY_COMPARE_WITH_TIMEOUT(loadFinishedSpy.count(), 1, 30000); // The default favicon has to be loaded even if its URL is also set as a touch icon while touch icons are disabled. - QTRY_COMPARE_WITH_TIMEOUT(loadFinishedSpy.count(), 1, 30000); + QTRY_COMPARE(iconUrlChangedSpy.count(), 1); QCOMPARE(m_page->iconUrl().toString(), icon); QTRY_COMPARE(iconChangedSpy.count(), 1); |