summaryrefslogtreecommitdiffstats
path: root/tests/baseline
diff options
context:
space:
mode:
authorVolker Hilsheimer <volker.hilsheimer@qt.io>2021-11-11 20:29:22 +0100
committerVolker Hilsheimer <volker.hilsheimer@qt.io>2021-11-18 03:36:24 +0000
commit79b2a43585cb409fea3699dff472ff48f058af5a (patch)
tree770d56ffc3d6305808d1b98cbcb34676249e4cf0 /tests/baseline
parent3b84187cd14b55f1456ebd9b195c7aeeb4ec7ef4 (diff)
Add baseline test that allows comparing the rendering of widgets
Provide basic boiler plate that sets up the baseline (aka lancelot) framework specifically for comparing the appearance of widgets, and implement test functions for QSlider and QPushButton. Widgets should always look the same if the QPA platform, the OS version, and certain UI-impacting attributes are identical. Ie. on any macOS 10.15 machine that runs in "Light" mode, widgets look the same. On a macOS 11 machine, they might look different. On an OpenSUSE machine using the fusion style things might look different from a Ubuntu machine. The helper function removes DPR differences - images are always scaled to a DPR of 1.0, which allows us to compare the image\ content and not get distracted by them having different dimensions (and the fuzzy comparison of images might make the system tolerate scaling artefacts). Note: For now, this test is meant to be run locally, either when testing changes to style code, or when checking how QWidget based UIs would look on newer version of an operating system. In CI the test is run, but then skipped in CI as the baseline server is not configured. Change-Id: Ie33a9d979d934f0df6883757333ce2c5e2f7ef84 Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo@qt.io>
Diffstat (limited to 'tests/baseline')
-rw-r--r--tests/baseline/CMakeLists.txt3
-rw-r--r--tests/baseline/widgets/CMakeLists.txt12
-rw-r--r--tests/baseline/widgets/tst_baseline_widgets.cpp289
3 files changed, 304 insertions, 0 deletions
diff --git a/tests/baseline/CMakeLists.txt b/tests/baseline/CMakeLists.txt
index 935df36ef9..28a1910fd2 100644
--- a/tests/baseline/CMakeLists.txt
+++ b/tests/baseline/CMakeLists.txt
@@ -1,3 +1,6 @@
if(TARGET Qt::Network)
add_subdirectory(painting)
endif()
+if(TARGET Qt::Network AND TARGET Qt::Widgets)
+ add_subdirectory(widgets)
+endif()
diff --git a/tests/baseline/widgets/CMakeLists.txt b/tests/baseline/widgets/CMakeLists.txt
new file mode 100644
index 0000000000..eac3ca6753
--- /dev/null
+++ b/tests/baseline/widgets/CMakeLists.txt
@@ -0,0 +1,12 @@
+qt_internal_add_test(tst_baseline_widgets
+ SOURCES
+ ../shared/baselineprotocol.cpp ../shared/baselineprotocol.h ../shared/lookup3.cpp
+ ../shared/qbaselinetest.cpp ../shared/qbaselinetest.h
+ tst_baseline_widgets.cpp
+ INCLUDE_DIRECTORIES
+ ../shared
+ PUBLIC_LIBRARIES
+ Qt::Gui
+ Qt::Widgets
+ Qt::Network
+)
diff --git a/tests/baseline/widgets/tst_baseline_widgets.cpp b/tests/baseline/widgets/tst_baseline_widgets.cpp
new file mode 100644
index 0000000000..13a6726008
--- /dev/null
+++ b/tests/baseline/widgets/tst_baseline_widgets.cpp
@@ -0,0 +1,289 @@
+/****************************************************************************
+**
+** Copyright (C) 2021 The Qt Company Ltd.
+** Contact: https://www.qt.io/licensing/
+**
+** This file is part of the test suite of the Qt Toolkit.
+**
+** $QT_BEGIN_LICENSE:GPL-EXCEPT$
+** Commercial License Usage
+** Licensees holding valid commercial Qt licenses may use this file in
+** accordance with the commercial license agreement provided with the
+** Software or, alternatively, in accordance with the terms contained in
+** a written agreement between you and The Qt Company. For licensing terms
+** and conditions see https://www.qt.io/terms-conditions. For further
+** information use the contact form at https://www.qt.io/contact-us.
+**
+** GNU General Public License Usage
+** Alternatively, this file may be used under the terms of the GNU
+** General Public License version 3 as published by the Free Software
+** Foundation with exceptions as appearing in the file LICENSE.GPL3-EXCEPT
+** included in the packaging of this file. Please review the following
+** information to ensure the GNU General Public License requirements will
+** be met: https://www.gnu.org/licenses/gpl-3.0.html.
+**
+** $QT_END_LICENSE$
+**
+****************************************************************************/
+
+#include <qbaselinetest.h>
+#include <QtWidgets>
+
+class tst_Widgets : public QObject
+{
+ Q_OBJECT
+
+public:
+ tst_Widgets();
+
+ void takeStandardSnapshots();
+ QWidget *testWindow() const { return window; }
+
+private slots:
+ void initTestCase();
+ void init();
+ void cleanup();
+
+ void tst_QSlider_data();
+ void tst_QSlider();
+
+ void tst_QPushButton_data();
+ void tst_QPushButton();
+
+private:
+ void makeVisible();
+ QImage takeSnapshot();
+
+ QWidget *window = nullptr;
+};
+
+tst_Widgets::tst_Widgets()
+{
+ QBaselineTest::addClientProperty("Project", "Widgets");
+
+ // Set key platform properties that are relevant for the appearance of widgets
+ const QString platformName = QGuiApplication::platformName() + "-" + QSysInfo::productType();
+ QBaselineTest::addClientProperty("PlatformName", platformName);
+ QBaselineTest::addClientProperty("OSVersion", QSysInfo::productVersion());
+
+ // Encode a number of parameters that impact the UI
+ QPalette palette;
+ QFont font;
+ QByteArray appearanceBytes;
+ {
+ QDataStream appearanceStream(&appearanceBytes, QIODevice::WriteOnly);
+ appearanceStream << palette << font << QApplication::style()->name();
+ const qreal screenDpr = QApplication::primaryScreen()->devicePixelRatio();
+ if (screenDpr != 1.0)
+ qWarning() << "DPR is" << screenDpr << "- images will be scaled";
+ }
+ const quint16 appearanceId = qChecksum(appearanceBytes);
+
+ // Assume that text that's darker than the background means we run in light mode
+ // This results in a more meaningful appearance ID between different runs than
+ // just the checksum of the various attributes.
+ const QColor windowColor = palette.window().color();
+ const QColor textColor = palette.text().color();
+ const QString appearanceIdString = (windowColor.value() > textColor.value()
+ ? QString("light-%1") : QString("dark-%1"))
+ .arg(appearanceId, 0, 16);
+ QBaselineTest::addClientProperty("AppearanceID", appearanceIdString);
+
+ // let users know where they can find the results
+ qDebug() << "PlatformName computed to be:" << platformName;
+ qDebug() << "Appearance ID computed as:" << appearanceIdString;
+}
+
+void tst_Widgets::initTestCase()
+{
+ // Check and setup the environment. Failure to do so skips the test.
+ QByteArray msg;
+ if (!QBaselineTest::connectToBaselineServer(&msg))
+ QSKIP(msg);
+}
+
+void tst_Widgets::init()
+{
+ QVERIFY(!window);
+ window = new QWidget;
+ window->setWindowTitle(QTest::currentDataTag());
+ window->setScreen(QGuiApplication::primaryScreen());
+ window->move(QGuiApplication::primaryScreen()->availableGeometry().topLeft());
+}
+
+void tst_Widgets::makeVisible()
+{
+ window->show();
+ window->window()->windowHandle()->requestActivate();
+ // explicitly unset focus, the test needs to control when focus is shown
+ if (window->focusWidget())
+ window->focusWidget()->clearFocus();
+ QVERIFY(QTest::qWaitForWindowActive(window));
+}
+
+/*
+ Always return images scaled to a DPR of 1.0.
+
+ This might produce some fuzzy differences, but lets us
+ compare those.
+*/
+QImage tst_Widgets::takeSnapshot()
+{
+ QGuiApplication::processEvents();
+ QPixmap pm = window->grab();
+ QTransform scaleTransform = QTransform::fromScale(1.0 / pm.devicePixelRatioF(), 1.0 / pm.devicePixelRatioF());
+ return pm.toImage().transformed(scaleTransform, Qt::SmoothTransformation);
+}
+
+/*!
+ Sets standard widget properties on the test window and its children,
+ and uploads snapshots. The widgets are returned in the same state
+ that they had before.
+
+ Call this helper after setting up the test window.
+*/
+void tst_Widgets::takeStandardSnapshots()
+{
+ makeVisible();
+ struct PublicWidget : QWidget { friend tst_Widgets; };
+
+ QBASELINE_CHECK(takeSnapshot(), "default");
+
+ // try hard to set focus
+ static_cast<PublicWidget*>(window)->focusNextPrevChild(true);
+ if (!window->focusWidget()) {
+ QWidget *firstChild = window->findChild<QWidget*>();
+ if (firstChild)
+ firstChild->setFocus();
+ }
+ if (window->focusWidget()) {
+ QBASELINE_CHECK(takeSnapshot(), "focused");
+ window->focusWidget()->clearFocus();
+ }
+
+ // this disables all children
+ window->setEnabled(false);
+ QBASELINE_CHECK(takeSnapshot(), "disabled");
+ window->setEnabled(true);
+
+ // show and activate another window so that our test window becomes inactive
+ QWidget otherWindow;
+ otherWindow.move(window->geometry().bottomRight() + QPoint(10, 10));
+ otherWindow.resize(50, 50);
+ otherWindow.setWindowFlags(Qt::CustomizeWindowHint | Qt::FramelessWindowHint);
+ otherWindow.show();
+ otherWindow.windowHandle()->requestActivate();
+ QVERIFY(QTest::qWaitForWindowActive(&otherWindow));
+ QBASELINE_CHECK(takeSnapshot(), "inactive");
+
+ window->windowHandle()->requestActivate();
+ QVERIFY(QTest::qWaitForWindowActive(window));
+ if (window->focusWidget())
+ window->focusWidget()->clearFocus();
+}
+
+void tst_Widgets::cleanup()
+{
+ delete window;
+ window = nullptr;
+}
+
+void tst_Widgets::tst_QSlider_data()
+{
+ QTest::addColumn<Qt::Orientation>("orientation");
+ QTest::addColumn<QSlider::TickPosition>("tickPosition");
+
+ QBaselineTest::newRow("horizontal") << Qt::Horizontal << QSlider::NoTicks;
+ QBaselineTest::newRow("horizontal ticks above") << Qt::Horizontal << QSlider::TicksAbove;
+ QBaselineTest::newRow("horizontal ticks below") << Qt::Horizontal << QSlider::TicksBelow;
+ QBaselineTest::newRow("horizontal ticks both") << Qt::Horizontal << QSlider::TicksBothSides;
+ QBaselineTest::newRow("vertical") << Qt::Vertical << QSlider::NoTicks;
+ QBaselineTest::newRow("vertical ticks left") << Qt::Vertical << QSlider::TicksLeft;
+ QBaselineTest::newRow("vertical ticks right") << Qt::Vertical << QSlider::TicksRight;
+ QBaselineTest::newRow("vertical ticks both") << Qt::Vertical << QSlider::TicksBothSides;
+}
+
+void tst_Widgets::tst_QSlider()
+{
+ struct PublicSlider : QSlider { friend tst_Widgets; };
+ QFETCH(Qt::Orientation, orientation);
+ QFETCH(QSlider::TickPosition, tickPosition);
+
+ QBoxLayout *box = new QBoxLayout(orientation == Qt::Horizontal ? QBoxLayout::TopToBottom
+ : QBoxLayout::LeftToRight);
+ QList<QSlider*> _sliders;
+ for (int i = 0; i < 3; ++i) {
+ QSlider *slider = new QSlider;
+ slider->setOrientation(orientation);
+ slider->setTickPosition(tickPosition);
+ _sliders << slider;
+ box->addWidget(slider);
+ }
+ const auto sliders = _sliders;
+
+ window->setLayout(box);
+
+ // we want to see sliders with different values
+ int value = 0;
+ for (const auto &slider : sliders)
+ slider->setValue(value += 33);
+
+ takeStandardSnapshots();
+
+ PublicSlider *slider = static_cast<PublicSlider*>(sliders.first());
+ QStyleOptionSlider sliderOptions;
+ slider->initStyleOption(&sliderOptions);
+ const QRect handleRect = slider->style()->subControlRect(QStyle::CC_Slider, &sliderOptions,
+ QStyle::SubControl::SC_SliderHandle, slider);
+ QTest::mousePress(slider, Qt::LeftButton, {}, handleRect.center());
+ QBASELINE_CHECK(takeSnapshot(), "pressed");
+ QTest::mouseRelease(slider, Qt::LeftButton, {}, handleRect.center());
+ QBASELINE_CHECK(takeSnapshot(), "released");
+
+ slider->setSliderDown(true);
+ QBASELINE_CHECK(takeSnapshot(), "down");
+
+ sliders.first()->setSliderDown(false);
+ QBASELINE_CHECK(takeSnapshot(), "notdown");
+}
+
+void tst_Widgets::tst_QPushButton_data()
+{
+ QTest::addColumn<bool>("isFlat");
+
+ QBaselineTest::newRow("normal") << false;
+ QBaselineTest::newRow("flat") << true;
+}
+
+void tst_Widgets::tst_QPushButton()
+{
+ QFETCH(bool, isFlat);
+
+ QVBoxLayout *vbox = new QVBoxLayout;
+ QPushButton *testButton = new QPushButton("Ok");
+ testButton->setFlat(isFlat);
+ vbox->addWidget(testButton);
+
+ testWindow()->setLayout(vbox);
+ takeStandardSnapshots();
+
+ testButton->setDown(true);
+ QBASELINE_CHECK(takeSnapshot(), "down");
+ testButton->setDown(false);
+ QBASELINE_CHECK(takeSnapshot(), "up");
+}
+
+
+#define main _realmain
+QTEST_MAIN(tst_Widgets)
+#undef main
+
+int main(int argc, char *argv[])
+{
+ qSetGlobalQHashSeed(0); // Avoid rendering variations caused by QHash randomization
+
+ QBaselineTest::handleCmdLineArgs(&argc, &argv);
+ return _realmain(argc, argv);
+}
+
+#include "tst_baseline_widgets.moc"