bcaff2b06f
Duplicating the number of classes is a high price to pay to be able to have some QAction functionality behave differently, or be only available in widgets applications. Instead, declare the entire API in QtGui in QAction* classes, and delegate the implementation of QtWidgets specific functionality to the private. The creation of the private is then delegated to the Q(Gui)ApplicationPrivate instance through a virtual factory function. Change some public APIs that are primarily useful for specialized tools such as Designer to operate on QObject* rather than QWidget*. APIs that depend on QtWidgets types have been turned into inline template functions, so that they are instantiated only at the caller side, where we can expect the respective types to be fully defined. This way, we only need to forward declare a few classes in the header, and don't need to generate any additional code for e.g. language bindings. Change-Id: Id0b27f9187652ec531a2e8b1b9837e82dc81625c Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
424 lines
13 KiB
C++
424 lines
13 KiB
C++
/****************************************************************************
|
|
**
|
|
** Copyright (C) 2016 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 <QDialog>
|
|
#include <QMainWindow>
|
|
#include <QtTest/QtTest>
|
|
|
|
#include <qapplication.h>
|
|
#include <qevent.h>
|
|
#include <qaction.h>
|
|
#include <qactiongroup.h>
|
|
#include <qmenu.h>
|
|
#include <qpa/qplatformtheme.h>
|
|
#include <qpa/qplatformintegration.h>
|
|
#include <private/qguiapplication_p.h>
|
|
|
|
class tst_QAction : public QObject
|
|
{
|
|
Q_OBJECT
|
|
|
|
public:
|
|
tst_QAction();
|
|
|
|
void updateState(QActionEvent *e);
|
|
|
|
private slots:
|
|
void init();
|
|
void cleanup();
|
|
void setUnknownFont();
|
|
void actionEvent();
|
|
#if QT_CONFIG(shortcut)
|
|
void alternateShortcuts();
|
|
void enabledVisibleInteraction();
|
|
#endif
|
|
void task229128TriggeredSignalWhenInActiongroup();
|
|
#if QT_CONFIG(shortcut)
|
|
void repeat();
|
|
void keysequence(); // QTBUG-53381
|
|
void disableShortcutsWithBlockedWidgets_data();
|
|
void disableShortcutsWithBlockedWidgets();
|
|
void shortcutFromKeyEvent(); // QTBUG-48325
|
|
#endif
|
|
|
|
private:
|
|
QEvent::Type m_lastEventType;
|
|
const int m_keyboardScheme;
|
|
QAction *m_lastAction;
|
|
};
|
|
|
|
tst_QAction::tst_QAction()
|
|
: m_keyboardScheme(QGuiApplicationPrivate::platformTheme()->themeHint(QPlatformTheme::KeyboardScheme).toInt())
|
|
{
|
|
}
|
|
|
|
void tst_QAction::init()
|
|
{
|
|
m_lastEventType = QEvent::None;
|
|
m_lastAction = nullptr;
|
|
}
|
|
|
|
void tst_QAction::cleanup()
|
|
{
|
|
QVERIFY(QApplication::topLevelWidgets().isEmpty());
|
|
}
|
|
|
|
class MyWidget : public QWidget
|
|
{
|
|
Q_OBJECT
|
|
public:
|
|
explicit MyWidget(tst_QAction *tst, QWidget *parent = nullptr) : QWidget(parent), m_test(tst)
|
|
{ setWindowTitle(QTest::currentTestFunction()); }
|
|
|
|
protected:
|
|
void actionEvent(QActionEvent *e) override { m_test->updateState(e); }
|
|
|
|
private:
|
|
tst_QAction *m_test;
|
|
};
|
|
|
|
void tst_QAction::setUnknownFont() // QTBUG-42728
|
|
{
|
|
QAction action(nullptr);
|
|
QFont font("DoesNotExist", 11);
|
|
action.setFont(font);
|
|
|
|
QMenu menu;
|
|
menu.addAction(&action); // should not crash
|
|
}
|
|
|
|
void tst_QAction::updateState(QActionEvent *e)
|
|
{
|
|
if (!e) {
|
|
m_lastEventType = QEvent::None;
|
|
m_lastAction = nullptr;
|
|
} else {
|
|
m_lastEventType = e->type();
|
|
m_lastAction = e->action();
|
|
}
|
|
}
|
|
|
|
void tst_QAction::actionEvent()
|
|
{
|
|
QAction a(nullptr);
|
|
a.setText("action text");
|
|
|
|
// add action
|
|
MyWidget testWidget(this);
|
|
testWidget.show();
|
|
QApplication::setActiveWindow(&testWidget);
|
|
testWidget.addAction(&a);
|
|
qApp->processEvents();
|
|
|
|
QCOMPARE(m_lastEventType, QEvent::ActionAdded);
|
|
QCOMPARE(m_lastAction, &a);
|
|
|
|
// change action
|
|
a.setText("new action text");
|
|
qApp->processEvents();
|
|
|
|
QCOMPARE(m_lastEventType, QEvent::ActionChanged);
|
|
QCOMPARE(m_lastAction, &a);
|
|
|
|
// remove action
|
|
testWidget.removeAction(&a);
|
|
qApp->processEvents();
|
|
|
|
QCOMPARE(m_lastEventType, QEvent::ActionRemoved);
|
|
QCOMPARE(m_lastAction, &a);
|
|
}
|
|
|
|
#if QT_CONFIG(shortcut)
|
|
|
|
void tst_QAction::alternateShortcuts()
|
|
{
|
|
//test the alternate shortcuts (by adding more than 1 shortcut)
|
|
|
|
MyWidget testWidget(this);
|
|
testWidget.show();
|
|
QApplication::setActiveWindow(&testWidget);
|
|
|
|
{
|
|
QAction act(&testWidget);
|
|
testWidget.addAction(&act);
|
|
QList<QKeySequence> shlist = QList<QKeySequence>() << QKeySequence("CTRL+P") << QKeySequence("CTRL+A");
|
|
act.setShortcuts(shlist);
|
|
|
|
QSignalSpy spy(&act, &QAction::triggered);
|
|
|
|
act.setAutoRepeat(true);
|
|
QTest::keyClick(&testWidget, Qt::Key_A, Qt::ControlModifier);
|
|
QCOMPARE(spy.count(), 1); //act should have been triggered
|
|
|
|
act.setAutoRepeat(false);
|
|
QTest::keyClick(&testWidget, Qt::Key_A, Qt::ControlModifier);
|
|
QCOMPARE(spy.count(), 2); //act should have been triggered a 2nd time
|
|
|
|
//end of the scope of the action, it will be destroyed and removed from wid
|
|
//This action should also unregister its shortcuts
|
|
}
|
|
|
|
|
|
//this tests a crash (if the action did not unregister its alternate shortcuts)
|
|
QTest::keyClick(&testWidget, Qt::Key_A, Qt::ControlModifier);
|
|
}
|
|
|
|
void tst_QAction::keysequence()
|
|
{
|
|
MyWidget testWidget(this);
|
|
testWidget.show();
|
|
QApplication::setActiveWindow(&testWidget);
|
|
|
|
{
|
|
QAction act(&testWidget);
|
|
testWidget.addAction(&act);
|
|
|
|
QKeySequence ks(QKeySequence::SelectAll);
|
|
|
|
act.setShortcut(ks);
|
|
|
|
QSignalSpy spy(&act, &QAction::triggered);
|
|
|
|
act.setAutoRepeat(true);
|
|
QTest::keySequence(&testWidget, ks);
|
|
QCoreApplication::processEvents();
|
|
QCOMPARE(spy.count(), 1); // act should have been triggered
|
|
|
|
act.setAutoRepeat(false);
|
|
QTest::keySequence(&testWidget, ks);
|
|
QCoreApplication::processEvents();
|
|
QCOMPARE(spy.count(), 2); //act should have been triggered a 2nd time
|
|
|
|
// end of the scope of the action, it will be destroyed and removed from widget
|
|
// This action should also unregister its shortcuts
|
|
}
|
|
|
|
// this tests a crash (if the action did not unregister its alternate shortcuts)
|
|
QTest::keyClick(&testWidget, Qt::Key_A, Qt::ControlModifier);
|
|
}
|
|
|
|
void tst_QAction::enabledVisibleInteraction()
|
|
{
|
|
MyWidget testWidget(this);
|
|
testWidget.show();
|
|
QApplication::setActiveWindow(&testWidget);
|
|
|
|
QAction act(nullptr);
|
|
// check defaults
|
|
QVERIFY(act.isEnabled());
|
|
QVERIFY(act.isVisible());
|
|
|
|
// !visible => !enabled
|
|
act.setVisible(false);
|
|
QVERIFY(!act.isEnabled());
|
|
act.setVisible(true);
|
|
QVERIFY(act.isEnabled());
|
|
act.setEnabled(false);
|
|
QVERIFY(act.isVisible());
|
|
|
|
// check if shortcut is disabled if not visible
|
|
testWidget.addAction(&act);
|
|
act.setShortcut(QKeySequence("Ctrl+T"));
|
|
QSignalSpy spy(&act, SIGNAL(triggered()));
|
|
act.setEnabled(true);
|
|
act.setVisible(false);
|
|
QTest::keyClick(&testWidget, Qt::Key_T, Qt::ControlModifier);
|
|
QCOMPARE(spy.count(), 0); //act is not visible, so don't trigger
|
|
act.setVisible(false);
|
|
act.setEnabled(true);
|
|
QTest::keyClick(&testWidget, Qt::Key_T, Qt::ControlModifier);
|
|
QCOMPARE(spy.count(), 0); //act is not visible, so don't trigger
|
|
act.setVisible(true);
|
|
act.setEnabled(true);
|
|
QTest::keyClick(&testWidget, Qt::Key_T, Qt::ControlModifier);
|
|
QCOMPARE(spy.count(), 1); //act is visible and enabled, so trigger
|
|
}
|
|
|
|
#endif // QT_CONFIG(shortcut)
|
|
|
|
void tst_QAction::task229128TriggeredSignalWhenInActiongroup()
|
|
{
|
|
QActionGroup ag(nullptr);
|
|
QAction *action = new QAction("Test", &ag);
|
|
QAction *checkedAction = new QAction("Test 2", &ag);
|
|
ag.addAction(action);
|
|
action->setCheckable(true);
|
|
ag.addAction(checkedAction);
|
|
checkedAction->setCheckable(true);
|
|
checkedAction->setChecked(true);
|
|
|
|
QSignalSpy actionSpy(checkedAction, QOverload<bool>::of(&QAction::triggered));
|
|
QSignalSpy actionGroupSpy(&ag, QOverload<QAction*>::of(&QActionGroup::triggered));
|
|
QCOMPARE(actionGroupSpy.count(), 0);
|
|
QCOMPARE(actionSpy.count(), 0);
|
|
checkedAction->trigger();
|
|
// check that both the group and the action have emitted the signal
|
|
QCOMPARE(actionGroupSpy.count(), 1);
|
|
QCOMPARE(actionSpy.count(), 1);
|
|
}
|
|
|
|
#if QT_CONFIG(shortcut)
|
|
|
|
void tst_QAction::repeat()
|
|
{
|
|
if (QGuiApplication::platformName().startsWith(QLatin1String("wayland"), Qt::CaseInsensitive))
|
|
QSKIP("Wayland: This fails. Figure out why.");
|
|
|
|
MyWidget testWidget(this);
|
|
testWidget.show();
|
|
QApplication::setActiveWindow(&testWidget);
|
|
QVERIFY(QTest::qWaitForWindowActive(&testWidget));
|
|
|
|
QAction act(&testWidget);
|
|
testWidget.addAction(&act);
|
|
act.setShortcut(QKeySequence(Qt::Key_F));
|
|
QSignalSpy spy(&act, SIGNAL(triggered()));
|
|
|
|
act.setAutoRepeat(true);
|
|
QTest::keyPress(&testWidget, Qt::Key_F);
|
|
QTest::keyRelease(&testWidget, Qt::Key_F);
|
|
QCOMPARE(spy.count(), 1);
|
|
|
|
spy.clear();
|
|
QTest::keyPress(&testWidget, Qt::Key_F);
|
|
// repeat event
|
|
QTest::simulateEvent(&testWidget, true, Qt::Key_F, Qt::NoModifier, QString("f"), true);
|
|
QTest::simulateEvent(&testWidget, true, Qt::Key_F, Qt::NoModifier, QString("f"), true);
|
|
QTest::keyRelease(&testWidget, Qt::Key_F);
|
|
QCOMPARE(spy.count(), 3);
|
|
|
|
spy.clear();
|
|
act.setAutoRepeat(false);
|
|
QTest::keyPress(&testWidget, Qt::Key_F);
|
|
QTest::simulateEvent(&testWidget, true, Qt::Key_F, Qt::NoModifier, QString("f"), true);
|
|
QTest::simulateEvent(&testWidget, true, Qt::Key_F, Qt::NoModifier, QString("f"), true);
|
|
QTest::keyRelease(&testWidget, Qt::Key_F);
|
|
QCOMPARE(spy.count(), 1);
|
|
|
|
spy.clear();
|
|
act.setAutoRepeat(true);
|
|
QTest::keyPress(&testWidget, Qt::Key_F);
|
|
QTest::simulateEvent(&testWidget, true, Qt::Key_F, Qt::NoModifier, QString("f"), true);
|
|
QTest::keyRelease(&testWidget, Qt::Key_F);
|
|
QCOMPARE(spy.count(), 2);
|
|
}
|
|
|
|
void tst_QAction::disableShortcutsWithBlockedWidgets_data()
|
|
{
|
|
QTest::addColumn<Qt::ShortcutContext>("shortcutContext");
|
|
QTest::addColumn<Qt::WindowModality>("windowModality");
|
|
|
|
QTest::newRow("application modal dialog should block window shortcut.")
|
|
<< Qt::WindowShortcut << Qt::ApplicationModal;
|
|
|
|
QTest::newRow("application modal dialog should block application shortcut.")
|
|
<< Qt::ApplicationShortcut << Qt::ApplicationModal;
|
|
|
|
QTest::newRow("window modal dialog should block application shortcut.")
|
|
<< Qt::ApplicationShortcut << Qt::WindowModal;
|
|
|
|
QTest::newRow("window modal dialog should block window shortcut.")
|
|
<< Qt::WindowShortcut << Qt::WindowModal;
|
|
}
|
|
|
|
|
|
void tst_QAction::disableShortcutsWithBlockedWidgets()
|
|
{
|
|
if (!QGuiApplicationPrivate::platformIntegration()->hasCapability(QPlatformIntegration::WindowActivation))
|
|
QSKIP("Window activation is not supported");
|
|
|
|
QMainWindow window;
|
|
|
|
QFETCH(Qt::ShortcutContext, shortcutContext);
|
|
QAction action(&window);
|
|
window.addAction(&action);
|
|
action.setShortcut(QKeySequence(Qt::Key_1));
|
|
action.setShortcutContext(shortcutContext);
|
|
|
|
window.show();
|
|
QVERIFY(QTest::qWaitForWindowExposed(&window));
|
|
|
|
QDialog dialog(&window);
|
|
QFETCH(Qt::WindowModality, windowModality);
|
|
dialog.setWindowModality(windowModality);
|
|
|
|
dialog.show();
|
|
QVERIFY(QTest::qWaitForWindowExposed(&dialog));
|
|
|
|
QApplication::setActiveWindow(&window);
|
|
QVERIFY(QTest::qWaitForWindowActive(&window));
|
|
|
|
QSignalSpy spy(&action, &QAction::triggered);
|
|
QTest::keyPress(&window, Qt::Key_1);
|
|
QCOMPARE(spy.count(), 0);
|
|
}
|
|
|
|
class ShortcutOverrideWidget : public QWidget
|
|
{
|
|
public:
|
|
using QWidget::QWidget;
|
|
int shortcutOverrideCount = 0;
|
|
protected:
|
|
bool event(QEvent *e) override
|
|
{
|
|
if (e->type() == QEvent::ShortcutOverride)
|
|
++shortcutOverrideCount;
|
|
return QWidget::event(e);
|
|
}
|
|
};
|
|
|
|
// Test that a key press event sent with sendEvent() still gets handled as a possible
|
|
// ShortcutOverride event first before passing it on as a normal KeyEvent.
|
|
void tst_QAction::shortcutFromKeyEvent()
|
|
{
|
|
if (QGuiApplication::platformName().startsWith(QLatin1String("wayland"), Qt::CaseInsensitive))
|
|
QSKIP("Wayland: This fails. Figure out why.");
|
|
|
|
ShortcutOverrideWidget testWidget;
|
|
QAction action;
|
|
action.setShortcut(Qt::Key_1);
|
|
testWidget.addAction(&action);
|
|
testWidget.show();
|
|
QSignalSpy spy(&action, &QAction::triggered);
|
|
QVERIFY(spy.isValid());
|
|
QVERIFY(QTest::qWaitForWindowActive(&testWidget));
|
|
QCOMPARE(testWidget.shortcutOverrideCount, 0);
|
|
|
|
// Don't use the QTest::keyPress approach as this will take the
|
|
// shortcut route for us
|
|
QKeyEvent e(QEvent::KeyPress, Qt::Key_1, Qt::NoModifier);
|
|
QApplication::sendEvent(&testWidget, &e);
|
|
QCOMPARE(spy.count(), 1);
|
|
QCOMPARE(testWidget.shortcutOverrideCount, 1);
|
|
}
|
|
|
|
#endif // QT_CONFIG(shortcut)
|
|
|
|
QTEST_MAIN(tst_QAction)
|
|
#include "tst_qaction.moc"
|