Skip to content

Commit

Permalink
QmlDebuggingTests: Separate out public and private tests
Browse files Browse the repository at this point in the history
Remove dependencies on quick-private from all
possible debugging auto tests and list them under public
tests.

Change-Id: I688b5b36fdf3d6fbcb6cef2a975ecd1bf679af2b
Reviewed-by: Kai Koehne <kai.koehne@nokia.com>
  • Loading branch information
Aurindam Jana authored and Qt by Nokia committed Mar 19, 2012
1 parent c975f80 commit 5c05f5d
Show file tree
Hide file tree
Showing 32 changed files with 1,000 additions and 187 deletions.
8 changes: 6 additions & 2 deletions src/plugins/qmltooling/qmldbg_tcp/qmldbg_tcp.pro
Expand Up @@ -9,10 +9,14 @@ DESTDIR = $$QT.qml.plugins/qmltooling
QTDIR_build:REQUIRES += "contains(QT_CONFIG, qml)"

SOURCES += \
qtcpserverconnection.cpp
qtcpserverconnection.cpp \
../shared/qpacketprotocol.cpp

HEADERS += \
qtcpserverconnection.h
qtcpserverconnection.h \
../shared/qpacketprotocol.h

INCLUDEPATH += ../shared

OTHER_FILES += qtcpserverconnection.json

Expand Down
2 changes: 1 addition & 1 deletion src/plugins/qmltooling/qmldbg_tcp/qtcpserverconnection.cpp
Expand Up @@ -40,13 +40,13 @@
****************************************************************************/

#include "qtcpserverconnection.h"
#include "qpacketprotocol.h"

#include <QtCore/qplugin.h>
#include <QtNetwork/qtcpserver.h>
#include <QtNetwork/qtcpsocket.h>

#include <private/qqmldebugserver_p.h>
#include <private/qpacketprotocol_p.h>

QT_BEGIN_NAMESPACE

Expand Down
Expand Up @@ -39,7 +39,7 @@
**
****************************************************************************/

#include "qpacketprotocol_p.h"
#include "qpacketprotocol.h"

#include <QtCore/QBuffer>
#include <QtCore/QElapsedTimer>
Expand Down
Expand Up @@ -42,34 +42,18 @@
#ifndef QPACKETPROTOCOL_H
#define QPACKETPROTOCOL_H

//
// W A R N I N G
// -------------
//
// This file is not part of the Qt API. It exists purely as an
// implementation detail. This header file may change from version to
// version without notice, or even be removed.
//
// We mean it.
//

#include <QtCore/qobject.h>
#include <QtCore/qdatastream.h>

#include <private/qtqmlglobal_p.h>

QT_BEGIN_HEADER

QT_BEGIN_NAMESPACE


class QIODevice;
class QBuffer;
class QPacket;
class QPacketAutoSend;
class QPacketProtocolPrivate;

class Q_QML_PRIVATE_EXPORT QPacketProtocol : public QObject
class QPacketProtocol : public QObject
{
Q_OBJECT
public:
Expand Down Expand Up @@ -101,7 +85,7 @@ class Q_QML_PRIVATE_EXPORT QPacketProtocol : public QObject
};


class Q_QML_PRIVATE_EXPORT QPacket : public QDataStream
class QPacket : public QDataStream
{
public:
QPacket();
Expand All @@ -119,7 +103,7 @@ class Q_QML_PRIVATE_EXPORT QPacket : public QDataStream
QBuffer *buf;
};

