2436e259ce
The function is used the internal window activation machinery and should not be called by user code. Many tests still use this function, and should be ported over to QWidget::activateWindow(). For now they are using the private helper in QApplicationPrivate, so that we can progress with the public API deprecation. Change-Id: I29f1575acf9efdcbae4c005ee9b2eb1bb0c8e5b5 Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io> Reviewed-by: Qt CI Bot <qt_ci_bot@qt-project.org>
476 lines
14 KiB
C++
476 lines
14 KiB
C++
// Copyright (C) 2016 The Qt Company Ltd.
|
|
// SPDX-License-Identifier: LicenseRef-Qt-Commercial OR GPL-3.0-only WITH Qt-GPL-exception-1.0
|
|
|
|
#include <QDialog>
|
|
#include <QMainWindow>
|
|
#include <QTest>
|
|
#include <QSignalSpy>
|
|
|
|
#include <qapplication.h>
|
|
#include <qevent.h>
|
|
#include <qaction.h>
|
|
#include <qactiongroup.h>
|
|
#include <qmenu.h>
|
|
#include <qmenubar.h>
|
|
#include <qtoolbar.h>
|
|
#include <qpa/qplatformtheme.h>
|
|
#include <qpa/qplatformintegration.h>
|
|
#include <private/qguiapplication_p.h>
|
|
|
|
#include <QtWidgets/private/qapplication_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
|
|
void disableShortcutInMenuAction_data();
|
|
void disableShortcutInMenuAction();
|
|
#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();
|
|
QApplicationPrivate::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();
|
|
QApplicationPrivate::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();
|
|
QApplicationPrivate::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();
|
|
QApplicationPrivate::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();
|
|
QApplicationPrivate::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));
|
|
|
|
QApplicationPrivate::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);
|
|
}
|
|
|
|
/*
|
|
Ignore actions in menus whose menu action has been hidden or disabled.
|
|
The menu entry will not be in the menu bar or parent menu, so the action
|
|
is not reachable through interactive means. QTBUG-25743
|
|
*/
|
|
void tst_QAction::disableShortcutInMenuAction_data()
|
|
{
|
|
QTest::addColumn<QByteArray>("property");
|
|
|
|
QTest::addRow("visible") << QByteArray("visible");
|
|
QTest::addRow("enabled") << QByteArray("enabled");
|
|
}
|
|
|
|
void tst_QAction::disableShortcutInMenuAction()
|
|
{
|
|
QFETCH(QByteArray, property);
|
|
|
|
QMainWindow mw;
|
|
QMenu *testMenu = mw.menuBar()->addMenu("Test");
|
|
QAction *testAction = testMenu->addAction("Test Action");
|
|
testAction->setShortcut(Qt::ControlModifier | Qt::Key_A);
|
|
QToolBar *toolBar = new QToolBar;
|
|
mw.addToolBar(toolBar);
|
|
|
|
mw.show();
|
|
QVERIFY(QTest::qWaitForWindowActive(&mw));
|
|
|
|
int expectedTriggerCount = 0;
|
|
QSignalSpy spy(testAction, &QAction::triggered);
|
|
|
|
QKeyEvent event(QEvent::KeyPress, Qt::Key_A, Qt::ControlModifier);
|
|
QApplication::sendEvent(&mw, &event);
|
|
QCOMPARE(spy.count(), ++expectedTriggerCount);
|
|
|
|
testMenu->menuAction()->setProperty(property, false);
|
|
QApplication::sendEvent(&mw, &event);
|
|
QCOMPARE(spy.count(), expectedTriggerCount);
|
|
|
|
testMenu->menuAction()->setProperty(property, true);
|
|
QApplication::sendEvent(&mw, &event);
|
|
QCOMPARE(spy.count(), ++expectedTriggerCount);
|
|
|
|
// If the action lives somewhere else, then keep firing even
|
|
// if the menu has been hidden or disabled.
|
|
toolBar->addAction(testAction);
|
|
QApplication::sendEvent(&mw, &event);
|
|
QCOMPARE(spy.count(), ++expectedTriggerCount);
|
|
|
|
testMenu->menuAction()->setProperty(property, false);
|
|
QApplication::sendEvent(&mw, &event);
|
|
QCOMPARE(spy.count(), ++expectedTriggerCount);
|
|
|
|
// unless all other widgets in which the action lives have
|
|
// been hidden...
|
|
toolBar->hide();
|
|
QApplication::sendEvent(&mw, &event);
|
|
QCOMPARE(spy.count(), expectedTriggerCount);
|
|
|
|
// ... or disabled
|
|
toolBar->show();
|
|
toolBar->setEnabled(false);
|
|
QApplication::sendEvent(&mw, &event);
|
|
QCOMPARE(spy.count(), expectedTriggerCount);
|
|
|
|
// back to normal
|
|
toolBar->setEnabled(true);
|
|
QApplication::sendEvent(&mw, &event);
|
|
QCOMPARE(spy.count(), ++expectedTriggerCount);
|
|
}
|
|
|
|
#endif // QT_CONFIG(shortcut)
|
|
|
|
QTEST_MAIN(tst_QAction)
|
|
#include "tst_qaction.moc"
|