1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

tst_qnetworkreply: localhost may resolve to ipv4 and 6

Browse Source 
On some platforms, localhost resolves to both ipv4 and ipv6 addresses.
In that case we might get multiple emissions of the
socketStartedConnecting signal, for Happy Eyeballs.

Pick-to: 6.8 6.7 6.5
Change-Id: Ib39a167bdde2089106c89a3e297cbb0ad2f357d9
Reviewed-by: Timur Pocheptsov <timur.pocheptsov@qt.io>
This commit is contained in:
Mårten Nordheim 2024-09-05 15:00:10 +02:00
parent e02813eeb5
commit 13a0f294b5
1 changed files with 8 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
tests/auto/network/access/qnetworkreply/tst_qnetworkreply.cpp
View File

@ -10164,7 +10164,14 @@ void tst_QNetworkReply::moreActivitySignals()
QSignalSpy spy3(reply.data(), SIGNAL(metaDataChanged()));
QSignalSpy spy4(reply.data(), SIGNAL(finished()));
spy1.wait();
QCOMPARE(spy1.size(), 1);
if (url.host() == u"localhost") {
// localhost may resolve to both v4 and v6, so we may get two
// socketStartedConnecting signals
QCOMPARE_GE(spy1.size(), 1);
QCOMPARE_LE(spy1.size(), 2);
} else {
QCOMPARE(spy1.size(), 1);
}
spy2.wait();
QCOMPARE(spy2.size(), 1);
spy3.wait();