From c1e1d1db2b8dd7ee5f249b17e9531c819e316a25 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Martin=20Gr=C3=A4=C3=9Flin?= Date: Wed, 20 Aug 2014 11:27:09 +0200 Subject: [PATCH] [kwin_wayland] Split out wl_shell and wl_shell_surface into dedicated classes New classes Shell and ShellSurface are created. Both are in shell.[h|cpp] to indicate their close relationship with the Shell having to create the ShellSurface. WaylandBackend is adjusted to hold a Shell* and ShellSurface* instead of the lower level structs. This also required adjustements to the creation of the Backend as it now doesn't set a default size any more. Thus the backendReady signal may not be emitted before the initial configure event arrived. This also makes it easier to support either the fullscreen shell or wl_shell at the same time. Of course a unit test is added for the two new classes. This needs to be extended once we have more control over the mock Wayland server. --- src/wayland/test_wayland_shell.cpp | 174 +++++++++++++++++++++++++++++ 1 file changed, 174 insertions(+) create mode 100644 src/wayland/test_wayland_shell.cpp diff --git a/src/wayland/test_wayland_shell.cpp b/src/wayland/test_wayland_shell.cpp new file mode 100644 index 0000000000..5be894b7c6 --- /dev/null +++ b/src/wayland/test_wayland_shell.cpp @@ -0,0 +1,174 @@ +/******************************************************************** +KWin - the KDE window manager +This file is part of the KDE project. + +Copyright (C) 2014 Martin Gräßlin + +This program is free software; you can redistribute it and/or modify +it under the terms of the GNU General Public License as published by +the Free Software Foundation; either version 2 of the License, or +(at your option) any later version. + +This program is distributed in the hope that it will be useful, +but WITHOUT ANY WARRANTY; without even the implied warranty of +MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +GNU General Public License for more details. + +You should have received a copy of the GNU General Public License +along with this program. If not, see . +*********************************************************************/ +// Qt +#include +// KWin +#include "../../wayland_client/connection_thread.h" +#include "../../wayland_client/shell.h" +#include "../../wayland_client/registry.h" +// Wayland +#include + +class TestWaylandShell : public QObject +{ + Q_OBJECT +public: + explicit TestWaylandShell(QObject *parent = nullptr); +private Q_SLOTS: + void init(); + void cleanup(); + + void testRegistry(); + void testShell(); + + // TODO: add tests for removal - requires more control over the compositor + +private: + QProcess *m_westonProcess; +}; + +static const QString s_socketName = QStringLiteral("kwin-test-wayland-shell-0"); + +TestWaylandShell::TestWaylandShell(QObject *parent) + : QObject(parent) + , m_westonProcess(nullptr) +{ +} + +void TestWaylandShell::init() +{ + QVERIFY(!m_westonProcess); + // starts weston + m_westonProcess = new QProcess(this); + m_westonProcess->setProgram(QStringLiteral("weston")); + + m_westonProcess->setArguments(QStringList({QStringLiteral("--socket=%1").arg(s_socketName), + QStringLiteral("--use-pixman"), + QStringLiteral("--width=1024"), + QStringLiteral("--height=768")})); + m_westonProcess->start(); + QVERIFY(m_westonProcess->waitForStarted()); + + // wait for the socket to appear + QDir runtimeDir(qgetenv("XDG_RUNTIME_DIR")); + if (runtimeDir.exists(s_socketName)) { + return; + } + QFileSystemWatcher *socketWatcher = new QFileSystemWatcher(QStringList({runtimeDir.absolutePath()}), this); + QSignalSpy socketSpy(socketWatcher, SIGNAL(directoryChanged(QString))); + + // limit to maximum of 10 waits + for (int i = 0; i < 10; ++i) { + QVERIFY(socketSpy.wait()); + if (runtimeDir.exists(s_socketName)) { + delete socketWatcher; + return; + } + } +} + +void TestWaylandShell::cleanup() +{ + // terminates weston + m_westonProcess->terminate(); + QVERIFY(m_westonProcess->waitForFinished()); + delete m_westonProcess; + m_westonProcess = nullptr; +} + +void TestWaylandShell::testRegistry() +{ + if (m_westonProcess->state() != QProcess::Running) { + QSKIP("This test requires a running wayland server"); + } + KWin::Wayland::ConnectionThread connection; + QSignalSpy connectedSpy(&connection, SIGNAL(connected())); + connection.setSocketName(s_socketName); + connection.initConnection(); + QVERIFY(connectedSpy.wait()); + + KWin::Wayland::Registry registry; + QSignalSpy announced(®istry, SIGNAL(shellAnnounced(quint32,quint32))); + registry.create(connection.display()); + QVERIFY(registry.isValid()); + registry.setup(); + wl_display_flush(connection.display()); + QVERIFY(announced.wait()); + + KWin::Wayland::Shell shell; + QVERIFY(!shell.isValid()); + + shell.setup(registry.bindShell(announced.first().first().value(), announced.first().last().value())); + wl_display_flush(connection.display()); + QVERIFY(shell.isValid()); + + shell.release(); + QVERIFY(!shell.isValid()); +} + +void TestWaylandShell::testShell() +{ + if (m_westonProcess->state() != QProcess::Running) { + QSKIP("This test requires a running wayland server"); + } + KWin::Wayland::ConnectionThread connection; + QSignalSpy connectedSpy(&connection, SIGNAL(connected())); + connection.setSocketName(s_socketName); + connection.initConnection(); + QVERIFY(connectedSpy.wait()); + + KWin::Wayland::Registry registry; + QSignalSpy compositorSpy(®istry, SIGNAL(compositorAnnounced(quint32,quint32))); + QSignalSpy announced(®istry, SIGNAL(shellAnnounced(quint32,quint32))); + registry.create(connection.display()); + QVERIFY(registry.isValid()); + registry.setup(); + wl_display_flush(connection.display()); + QVERIFY(announced.wait()); + + if (compositorSpy.isEmpty()) { + QVERIFY(compositorSpy.wait()); + } + wl_compositor *compositor = registry.bindCompositor(compositorSpy.first().first().value(), compositorSpy.first().last().value()); + + KWin::Wayland::Shell shell; + shell.setup(registry.bindShell(announced.first().first().value(), announced.first().last().value())); + wl_display_flush(connection.display()); + KWin::Wayland::ShellSurface *surface = shell.createSurface(wl_compositor_create_surface(compositor), &shell); + QSignalSpy sizeSpy(surface, SIGNAL(sizeChanged(QSize))); + QVERIFY(sizeSpy.isValid()); + QCOMPARE(surface->size(), QSize()); + + surface->setFullscreen(); + wl_display_flush(connection.display()); + QVERIFY(sizeSpy.wait()); + QCOMPARE(sizeSpy.count(), 1); + QCOMPARE(sizeSpy.first().first().toSize(), QSize(1024, 768)); + QCOMPARE(surface->size(), QSize(1024, 768)); + + QVERIFY(surface->isValid()); + shell.release(); + QVERIFY(!surface->isValid()); + + wl_compositor_destroy(compositor); +} + +QTEST_MAIN(TestWaylandShell) +#include "test_wayland_shell.moc"