Skip to content

Commit

Permalink
Add baseline test that allows comparing the rendering of widgets
Browse files Browse the repository at this point in the history
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ø <[email protected]>
  • Loading branch information
vohi committed Nov 18, 2021
1 parent 3b84187 commit 79b2a43
Show file tree
Hide file tree
Showing 3 changed files with 304 additions and 0 deletions.
3 changes: 3 additions & 0 deletions tests/baseline/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
if(TARGET Qt::Network)
add_subdirectory(painting)
endif()
if(TARGET Qt::Network AND TARGET Qt::Widgets)
add_subdirectory(widgets)
endif()
12 changes: 12 additions & 0 deletions tests/baseline/widgets/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -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
)
289 changes: 289 additions & 0 deletions tests/baseline/widgets/tst_baseline_widgets.cpp
Original file line number Diff line number Diff line change
@@ -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"

0 comments on commit 79b2a43

Please sign in to comment.