diff options
author | Friedemann Kleint <Friedemann.Kleint@nokia.com> | 2012-09-07 10:32:26 +0200 |
---|---|---|
committer | Qt by Nokia <qt-info@nokia.com> | 2012-09-07 12:01:47 +0200 |
commit | 20563c1334312604a7a7fb1978e0822a78e3b90f (patch) | |
tree | 97d4922923d041298f765cac3506a768179d4736 /tests/auto/widgets/widgets/qspinbox | |
parent | 75cd2dec71d81868e1c4d5df32eb602faeb165fc (diff) |
Stabilize test of QSpinBox.
The test shows spurious failures in editingFinished() on some Mac
platforms (QTest::qWaitForWindowActive(testFocusWidget)).
This is apparently caused by the widget testFocusWidget (member
variable) interfering with the other tests widgets.
As it is used in one test only, instantiate it on the stack there.
Change-Id: I688cd21a2668d072660658302cf59197abe0b4d8
Reviewed-by: Qt Doc Bot <qt_docbot@qt-project.org>
Reviewed-by: Sergio Ahumada <sergio.ahumada@nokia.com>
Diffstat (limited to 'tests/auto/widgets/widgets/qspinbox')
-rw-r--r-- | tests/auto/widgets/widgets/qspinbox/tst_qspinbox.cpp | 50 |
1 files changed, 15 insertions, 35 deletions
diff --git a/tests/auto/widgets/widgets/qspinbox/tst_qspinbox.cpp b/tests/auto/widgets/widgets/qspinbox/tst_qspinbox.cpp index 32b7ffe2b0..a6054a9c52 100644 --- a/tests/auto/widgets/widgets/qspinbox/tst_qspinbox.cpp +++ b/tests/auto/widgets/widgets/qspinbox/tst_qspinbox.cpp @@ -91,11 +91,8 @@ class tst_QSpinBox : public QObject Q_OBJECT public: tst_QSpinBox(); - virtual ~tst_QSpinBox(); public slots: - void initTestCase(); void init(); - void cleanupTestCase(); private slots: void getSetCheck(); void setValue_data(); @@ -146,7 +143,6 @@ public slots: private: QStringList actualTexts; QList<int> actualValues; - QWidget *testFocusWidget; }; typedef QList<int> IntList; @@ -249,19 +245,6 @@ void tst_QSpinBox::getSetCheck() tst_QSpinBox::tst_QSpinBox() { - -} - -tst_QSpinBox::~tst_QSpinBox() -{ - -} - -void tst_QSpinBox::initTestCase() -{ - testFocusWidget = new QWidget(0); - testFocusWidget->resize(200, 100); - testFocusWidget->show(); } void tst_QSpinBox::init() @@ -269,12 +252,6 @@ void tst_QSpinBox::init() QLocale::setDefault(QLocale(QLocale::C)); } -void tst_QSpinBox::cleanupTestCase() -{ - delete testFocusWidget; - testFocusWidget = 0; -} - void tst_QSpinBox::setValue_data() { QTest::addColumn<int>("set"); @@ -738,15 +715,20 @@ void tst_QSpinBox::locale() void tst_QSpinBox::editingFinished() { - QVBoxLayout *layout = new QVBoxLayout(testFocusWidget); - QSpinBox *box = new QSpinBox(testFocusWidget); + QWidget testFocusWidget; + testFocusWidget.setObjectName(QLatin1String("tst_qspinbox")); + testFocusWidget.setWindowTitle(objectName()); + testFocusWidget.resize(200, 100); + + QVBoxLayout *layout = new QVBoxLayout(&testFocusWidget); + QSpinBox *box = new QSpinBox(&testFocusWidget); layout->addWidget(box); - QSpinBox *box2 = new QSpinBox(testFocusWidget); + QSpinBox *box2 = new QSpinBox(&testFocusWidget); layout->addWidget(box2); - testFocusWidget->show(); - QApplication::setActiveWindow(testFocusWidget); - QVERIFY(QTest::qWaitForWindowActive(testFocusWidget)); + testFocusWidget.show(); + QApplication::setActiveWindow(&testFocusWidget); + QVERIFY(QTest::qWaitForWindowActive(&testFocusWidget)); box->activateWindow(); box->setFocus(); @@ -788,18 +770,18 @@ void tst_QSpinBox::editingFinished() QCOMPARE(editingFinishedSpy1.count(), 4); QCOMPARE(editingFinishedSpy2.count(), 3); - testFocusWidget->hide(); + testFocusWidget.hide(); QCOMPARE(editingFinishedSpy1.count(), 4); QCOMPARE(editingFinishedSpy2.count(), 4); QTest::qWait(100); //task203285 editingFinishedSpy1.clear(); - testFocusWidget->show(); + testFocusWidget.show(); QTest::qWait(100); box->setKeyboardTracking(false); - qApp->setActiveWindow(testFocusWidget); - testFocusWidget->activateWindow(); + qApp->setActiveWindow(&testFocusWidget); + testFocusWidget.activateWindow(); box->setFocus(); QTRY_VERIFY(box->hasFocus()); box->setValue(0); @@ -809,8 +791,6 @@ void tst_QSpinBox::editingFinished() QTRY_VERIFY(qApp->focusWidget() != box); QCOMPARE(box->text(), QLatin1String("20")); QCOMPARE(editingFinishedSpy1.count(), 1); - - testFocusWidget->hide(); } void tst_QSpinBox::removeAll() |