class Q_QML_PRIVATE_EXPORT QPacketAutoSend : public QPacket
class QPacketAutoSend : public QPacket
{
public:
virtual ~QPacketAutoSend();
Expand All @@ -132,6 +116,4 @@ class Q_QML_PRIVATE_EXPORT QPacketAutoSend : public QPacket

QT_END_NAMESPACE

QT_END_HEADER

#endif
2 changes: 0 additions & 2 deletions src/qml/debugger/debugger.pri
@@ -1,5 +1,4 @@
SOURCES += \
$$PWD/qpacketprotocol.cpp \
$$PWD/qqmldebugservice.cpp \
$$PWD/qqmlprofilerservice.cpp \
$$PWD/qqmldebugserver.cpp \
Expand All @@ -10,7 +9,6 @@ SOURCES += \
$$PWD/qdebugmessageservice.cpp

HEADERS += \
$$PWD/qpacketprotocol_p.h \
$$PWD/qqmldebugservice_p.h \
$$PWD/qqmldebugservice_p_p.h \
$$PWD/qqmlprofilerservice_p.h \
Expand Down
10 changes: 7 additions & 3 deletions tests/auto/qml/debugger/debugger.pro
@@ -1,16 +1,20 @@
TEMPLATE = subdirs

PRIVATETESTS += \
PUBLICTESTS += \
qqmlenginedebugservice \
qqmldebugclient \
qqmldebugservice \
qqmldebugjs \
qqmlinspector \
qqmlprofilerservice \
qpacketprotocol \
qv8profilerservice \
qdebugmessageservice

PRIVATETESTS += \
qqmldebugclient \
qqmldebugservice

SUBDIRS += $$PUBLICTESTS

contains(QT_CONFIG, private_tests) {
SUBDIRS += $$PRIVATETESTS
}
@@ -1,6 +1,6 @@
CONFIG += testcase
TARGET = tst_qdebugmessageservice
QT += network qml-private testlib
QT += qml network testlib
macx:CONFIG -= app_bundle

SOURCES += tst_qdebugmessageservice.cpp
Expand Down
Expand Up @@ -8,4 +8,4 @@ INCLUDEPATH += ../shared
include(../shared/debugutil.pri)

CONFIG += parallel_test
QT += qml-private network testlib
QT += qml network testlib
Expand Up @@ -46,7 +46,7 @@
#include <QDebug>
#include <QBuffer>

#include <private/qpacketprotocol_p.h>
#include "../../../../../src/plugins/qmltooling/shared/qpacketprotocol.h"

#include "debugutil_p.h"

Expand Down
5 changes: 4 additions & 1 deletion tests/auto/qml/debugger/qqmldebugclient/qqmldebugclient.pro
Expand Up @@ -2,7 +2,10 @@ CONFIG += testcase
TARGET = tst_qqmldebugclient
macx:CONFIG -= app_bundle

SOURCES += tst_qqmldebugclient.cpp
HEADERS += ../shared/qqmldebugtestservice.h

SOURCES += tst_qqmldebugclient.cpp \
../shared/qqmldebugtestservice.cpp

INCLUDEPATH += ../shared
include(../shared/debugutil.pri)
Expand Down
Expand Up @@ -48,6 +48,7 @@
#include <QtQml/qqmlengine.h>

#include "debugutil_p.h"
#include "qqmldebugtestservice.h"

#define PORT 13770
#define STR_PORT "13770"
Expand Down
2 changes: 1 addition & 1 deletion tests/auto/qml/debugger/qqmldebugjs/qqmldebugjs.pro
@@ -1,6 +1,6 @@
CONFIG += testcase
TARGET = tst_qqmldebugjs
QT += qml-private testlib
QT += qml testlib
macx:CONFIG -= app_bundle

SOURCES += tst_qqmldebugjs.cpp
Expand Down
Expand Up @@ -2,7 +2,11 @@ CONFIG += testcase
TARGET = tst_qqmldebugservice
macx:CONFIG -= app_bundle

SOURCES += tst_qqmldebugservice.cpp
HEADERS += ../shared/qqmldebugtestservice.h

SOURCES += tst_qqmldebugservice.cpp \
../shared/qqmldebugtestservice.cpp

INCLUDEPATH += ../shared
include(../../../shared/util.pri)
include(../shared/debugutil.pri)
Expand Down
Expand Up @@ -47,11 +47,10 @@

#include <QtQml/qqmlengine.h>

#include <private/qqmldebugservice_p.h>

#include "../../../shared/util.h"
#include "debugutil_p.h"
#include "qqmldebugclient.h"
#include "qqmldebugtestservice.h"

#define PORT 13769
#define STR_PORT "13769"
Expand Down
Expand Up @@ -9,4 +9,4 @@ include(../shared/debugutil.pri)

CONFIG += parallel_test declarative_debug

QT += qml-private testlib
QT += qml testlib
Expand Up @@ -12,4 +12,4 @@ OTHER_FILES += data/test.qml

CONFIG += parallel_test declarative_debug

QT += core-private v8-private qml-private testlib
QT += core qml testlib

0 comments on commit 5c05f5d

Please sign in to comment.