From c42f3dfa70212a601ef476e8258cbfc1caaf5477 Mon Sep 17 00:00:00 2001 From: Friedemann Kleint Date: Tue, 28 May 2019 09:15:51 +0200 Subject: Fix deprecation warnings about QtCharts::setAxisX(), setAxisY() Use new API, fixing warnings like: tst_qchart.cpp:343:34: warning: 'void QtCharts::QChart::setAxisX(QtCharts::QAbstractAxis*, QtCharts::QAbstractSeries*)' is deprecated [-Wdeprecated-declarations] ... tst_qchart.cpp:361:34: warning: 'void QtCharts::QChart::setAxisY(QtCharts::QAbstractAxis*, QtCharts::QAbstractSeries*)' is deprecated [-Wdeprecated-declarations] ... Some test appear to be relying on the axes being removed in a certain order. Change-Id: I86fab08d661afcd1087a55f847133a9e90d758f9 Reviewed-by: Miikka Heikkinen --- examples/charts/audio/widget.cpp | 6 ++- tests/auto/qabstractaxis/tst_qabstractaxis.cpp | 45 ++++++++++++++------- .../auto/qbarcategoryaxis/tst_qbarcategoryaxis.cpp | 36 +++++++++++------ tests/auto/qcategoryaxis/tst_qcategoryaxis.cpp | 15 ++++--- tests/auto/qchart/tst_qchart.cpp | 16 +++++--- tests/auto/qdatetimeaxis/tst_qdatetimeaxis.cpp | 6 ++- tests/auto/qlogvalueaxis/tst_qlogvalueaxis.cpp | 31 +++++++++++---- tests/auto/qvalueaxis/tst_qvalueaxis.cpp | 46 ++++++++++++++++------ 8 files changed, 141 insertions(+), 60 deletions(-) diff --git a/examples/charts/audio/widget.cpp b/examples/charts/audio/widget.cpp index 86dd6206..70d2387c 100644 --- a/examples/charts/audio/widget.cpp +++ b/examples/charts/audio/widget.cpp @@ -57,8 +57,10 @@ Widget::Widget(const QAudioDeviceInfo &deviceInfo, QWidget *parent) : QValueAxis *axisY = new QValueAxis; axisY->setRange(-1, 1); axisY->setTitleText("Audio level"); - m_chart->setAxisX(axisX, m_series); - m_chart->setAxisY(axisY, m_series); + m_chart->addAxis(axisX, Qt::AlignBottom); + m_series->attachAxis(axisX); + m_chart->addAxis(axisY, Qt::AlignLeft); + m_series->attachAxis(axisY); m_chart->legend()->hide(); m_chart->setTitle("Data from the microphone (" + deviceInfo.deviceName() + ')'); diff --git a/tests/auto/qabstractaxis/tst_qabstractaxis.cpp b/tests/auto/qabstractaxis/tst_qabstractaxis.cpp index d36af170..31a1b102 100644 --- a/tests/auto/qabstractaxis/tst_qabstractaxis.cpp +++ b/tests/auto/qabstractaxis/tst_qabstractaxis.cpp @@ -144,7 +144,8 @@ void tst_QAbstractAxis::axisPen() QCOMPARE(spy10.count(), 0); QCOMPARE(spy11.count(), 0); - m_chart->setAxisX(m_axis, m_series); + m_chart->addAxis(m_axis, Qt::AlignBottom); + m_series->attachAxis(m_axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); //TODO QCOMPARE(m_axis->axisPen(), axisPen); @@ -203,7 +204,8 @@ void tst_QAbstractAxis::gridLinePen() QCOMPARE(spy10.count(), 0); QCOMPARE(spy11.count(), 0); - m_chart->setAxisX(m_axis, m_series); + m_chart->addAxis(m_axis, Qt::AlignBottom); + m_series->attachAxis(m_axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); //TODO QCOMPARE(m_axis->gridLinePen(), gridLinePen); @@ -253,7 +255,8 @@ void tst_QAbstractAxis::minorGridLinePen() QCOMPARE(spy10.count(), 0); QCOMPARE(spy11.count(), 0); - m_chart->setAxisX(m_axis, m_series); + m_chart->addAxis(m_axis, Qt::AlignBottom); + m_series->attachAxis(m_axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); @@ -301,7 +304,8 @@ void tst_QAbstractAxis::lineVisible() QCOMPARE(spy10.count(), 0); QCOMPARE(spy11.count(), 0); - m_chart->setAxisX(m_axis, m_series); + m_chart->addAxis(m_axis, Qt::AlignBottom); + m_series->attachAxis(m_axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QCOMPARE(m_axis->isLineVisible(), lineVisible); @@ -345,7 +349,8 @@ void tst_QAbstractAxis::gridLineVisible() QCOMPARE(spy8.count(), 0); QCOMPARE(spy9.count(), 0); - m_chart->setAxisX(m_axis, m_series); + m_chart->addAxis(m_axis, Qt::AlignBottom); + m_series->attachAxis(m_axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QCOMPARE(m_axis->isGridLineVisible(), gridLineVisible); @@ -394,7 +399,8 @@ void tst_QAbstractAxis::minorGridLineVisible() QCOMPARE(spy10.count(), 0); QCOMPARE(spy11.count(), 0); - m_chart->setAxisX(m_axis, m_series); + m_chart->addAxis(m_axis, Qt::AlignBottom); + m_series->attachAxis(m_axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QCOMPARE(m_axis->isMinorGridLineVisible(), minorGridLineVisible); @@ -442,7 +448,8 @@ void tst_QAbstractAxis::gridLineColor() QCOMPARE(spy10.count(), 1); QCOMPARE(spy11.count(), 0); - m_chart->setAxisX(m_axis, m_series); + m_chart->addAxis(m_axis, Qt::AlignBottom); + m_series->attachAxis(m_axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); @@ -489,7 +496,8 @@ void tst_QAbstractAxis::minorGridLineColor() QCOMPARE(spy10.count(), 0); QCOMPARE(spy11.count(), 1); - m_chart->setAxisX(m_axis, m_series); + m_chart->addAxis(m_axis, Qt::AlignBottom); + m_series->attachAxis(m_axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); @@ -537,7 +545,8 @@ void tst_QAbstractAxis::visible() QCOMPARE(spy10.count(), 0); QCOMPARE(spy11.count(), 0); - m_chart->setAxisX(m_axis, m_series); + m_chart->addAxis(m_axis, Qt::AlignBottom); + m_series->attachAxis(m_axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QCOMPARE(m_axis->isVisible(), visible); @@ -584,7 +593,8 @@ void tst_QAbstractAxis::labelsAngle() QCOMPARE(spy10.count(), 0); QCOMPARE(spy11.count(), 0); - m_chart->setAxisX(m_axis, m_series); + m_chart->addAxis(m_axis, Qt::AlignBottom); + m_series->attachAxis(m_axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QCOMPARE(m_axis->labelsAngle(), labelsAngle); @@ -738,7 +748,8 @@ void tst_QAbstractAxis::labelsVisible() QCOMPARE(spy10.count(), 0); QCOMPARE(spy11.count(), 0); - m_chart->setAxisX(m_axis, m_series); + m_chart->addAxis(m_axis, Qt::AlignBottom); + m_series->attachAxis(m_axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QCOMPARE(m_axis->labelsVisible(), labelsVisible); @@ -769,9 +780,11 @@ void tst_QAbstractAxis::orientation() QSignalSpy spy11(m_axis, SIGNAL(minorGridLineColorChanged(QColor))); if(orientation==Qt::Vertical){ - m_chart->setAxisY(m_axis,m_series); + m_chart->addAxis(m_axis, Qt::AlignLeft); + m_series->attachAxis(m_axis); }else{ - m_chart->setAxisX(m_axis,m_series); + m_chart->addAxis(m_axis, Qt::AlignBottom); + m_series->attachAxis(m_axis); } QCOMPARE(m_axis->orientation(), orientation); @@ -948,7 +961,8 @@ void tst_QAbstractAxis::shadesPen() QCOMPARE(spy10.count(), 0); QCOMPARE(spy11.count(), 0); - m_chart->setAxisX(m_axis, m_series); + m_chart->addAxis(m_axis, Qt::AlignBottom); + m_series->attachAxis(m_axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QCOMPARE(m_axis->shadesPen(), shadesPen); @@ -996,7 +1010,8 @@ void tst_QAbstractAxis::shadesVisible() QCOMPARE(spy10.count(), 0); QCOMPARE(spy11.count(), 0); - m_chart->setAxisX(m_axis, m_series); + m_chart->addAxis(m_axis, Qt::AlignBottom); + m_series->attachAxis(m_axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QCOMPARE(m_axis->shadesVisible(), shadesVisible); diff --git a/tests/auto/qbarcategoryaxis/tst_qbarcategoryaxis.cpp b/tests/auto/qbarcategoryaxis/tst_qbarcategoryaxis.cpp index 0914706c..40294eb7 100644 --- a/tests/auto/qbarcategoryaxis/tst_qbarcategoryaxis.cpp +++ b/tests/auto/qbarcategoryaxis/tst_qbarcategoryaxis.cpp @@ -183,7 +183,8 @@ void tst_QBarCategoriesAxis::append() QCOMPARE(spy2.count(), 1); QCOMPARE(spy3.count(), 1); - m_chart->setAxisX(&axis, m_series); + m_chart->addAxis(&axis, Qt::AlignBottom); + m_series->attachAxis(&axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QCOMPARE(axis.categories(), categories); @@ -222,7 +223,8 @@ void tst_QBarCategoriesAxis::append2() QCOMPARE(spy2.count(), 1); QCOMPARE(spy3.count(), categories.count()); - m_chart->setAxisX(&axis, m_series); + m_chart->addAxis(&axis, Qt::AlignBottom); + m_series->attachAxis(&axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QCOMPARE(axis.categories(), categories); @@ -266,7 +268,8 @@ void tst_QBarCategoriesAxis::at() QCOMPARE(spy2.count(), 0); QCOMPARE(spy3.count(), 0); - m_chart->setAxisX(&axis, m_series); + m_chart->addAxis(&axis, Qt::AlignBottom); + m_series->attachAxis(&axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QCOMPARE(axis.at(index), string); @@ -304,7 +307,8 @@ void tst_QBarCategoriesAxis::categories() QCOMPARE(spy2.count(), 1); QCOMPARE(spy3.count(), 1); - m_chart->setAxisX(&axis, m_series); + m_chart->addAxis(&axis, Qt::AlignBottom); + m_series->attachAxis(&axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QCOMPARE(axis.categories(), categories); @@ -345,7 +349,8 @@ void tst_QBarCategoriesAxis::clear() QCOMPARE(spy2.count(), 1); QCOMPARE(spy3.count(), 1); - m_chart->setAxisX(&axis, m_series); + m_chart->addAxis(&axis, Qt::AlignBottom); + m_series->attachAxis(&axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); @@ -393,7 +398,8 @@ void tst_QBarCategoriesAxis::count() QCOMPARE(spy3.count(), 0); QCOMPARE(spy4.count(), 0); - m_chart->setAxisX(&axis, m_series); + m_chart->addAxis(&axis, Qt::AlignBottom); + m_series->attachAxis(&axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QCOMPARE(axis.count(), count); @@ -432,7 +438,8 @@ void tst_QBarCategoriesAxis::insert() QCOMPARE(spy3.count(), 0); QCOMPARE(spy4.count(), 1); - m_chart->setAxisX(&axis, m_series); + m_chart->addAxis(&axis, Qt::AlignBottom); + m_series->attachAxis(&axis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); } @@ -511,7 +518,8 @@ void tst_QBarCategoriesAxis::max_data() void tst_QBarCategoriesAxis::max() { - m_chart->setAxisX(m_baraxis, m_series); + m_chart->addAxis(m_baraxis, Qt::AlignBottom); + m_series->attachAxis(m_baraxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); max_raw(); @@ -565,7 +573,8 @@ void tst_QBarCategoriesAxis::min_data() void tst_QBarCategoriesAxis::min() { min_raw(); - m_chart->setAxisX(m_baraxis, m_series); + m_chart->addAxis(m_baraxis, Qt::AlignBottom); + m_series->attachAxis(m_baraxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); @@ -623,7 +632,8 @@ void tst_QBarCategoriesAxis::range_data() void tst_QBarCategoriesAxis::range() { range_raw(); - m_chart->setAxisX(m_baraxis, m_series); + m_chart->addAxis(m_baraxis, Qt::AlignBottom); + m_series->attachAxis(m_baraxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); @@ -667,7 +677,8 @@ void tst_QBarCategoriesAxis::noautoscale() QCOMPARE(spy1.count(), 1); QCOMPARE(spy2.count(), 1); - m_chart->setAxisX(m_baraxis, m_series); + m_chart->addAxis(m_baraxis, Qt::AlignBottom); + m_series->attachAxis(m_baraxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QCOMPARE(m_baraxis->min(),min); @@ -690,7 +701,8 @@ void tst_QBarCategoriesAxis::autoscale() QCOMPARE(m_baraxis->min(),QString()); QCOMPARE(m_baraxis->max(),QString()); - m_chart->setAxisX(m_baraxis, m_series); + m_chart->addAxis(m_baraxis, Qt::AlignBottom); + m_series->attachAxis(m_baraxis); QCOMPARE(spy0.count(), 1); QCOMPARE(spy1.count(), 1); diff --git a/tests/auto/qcategoryaxis/tst_qcategoryaxis.cpp b/tests/auto/qcategoryaxis/tst_qcategoryaxis.cpp index 2a33d0a3..063db211 100644 --- a/tests/auto/qcategoryaxis/tst_qcategoryaxis.cpp +++ b/tests/auto/qcategoryaxis/tst_qcategoryaxis.cpp @@ -116,7 +116,8 @@ void tst_QCategoryAxis::qcategoryaxis() QCOMPARE(m_categoryaxis->type(), QAbstractAxis::AxisTypeCategory); QCOMPARE(m_categoryaxis->labelsPosition(), QCategoryAxis::AxisLabelsPositionCenter); - m_chart->setAxisX(m_categoryaxis, m_series); + m_chart->addAxis(m_categoryaxis, Qt::AlignBottom); + m_series->attachAxis(m_categoryaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); @@ -157,7 +158,8 @@ void tst_QCategoryAxis::max_data() void tst_QCategoryAxis::max() { - m_chart->setAxisX(m_categoryaxis, m_series); + m_chart->addAxis(m_categoryaxis, Qt::AlignBottom); + m_series->attachAxis(m_categoryaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); max_raw(); @@ -205,7 +207,8 @@ void tst_QCategoryAxis::min_data() void tst_QCategoryAxis::min() { - m_chart->setAxisX(m_categoryaxis, m_series); + m_chart->addAxis(m_categoryaxis, Qt::AlignBottom); + m_series->attachAxis(m_categoryaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); min_raw(); @@ -259,7 +262,8 @@ void tst_QCategoryAxis::range_data() void tst_QCategoryAxis::range() { - m_chart->setAxisX(m_categoryaxis, m_series); + m_chart->addAxis(m_categoryaxis, Qt::AlignBottom); + m_series->attachAxis(m_categoryaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); range_raw(); @@ -335,7 +339,8 @@ void tst_QCategoryAxis::reverse() m_categoryaxis->setReverse(); QCOMPARE(m_categoryaxis->isReverse(), true); - m_chart->setAxisX(m_categoryaxis, m_series); + m_chart->addAxis(m_categoryaxis, Qt::AlignBottom); + m_series->attachAxis(m_categoryaxis); QCOMPARE(spy.count(), 1); m_view->show(); diff --git a/tests/auto/qchart/tst_qchart.cpp b/tests/auto/qchart/tst_qchart.cpp index b30c3322..49313db8 100644 --- a/tests/auto/qchart/tst_qchart.cpp +++ b/tests/auto/qchart/tst_qchart.cpp @@ -340,7 +340,8 @@ void tst_QChart::axisX() m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); m_chart->addSeries(series); - m_chart->setAxisX(axis,series); + m_chart->addAxis(axis, Qt::AlignBottom); + series->attachAxis(axis); QCOMPARE(m_chart->axes(Qt::Horizontal, series).value(0), axis); } @@ -358,7 +359,8 @@ void tst_QChart::axisY() m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); m_chart->addSeries(series); - m_chart->setAxisY(axis,series); + m_chart->addAxis(axis, Qt::AlignLeft); + series->attachAxis(axis); QCOMPARE(m_chart->axes(Qt::Vertical, series).value(0), axis); } @@ -574,7 +576,8 @@ void tst_QChart::removeSeries() axis = m_chart->axes(Qt::Vertical).value(0); QVERIFY(axis); m_chart->addSeries(series); - m_chart->setAxisY(axis,series); + m_chart->addAxis(axis, Qt::AlignLeft); + series->attachAxis(axis); QCOMPARE(m_chart->axes(Qt::Vertical, series).value(0), axis); m_chart->removeSeries(series); QCOMPARE(m_chart->axes().count(), 1); @@ -1167,8 +1170,11 @@ void tst_QChart::zoomInAndOut() dateTimeAxisY->setRange(QDateTime::fromMSecsSinceEpoch(minY), QDateTime::fromMSecsSinceEpoch(maxY)); } - m_chart->setAxisX(axisX, m_chart->series().first()); - m_chart->setAxisY(axisY, m_chart->series().first()); + const auto series = m_chart->series().constFirst(); + m_chart->addAxis(axisX, Qt::AlignBottom); + series->attachAxis(axisX); + m_chart->addAxis(axisY, Qt::AlignLeft); + series->attachAxis(axisY); CHECK_AXIS_RANGES_MATCH m_chart->zoomIn(); diff --git a/tests/auto/qdatetimeaxis/tst_qdatetimeaxis.cpp b/tests/auto/qdatetimeaxis/tst_qdatetimeaxis.cpp index 1a5f2c27..13b6fbdf 100644 --- a/tests/auto/qdatetimeaxis/tst_qdatetimeaxis.cpp +++ b/tests/auto/qdatetimeaxis/tst_qdatetimeaxis.cpp @@ -94,8 +94,10 @@ void tst_QDateTimeAxis::init() m_view->resize(200, 200); m_chart = m_view->chart(); m_chart->addSeries(m_series); - m_chart->setAxisY(m_dateTimeAxisY, m_series); - m_chart->setAxisX(m_dateTimeAxisX, m_series); + m_chart->addAxis(m_dateTimeAxisY, Qt::AlignLeft); + m_series->attachAxis(m_dateTimeAxisY); + m_chart->addAxis(m_dateTimeAxisX, Qt::AlignBottom); + m_series->attachAxis(m_dateTimeAxisX); } void tst_QDateTimeAxis::cleanup() diff --git a/tests/auto/qlogvalueaxis/tst_qlogvalueaxis.cpp b/tests/auto/qlogvalueaxis/tst_qlogvalueaxis.cpp index ba987670..fae92837 100644 --- a/tests/auto/qlogvalueaxis/tst_qlogvalueaxis.cpp +++ b/tests/auto/qlogvalueaxis/tst_qlogvalueaxis.cpp @@ -106,7 +106,12 @@ void tst_QLogValueAxis::qlogvalueaxis() QCOMPARE(m_logvaluesaxis->min(), (qreal)1); QCOMPARE(m_logvaluesaxis->type(), QAbstractAxis::AxisTypeLogValue); - m_chart->setAxisX(m_logvaluesaxis, m_series); + const auto oldXAxes = m_chart->axes(Qt::Horizontal); + for (auto oldXAxis : oldXAxes) + m_chart->removeAxis(oldXAxis); + + m_chart->addAxis(m_logvaluesaxis, Qt::AlignBottom); + m_series->attachAxis(m_logvaluesaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); @@ -159,7 +164,8 @@ void tst_QLogValueAxis::max_data() void tst_QLogValueAxis::max() { - m_chart->setAxisX(m_logvaluesaxis, m_series); + m_chart->addAxis(m_logvaluesaxis, Qt::AlignBottom); + m_series->attachAxis(m_logvaluesaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); max_raw(); @@ -214,7 +220,8 @@ void tst_QLogValueAxis::min_data() void tst_QLogValueAxis::min() { - m_chart->setAxisX(m_logvaluesaxis, m_series); + m_chart->addAxis(m_logvaluesaxis, Qt::AlignBottom); + m_series->attachAxis(m_logvaluesaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); min_raw(); @@ -278,7 +285,8 @@ void tst_QLogValueAxis::range_data() void tst_QLogValueAxis::range() { - m_chart->setAxisX(m_logvaluesaxis, m_series); + m_chart->addAxis(m_logvaluesaxis, Qt::AlignBottom); + m_series->attachAxis(m_logvaluesaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); range_raw(); @@ -293,7 +301,8 @@ void tst_QLogValueAxis::range_before_show() { range_raw(); - m_chart->setAxisX(m_logvaluesaxis, m_series); + m_chart->addAxis(m_logvaluesaxis, Qt::AlignBottom); + m_series->attachAxis(m_logvaluesaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); } @@ -352,7 +361,11 @@ void tst_QLogValueAxis::autoscale() QCOMPARE(m_logvaluesaxis->min(), (qreal)1); QCOMPARE(m_logvaluesaxis->max(), (qreal)1); - m_chart->setAxisX(m_logvaluesaxis, m_series); + const auto oldXAxes = m_chart->axes(Qt::Horizontal); + for (auto oldXAxis : oldXAxes) + m_chart->removeAxis(oldXAxis); + m_chart->addAxis(m_logvaluesaxis, Qt::AlignBottom); + m_series->attachAxis(m_logvaluesaxis); QCOMPARE(spy0.count(), 1); QCOMPARE(spy1.count(), 0); @@ -366,7 +379,11 @@ void tst_QLogValueAxis::autoscale() void tst_QLogValueAxis::zoom() { - m_chart->setAxisX(m_logvaluesaxis, m_series); + const auto oldXAxes = m_chart->axes(Qt::Horizontal); + for (auto oldXAxis : oldXAxes) + m_chart->removeAxis(oldXAxis); + m_chart->addAxis(m_logvaluesaxis, Qt::AlignBottom); + m_series->attachAxis(m_logvaluesaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); m_logvaluesaxis->setBase(2); diff --git a/tests/auto/qvalueaxis/tst_qvalueaxis.cpp b/tests/auto/qvalueaxis/tst_qvalueaxis.cpp index d1dabd3e..7bedee0b 100644 --- a/tests/auto/qvalueaxis/tst_qvalueaxis.cpp +++ b/tests/auto/qvalueaxis/tst_qvalueaxis.cpp @@ -73,6 +73,8 @@ private slots: void dynamicTicks(); private: + void removeAxes(); + QValueAxis* m_valuesaxis; QLineSeries* m_series; }; @@ -89,6 +91,7 @@ void tst_QValueAxis::init() void tst_QValueAxis::cleanup() { + removeAxes(); delete m_series; delete m_valuesaxis; m_series = 0; @@ -96,6 +99,13 @@ void tst_QValueAxis::cleanup() tst_QAbstractAxis::cleanup(); } +void tst_QValueAxis::removeAxes() +{ + const auto oldAxes = m_chart->axes(Qt::Horizontal) + m_chart->axes(Qt::Vertical); + for (auto oldAxis : oldAxes) + m_chart->removeAxis(oldAxis); +} + void tst_QValueAxis::qvalueaxis_data() { } @@ -109,7 +119,8 @@ void tst_QValueAxis::qvalueaxis() QCOMPARE(m_valuesaxis->tickCount(), 5); QCOMPARE(m_valuesaxis->type(), QAbstractAxis::AxisTypeValue); - m_chart->setAxisX(m_valuesaxis, m_series); + m_chart->addAxis(m_valuesaxis, Qt::AlignBottom); + m_series->attachAxis(m_valuesaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); @@ -152,7 +163,8 @@ void tst_QValueAxis::max_data() void tst_QValueAxis::max() { - m_chart->setAxisX(m_valuesaxis, m_series); + m_chart->addAxis(m_valuesaxis, Qt::AlignBottom); + m_series->attachAxis(m_valuesaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); max_raw(); @@ -200,7 +212,8 @@ void tst_QValueAxis::min_data() void tst_QValueAxis::min() { - m_chart->setAxisX(m_valuesaxis, m_series); + m_chart->addAxis(m_valuesaxis, Qt::AlignBottom); + m_series->attachAxis(m_valuesaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); min_raw(); @@ -304,7 +317,8 @@ void tst_QValueAxis::range_data() void tst_QValueAxis::range() { - m_chart->setAxisX(m_valuesaxis, m_series); + m_chart->addAxis(m_valuesaxis, Qt::AlignBottom); + m_series->attachAxis(m_valuesaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); range_raw(); @@ -347,7 +361,8 @@ void tst_QValueAxis::ticksCount() QCOMPARE(spy1.count(), 0); QCOMPARE(spy2.count(), 0); - m_chart->setAxisX(m_valuesaxis, m_series); + m_chart->addAxis(m_valuesaxis, Qt::AlignBottom); + m_series->attachAxis(m_valuesaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); @@ -380,7 +395,8 @@ void tst_QValueAxis::noautoscale() QCOMPARE(spy1.count(), 1); QCOMPARE(spy2.count(), 1); - m_chart->setAxisX(m_valuesaxis, m_series); + m_chart->addAxis(m_valuesaxis, Qt::AlignBottom); + m_series->attachAxis(m_valuesaxis); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); QVERIFY2(qFuzzyCompare(m_valuesaxis->min(), min), "Min not equal"); @@ -400,7 +416,8 @@ void tst_QValueAxis::autoscale() QVERIFY2(qFuzzyCompare(m_valuesaxis->min(), 0), "Min not equal"); QVERIFY2(qFuzzyCompare(m_valuesaxis->max(), 0), "Max not equal"); - m_chart->setAxisX(m_valuesaxis, m_series); + m_chart->addAxis(m_valuesaxis, Qt::AlignBottom); + m_series->attachAxis(m_valuesaxis); QCOMPARE(spy0.count(), 1); QCOMPARE(spy1.count(), 1); @@ -420,7 +437,8 @@ void tst_QValueAxis::reverse() m_valuesaxis->setReverse(); QCOMPARE(m_valuesaxis->isReverse(), true); - m_chart->setAxisX(m_valuesaxis, m_series); + m_chart->addAxis(m_valuesaxis, Qt::AlignBottom); + m_series->attachAxis(m_valuesaxis); QCOMPARE(spy.count(), 1); m_view->show(); @@ -430,7 +448,8 @@ void tst_QValueAxis::reverse() void tst_QValueAxis::labels() { - m_chart->setAxisX(m_valuesaxis, m_series); + m_chart->addAxis(m_valuesaxis, Qt::AlignBottom); + m_series->attachAxis(m_valuesaxis); m_view->resize(300, 300); m_view->show(); QVERIFY(QTest::qWaitForWindowExposed(m_view)); @@ -465,9 +484,12 @@ void tst_QValueAxis::labels() void tst_QValueAxis::dynamicTicks() { + removeAxes(); // remove default axes created by init() QValueAxis *valuesaxis = new QValueAxis(); - m_chart->setAxisX(m_valuesaxis, m_series); - m_chart->setAxisY(valuesaxis, m_series); + m_chart->addAxis(m_valuesaxis, Qt::AlignBottom); + m_series->attachAxis(m_valuesaxis); + m_chart->addAxis(valuesaxis, Qt::AlignLeft); + m_series->attachAxis(valuesaxis); m_view->resize(400, 400); m_valuesaxis->setRange(-111.0, 111); m_valuesaxis->setTickType(QValueAxis::TicksDynamic); @@ -497,7 +519,7 @@ void tst_QValueAxis::dynamicTicks() } } - QCOMPARE(expectedList, observedStrings); + QCOMPARE(observedStrings, expectedList); } QTEST_MAIN(tst_QValueAxis) -- cgit v1.2.3