diff options
author | Liang Qi <liang.qi@qt.io> | 2016-11-26 12:52:46 +0100 |
---|---|---|
committer | Miikka Heikkinen <miikka.heikkinen@qt.io> | 2016-11-28 11:23:46 +0200 |
commit | 38386fda213cd50e3a3284b69ae15d5f33a7e049 (patch) | |
tree | ef7c593801a54b0efa5c2107e28653037cd63f05 /tests | |
parent | 9b68fb59c7b4d2b2b273bf83fe25ac782f7a2d00 (diff) | |
parent | 01ec7b7ae4cdfabf2228911a96f57e2da8f333bb (diff) |
Merge remote-tracking branch 'origin/5.7' into 5.8
We needed to add SKIP_IF_FLAKY_MOUSE_MOVE(); line to
tst_QCandlestickSeries::mouseHovered() test case in this
merge, as SKIP_IF_FLAKY_MOUSE_MOVE was added by the merged
content and candlestick series is was introduced in 5.8,
so a separate commit in neither branch will work.
Conflicts:
src/charts/piechart/piechartitem.cpp
tests/auto/qcandlestickseries/tst_qcandlestickseries.cpp
Change-Id: I444306814678a9843460ec2e7fba78b616c7d8bd
Diffstat (limited to 'tests')
-rw-r--r-- | tests/auto/auto.pri | 1 | ||||
-rw-r--r-- | tests/auto/inc/tst_definitions.h | 10 | ||||
-rw-r--r-- | tests/auto/qareaseries/tst_qareaseries.cpp | 3 | ||||
-rw-r--r-- | tests/auto/qbarseries/tst_qbarseries.cpp | 1 | ||||
-rw-r--r-- | tests/auto/qcandlestickseries/tst_qcandlestickseries.cpp | 1 | ||||
-rw-r--r-- | tests/auto/qchartview/tst_qchartview.cpp | 3 | ||||
-rw-r--r-- | tests/auto/qhorizontalbarseries/tst_qhorizontalbarseries.cpp | 1 | ||||
-rw-r--r-- | tests/auto/qhorizontalpercentbarseries/tst_qhorizontalpercentbarseries.cpp | 1 | ||||
-rw-r--r-- | tests/auto/qhorizontalstackedbarseries/tst_qhorizontalstackedbarseries.cpp | 1 | ||||
-rw-r--r-- | tests/auto/qlegend/tst_qlegend.cpp | 3 | ||||
-rw-r--r-- | tests/auto/qml-qtquicktest/qml-qtquicktest.pro | 1 | ||||
-rw-r--r-- | tests/auto/qpercentbarseries/tst_qpercentbarseries.cpp | 1 | ||||
-rw-r--r-- | tests/auto/qpieseries/tst_qpieseries.cpp | 8 | ||||
-rw-r--r-- | tests/auto/qpieslice/tst_qpieslice.cpp | 1 | ||||
-rw-r--r-- | tests/auto/qstackedbarseries/tst_qstackedbarseries.cpp | 1 |
15 files changed, 28 insertions, 9 deletions
diff --git a/tests/auto/auto.pri b/tests/auto/auto.pri index a04f5b23..946b0e4a 100644 --- a/tests/auto/auto.pri +++ b/tests/auto/auto.pri @@ -2,6 +2,7 @@ error( "Couldn't find the tests.pri file!" ) } +CONFIG += testcase QT += testlib widgets !contains(TARGET, ^tst_.*):TARGET = $$join(TARGET,,"tst_") diff --git a/tests/auto/inc/tst_definitions.h b/tests/auto/inc/tst_definitions.h index e12dac18..a35d53e4 100644 --- a/tests/auto/inc/tst_definitions.h +++ b/tests/auto/inc/tst_definitions.h @@ -63,14 +63,16 @@ QT_END_NAMESPACE #define SKIP_IF_CANNOT_TEST_MOUSE_EVENTS() { \ do { \ QPushButton b; \ - b.resize(100, 100); \ + b.resize(120, 100); \ b.show(); \ QTest::qWaitForWindowShown(&b); \ QSignalSpy spy(&b, SIGNAL(clicked())); \ QTest::mouseClick(&b, Qt::LeftButton, 0, b.rect().center()); \ + QApplication::processEvents(); \ if (spy.count() == 0) \ QSKIP("Cannot test mouse events in this environment"); \ } while (0); \ + QApplication::processEvents(); \ } #define SKIP_ON_POLAR() { \ @@ -83,6 +85,12 @@ QT_END_NAMESPACE QSKIP("Test not supported by cartesian chart"); \ } +// Synthetic mouse moves do not trigger hover events reliably in many virtual machines, +// so we skip tests involving mouse moves. + #define SKIP_IF_FLAKY_MOUSE_MOVE() { \ + QSKIP("Skipping test with synthetic mouse moves."); \ + } + static inline bool isPolarTest() { static bool isPolar = false; diff --git a/tests/auto/qareaseries/tst_qareaseries.cpp b/tests/auto/qareaseries/tst_qareaseries.cpp index e03989c0..16fa94b3 100644 --- a/tests/auto/qareaseries/tst_qareaseries.cpp +++ b/tests/auto/qareaseries/tst_qareaseries.cpp @@ -78,6 +78,9 @@ void tst_QAreaSeries::init() m_axisY = new QValueAxis; m_axisX->setRange(0, 4); m_axisY->setRange(0, 10); + // Hide axes so they don't confuse color checks + m_axisX->setVisible(false); + m_axisY->setVisible(false); m_chart->addAxis(m_axisX, Qt::AlignBottom); m_chart->addAxis(m_axisY, Qt::AlignRight); } diff --git a/tests/auto/qbarseries/tst_qbarseries.cpp b/tests/auto/qbarseries/tst_qbarseries.cpp index 2eb4fe7d..8922af25 100644 --- a/tests/auto/qbarseries/tst_qbarseries.cpp +++ b/tests/auto/qbarseries/tst_qbarseries.cpp @@ -627,6 +627,7 @@ void tst_QBarSeries::mousehovered_data() void tst_QBarSeries::mousehovered() { SKIP_IF_CANNOT_TEST_MOUSE_EVENTS(); + SKIP_IF_FLAKY_MOUSE_MOVE(); QBarSeries* series = new QBarSeries(); diff --git a/tests/auto/qcandlestickseries/tst_qcandlestickseries.cpp b/tests/auto/qcandlestickseries/tst_qcandlestickseries.cpp index 89686a6f..ac056314 100644 --- a/tests/auto/qcandlestickseries/tst_qcandlestickseries.cpp +++ b/tests/auto/qcandlestickseries/tst_qcandlestickseries.cpp @@ -645,6 +645,7 @@ void tst_QCandlestickSeries::mouseClicked() void tst_QCandlestickSeries::mouseHovered() { SKIP_IF_CANNOT_TEST_MOUSE_EVENTS(); + SKIP_IF_FLAKY_MOUSE_MOVE(); QVERIFY(m_series->append(m_sets)); QCOMPARE(m_series->count(), m_sets.count()); diff --git a/tests/auto/qchartview/tst_qchartview.cpp b/tests/auto/qchartview/tst_qchartview.cpp index 4ad4d6f6..6a300391 100644 --- a/tests/auto/qchartview/tst_qchartview.cpp +++ b/tests/auto/qchartview/tst_qchartview.cpp @@ -143,6 +143,9 @@ void tst_QChartView::rubberBand_data() void tst_QChartView::rubberBand() { + SKIP_IF_CANNOT_TEST_MOUSE_EVENTS(); + SKIP_IF_FLAKY_MOUSE_MOVE(); + QFETCH(QChartView::RubberBands, rubberBand); QFETCH(int, Xcount); QFETCH(int, Ycount); diff --git a/tests/auto/qhorizontalbarseries/tst_qhorizontalbarseries.cpp b/tests/auto/qhorizontalbarseries/tst_qhorizontalbarseries.cpp index cc6ff8c4..d6e7976b 100644 --- a/tests/auto/qhorizontalbarseries/tst_qhorizontalbarseries.cpp +++ b/tests/auto/qhorizontalbarseries/tst_qhorizontalbarseries.cpp @@ -574,6 +574,7 @@ void tst_QHorizontalBarSeries::mousehovered_data() void tst_QHorizontalBarSeries::mousehovered() { SKIP_IF_CANNOT_TEST_MOUSE_EVENTS(); + SKIP_IF_FLAKY_MOUSE_MOVE(); QHorizontalBarSeries* series = new QHorizontalBarSeries(); diff --git a/tests/auto/qhorizontalpercentbarseries/tst_qhorizontalpercentbarseries.cpp b/tests/auto/qhorizontalpercentbarseries/tst_qhorizontalpercentbarseries.cpp index b432ceff..a7359288 100644 --- a/tests/auto/qhorizontalpercentbarseries/tst_qhorizontalpercentbarseries.cpp +++ b/tests/auto/qhorizontalpercentbarseries/tst_qhorizontalpercentbarseries.cpp @@ -321,6 +321,7 @@ void tst_QHorizontalPercentBarSeries::mousehovered_data() void tst_QHorizontalPercentBarSeries::mousehovered() { SKIP_IF_CANNOT_TEST_MOUSE_EVENTS(); + SKIP_IF_FLAKY_MOUSE_MOVE(); QHorizontalPercentBarSeries* series = new QHorizontalPercentBarSeries(); diff --git a/tests/auto/qhorizontalstackedbarseries/tst_qhorizontalstackedbarseries.cpp b/tests/auto/qhorizontalstackedbarseries/tst_qhorizontalstackedbarseries.cpp index a95f2aa7..8e5ad06c 100644 --- a/tests/auto/qhorizontalstackedbarseries/tst_qhorizontalstackedbarseries.cpp +++ b/tests/auto/qhorizontalstackedbarseries/tst_qhorizontalstackedbarseries.cpp @@ -325,6 +325,7 @@ void tst_QHorizontalStackedBarSeries::mousehovered_data() void tst_QHorizontalStackedBarSeries::mousehovered() { SKIP_IF_CANNOT_TEST_MOUSE_EVENTS(); + SKIP_IF_FLAKY_MOUSE_MOVE(); QHorizontalStackedBarSeries* series = new QHorizontalStackedBarSeries(); diff --git a/tests/auto/qlegend/tst_qlegend.cpp b/tests/auto/qlegend/tst_qlegend.cpp index 014c4e0d..605de863 100644 --- a/tests/auto/qlegend/tst_qlegend.cpp +++ b/tests/auto/qlegend/tst_qlegend.cpp @@ -506,8 +506,9 @@ void tst_QLegend::xyMarkerPropertiesScatter() void tst_QLegend::markerSignals() { SKIP_ON_POLAR(); - SKIP_IF_CANNOT_TEST_MOUSE_EVENTS(); + SKIP_IF_FLAKY_MOUSE_MOVE(); + QChart *chart = newQChartOrQPolarChart(); QLegend *legend = chart->legend(); diff --git a/tests/auto/qml-qtquicktest/qml-qtquicktest.pro b/tests/auto/qml-qtquicktest/qml-qtquicktest.pro index d9a1e74f..d4919c1d 100644 --- a/tests/auto/qml-qtquicktest/qml-qtquicktest.pro +++ b/tests/auto/qml-qtquicktest/qml-qtquicktest.pro @@ -4,6 +4,7 @@ TEMPLATE = app CONFIG += warn_on qmltestcase +CONFIG -= testcase SOURCES += main.cpp OTHER_FILES += tst_*.qml DEFINES += QUICK_TEST_SOURCE_DIR=\"\\\"$$PWD\\\"\" diff --git a/tests/auto/qpercentbarseries/tst_qpercentbarseries.cpp b/tests/auto/qpercentbarseries/tst_qpercentbarseries.cpp index b60ba92b..12c97b4b 100644 --- a/tests/auto/qpercentbarseries/tst_qpercentbarseries.cpp +++ b/tests/auto/qpercentbarseries/tst_qpercentbarseries.cpp @@ -323,6 +323,7 @@ void tst_QPercentBarSeries::mousehovered_data() void tst_QPercentBarSeries::mousehovered() { SKIP_IF_CANNOT_TEST_MOUSE_EVENTS(); + SKIP_IF_FLAKY_MOUSE_MOVE(); QPercentBarSeries* series = new QPercentBarSeries(); diff --git a/tests/auto/qpieseries/tst_qpieseries.cpp b/tests/auto/qpieseries/tst_qpieseries.cpp index 1a808385..b3fc6d12 100644 --- a/tests/auto/qpieseries/tst_qpieseries.cpp +++ b/tests/auto/qpieseries/tst_qpieseries.cpp @@ -563,6 +563,7 @@ void tst_qpieseries::hoverSignal() // Just for different signals. SKIP_IF_CANNOT_TEST_MOUSE_EVENTS(); + SKIP_IF_FLAKY_MOUSE_MOVE(); // add some slices m_series->append("slice 1", 1); @@ -576,13 +577,6 @@ void tst_qpieseries::hoverSignal() m_view->show(); QTest::qWaitForWindowShown(m_view); - // try to ensure focus - QApplication::setActiveWindow(m_view); - m_view->setFocus(); - QApplication::processEvents(); - QVERIFY(m_view->isActiveWindow()); - QVERIFY(m_view->hasFocus()); - // move inside the slices m_series->setPieSize(1.0); QRectF pieRect = m_view->chart()->plotArea(); diff --git a/tests/auto/qpieslice/tst_qpieslice.cpp b/tests/auto/qpieslice/tst_qpieslice.cpp index 913eb641..5e1dd099 100644 --- a/tests/auto/qpieslice/tst_qpieslice.cpp +++ b/tests/auto/qpieslice/tst_qpieslice.cpp @@ -279,6 +279,7 @@ void tst_qpieslice::hoverSignal() // Just for different signals. SKIP_IF_CANNOT_TEST_MOUSE_EVENTS(); + SKIP_IF_FLAKY_MOUSE_MOVE(); // add some slices QPieSeries *series = new QPieSeries(); diff --git a/tests/auto/qstackedbarseries/tst_qstackedbarseries.cpp b/tests/auto/qstackedbarseries/tst_qstackedbarseries.cpp index 8135fdd1..287106cd 100644 --- a/tests/auto/qstackedbarseries/tst_qstackedbarseries.cpp +++ b/tests/auto/qstackedbarseries/tst_qstackedbarseries.cpp @@ -325,6 +325,7 @@ void tst_QStackedBarSeries::mousehovered_data() void tst_QStackedBarSeries::mousehovered() { SKIP_IF_CANNOT_TEST_MOUSE_EVENTS(); + SKIP_IF_FLAKY_MOUSE_MOVE(); QStackedBarSeries* series = new QStackedBarSeries(); |