summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorKirill Burtsev <kirill.burtsev@qt.io>2019-11-20 18:46:25 +0100
committerKirill Burtsev <kirill.burtsev@qt.io>2019-11-28 14:08:16 +0100
commit7e653262ec32a3cfa8280df9e1e7d214d333d2f8 (patch)
tree9db4ca64350a0700ee5b5465c83bfaff3918c355
parent9d2521084dbec9403ab68ea5e95b8a77313af11f (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.cpp4
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);