diff options
author | Luca Niccoli <lultimouomo@gmail.com> | 2015-03-09 19:20:26 +0100 |
---|---|---|
committer | Liang Qi <liang.qi@theqtcompany.com> | 2015-04-08 06:55:15 +0000 |
commit | 30b5e99da347aea446a5a3a334a21a4406adc4fe (patch) | |
tree | b4c7ede1be187a9451c638fdaf2b636e21a32232 /tests/auto | |
parent | 22c10dc825cb8fe329391dc23d8cd30397a3823e (diff) | |
download | qtwebsockets-30b5e99da347aea446a5a3a334a21a4406adc4fe.tar.gz |
Always initialize m_handshakeState
The client-side constructor was leaving it uninitialized, which would
sometimes lead to spurious error signals being emitted.
Task-number: QTBUG-44893
Change-Id: I7a17f1a16d9a5fe4218beb6d92993c1889ad836b
Reviewed-by: Oswald Buddenhagen <oswald.buddenhagen@theqtcompany.com>
Reviewed-by: Liang Qi <liang.qi@theqtcompany.com>
Diffstat (limited to 'tests/auto')
-rw-r--r-- | tests/auto/qwebsocket/tst_qwebsocket.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/tests/auto/qwebsocket/tst_qwebsocket.cpp b/tests/auto/qwebsocket/tst_qwebsocket.cpp index 900f316..7659e89 100644 --- a/tests/auto/qwebsocket/tst_qwebsocket.cpp +++ b/tests/auto/qwebsocket/tst_qwebsocket.cpp @@ -410,6 +410,7 @@ void tst_QWebSocket::tst_sendTextMessage() QSignalSpy textFrameReceived(&socket, SIGNAL(textFrameReceived(QString,bool))); QSignalSpy binaryMessageReceived(&socket, SIGNAL(binaryMessageReceived(QByteArray))); QSignalSpy binaryFrameReceived(&socket, SIGNAL(binaryFrameReceived(QByteArray,bool))); + QSignalSpy socketError(&socket, SIGNAL(error(QAbstractSocket::SocketError))); QUrl url = QUrl(QStringLiteral("ws://") + echoServer.hostAddress().toString() + QStringLiteral(":") + QString::number(echoServer.port())); @@ -419,6 +420,7 @@ void tst_QWebSocket::tst_sendTextMessage() if (socketConnectedSpy.count() == 0) QVERIFY(socketConnectedSpy.wait(500)); + QCOMPARE(socketError.count(), 0); QCOMPARE(socket.state(), QAbstractSocket::ConnectedState); socket.sendTextMessage(QStringLiteral("Hello world!")); |