From e15548d3e49a8937856c78d8e6c1133d46df57ee Mon Sep 17 00:00:00 2001 From: Jason McDonald Date: Fri, 25 Nov 2011 17:04:12 +1000 Subject: Fix sanity check of network test server Some of Qt's autotests depend on access to a test server. For each test that used the test server, tests/auto/network-settings.h created a global object to verify at startup that host lookups to the test server will succeed (and abort the test otherwise). There are two problems with that approach: First, the sanity check happens before main(), and thus before the test framework has started logging test results. This means that if the sanity check aborts the test, the failure message will not be visible in the test output if logging to a file or will cause the output to be malformed if logging to the console in XML format. Second, since Qt 4.7, the host lookup uses a class that connects to the QCoreApplication instance, which doesn't exist before main(), and this caused all tests that included network-settings.h to output an error message from QObject::connect() at the beginning of the test. Both of these problems are solved by removing the global object from network-settings.h and instead performing the sanity check in the initTestCase() function of each test. Task-number: QTBUG-22876 Change-Id: Id49c1826906327bf571686cc11527f0265e5af44 Reviewed-by: Rohan McGovern --- .../socket/platformsocketengine/tst_platformsocketengine.cpp | 6 ++++++ .../auto/network/socket/qhttpsocketengine/tst_qhttpsocketengine.cpp | 5 +++++ .../network/socket/qsocks5socketengine/tst_qsocks5socketengine.cpp | 6 ++++++ tests/auto/network/socket/qtcpserver/tst_qtcpserver.cpp | 1 + tests/auto/network/socket/qtcpsocket/tst_qtcpsocket.cpp | 6 ++++++ tests/auto/network/socket/qudpsocket/tst_qudpsocket.cpp | 6 ++++++ 6 files changed, 30 insertions(+) (limited to 'tests/auto/network/socket') diff --git a/tests/auto/network/socket/platformsocketengine/tst_platformsocketengine.cpp b/tests/auto/network/socket/platformsocketengine/tst_platformsocketengine.cpp index 103ea1323e..bf3e2f2442 100644 --- a/tests/auto/network/socket/platformsocketengine/tst_platformsocketengine.cpp +++ b/tests/auto/network/socket/platformsocketengine/tst_platformsocketengine.cpp @@ -78,6 +78,7 @@ public: public slots: + void initTestCase(); void init(); void cleanup(); private slots: @@ -106,6 +107,11 @@ tst_PlatformSocketEngine::~tst_PlatformSocketEngine() { } +void tst_PlatformSocketEngine::initTestCase() +{ + QVERIFY(QtNetworkSettings::verifyTestNetworkSettings()); +} + void tst_PlatformSocketEngine::init() { } diff --git a/tests/auto/network/socket/qhttpsocketengine/tst_qhttpsocketengine.cpp b/tests/auto/network/socket/qhttpsocketengine/tst_qhttpsocketengine.cpp index 92da0e865b..93344a3bf4 100644 --- a/tests/auto/network/socket/qhttpsocketengine/tst_qhttpsocketengine.cpp +++ b/tests/auto/network/socket/qhttpsocketengine/tst_qhttpsocketengine.cpp @@ -66,6 +66,7 @@ public: public slots: + void initTestCase(); void init(); void cleanup(); private slots: @@ -147,6 +148,10 @@ tst_QHttpSocketEngine::~tst_QHttpSocketEngine() { } +void tst_QHttpSocketEngine::initTestCase() +{ + QVERIFY(QtNetworkSettings::verifyTestNetworkSettings()); +} void tst_QHttpSocketEngine::init() { diff --git a/tests/auto/network/socket/qsocks5socketengine/tst_qsocks5socketengine.cpp b/tests/auto/network/socket/qsocks5socketengine/tst_qsocks5socketengine.cpp index d738926ca8..8c600c5400 100644 --- a/tests/auto/network/socket/qsocks5socketengine/tst_qsocks5socketengine.cpp +++ b/tests/auto/network/socket/qsocks5socketengine/tst_qsocks5socketengine.cpp @@ -73,6 +73,7 @@ public: public slots: + void initTestCase(); void init(); void cleanup(); private slots: @@ -157,6 +158,11 @@ tst_QSocks5SocketEngine::~tst_QSocks5SocketEngine() { } +void tst_QSocks5SocketEngine::initTestCase() +{ + QVERIFY(QtNetworkSettings::verifyTestNetworkSettings()); +} + void tst_QSocks5SocketEngine::init() { tmpSocket = 0; diff --git a/tests/auto/network/socket/qtcpserver/tst_qtcpserver.cpp b/tests/auto/network/socket/qtcpserver/tst_qtcpserver.cpp index f96ad2781e..0cdd3a7120 100644 --- a/tests/auto/network/socket/qtcpserver/tst_qtcpserver.cpp +++ b/tests/auto/network/socket/qtcpserver/tst_qtcpserver.cpp @@ -155,6 +155,7 @@ void tst_QTcpServer::initTestCase_data() void tst_QTcpServer::initTestCase() { + QVERIFY(QtNetworkSettings::verifyTestNetworkSettings()); #ifndef QT_NO_BEARERMANAGEMENT QNetworkConfigurationManager man; networkSession = new QNetworkSession(man.defaultConfiguration(), this); diff --git a/tests/auto/network/socket/qtcpsocket/tst_qtcpsocket.cpp b/tests/auto/network/socket/qtcpsocket/tst_qtcpsocket.cpp index 3fb4333494..9c87ac04a0 100644 --- a/tests/auto/network/socket/qtcpsocket/tst_qtcpsocket.cpp +++ b/tests/auto/network/socket/qtcpsocket/tst_qtcpsocket.cpp @@ -133,6 +133,7 @@ public: public slots: void initTestCase_data(); + void initTestCase(); void init(); void cleanup(); private slots: @@ -348,6 +349,11 @@ void tst_QTcpSocket::initTestCase_data() #endif } +void tst_QTcpSocket::initTestCase() +{ + QVERIFY(QtNetworkSettings::verifyTestNetworkSettings()); +} + void tst_QTcpSocket::init() { QFETCH_GLOBAL(bool, setProxy); diff --git a/tests/auto/network/socket/qudpsocket/tst_qudpsocket.cpp b/tests/auto/network/socket/qudpsocket/tst_qudpsocket.cpp index f737b94e5e..786714af13 100644 --- a/tests/auto/network/socket/qudpsocket/tst_qudpsocket.cpp +++ b/tests/auto/network/socket/qudpsocket/tst_qudpsocket.cpp @@ -80,6 +80,7 @@ public: public slots: void initTestCase_data(); + void initTestCase(); void init(); void cleanup(); private slots: @@ -159,6 +160,11 @@ void tst_QUdpSocket::initTestCase_data() #endif } +void tst_QUdpSocket::initTestCase() +{ + QVERIFY(QtNetworkSettings::verifyTestNetworkSettings()); +} + void tst_QUdpSocket::init() { QFETCH_GLOBAL(bool, setProxy); -- cgit v1.2.3