4b63288a60
- Use nullptr - Use Qt 5 connection syntax - Remove C-Style casts Task-number: QTBUG-69478 Change-Id: Icf8faf3433ff3bff667db050e79b560b221867b0 Reviewed-by: Richard Moe Gustavsen <richard.gustavsen@qt.io>
551 lines
16 KiB
C++
551 lines
16 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 <qmenu.h>
|
|
#include <qpa/qplatformtheme.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 getSetCheck();
|
|
void setText_data();
|
|
void setText();
|
|
void setIconText_data() { setText_data(); }
|
|
void setIconText();
|
|
void setUnknownFont();
|
|
void actionEvent();
|
|
void setStandardKeys();
|
|
void alternateShortcuts();
|
|
void enabledVisibleInteraction();
|
|
void task200823_tooltip();
|
|
void task229128TriggeredSignalWithoutActiongroup();
|
|
void task229128TriggeredSignalWhenInActiongroup();
|
|
void repeat();
|
|
void setData();
|
|
void keysequence(); // QTBUG-53381
|
|
void disableShortcutsWithBlockedWidgets_data();
|
|
void disableShortcutsWithBlockedWidgets();
|
|
void shortcutFromKeyEvent(); // QTBUG-48325
|
|
|
|
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());
|
|
}
|
|
|
|
// Testing get/set functions
|
|
void tst_QAction::getSetCheck()
|
|
{
|
|
QAction obj1(nullptr);
|
|
// QActionGroup * QAction::actionGroup()
|
|
// void QAction::setActionGroup(QActionGroup *)
|
|
QActionGroup *var1 = new QActionGroup(nullptr);
|
|
obj1.setActionGroup(var1);
|
|
QCOMPARE(var1, obj1.actionGroup());
|
|
obj1.setActionGroup(nullptr);
|
|
QCOMPARE(obj1.actionGroup(), nullptr);
|
|
delete var1;
|
|
|
|
// QMenu * QAction::menu()
|
|
// void QAction::setMenu(QMenu *)
|
|
QMenu *var2 = new QMenu(nullptr);
|
|
obj1.setMenu(var2);
|
|
QCOMPARE(var2, obj1.menu());
|
|
obj1.setMenu(nullptr);
|
|
QCOMPARE(obj1.menu(), nullptr);
|
|
delete var2;
|
|
|
|
QCOMPARE(obj1.priority(), QAction::NormalPriority);
|
|
obj1.setPriority(QAction::LowPriority);
|
|
QCOMPARE(obj1.priority(), QAction::LowPriority);
|
|
}
|
|
|
|
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::setText_data()
|
|
{
|
|
QTest::addColumn<QString>("text");
|
|
QTest::addColumn<QString>("iconText");
|
|
QTest::addColumn<QString>("textFromIconText");
|
|
|
|
//next we fill it with data
|
|
QTest::newRow("Normal") << "Action" << "Action" << "Action";
|
|
QTest::newRow("Ampersand") << "Search && Destroy" << "Search & Destroy" << "Search && Destroy";
|
|
QTest::newRow("Mnemonic and ellipsis") << "O&pen File ..." << "Open File" << "Open File";
|
|
}
|
|
|
|
void tst_QAction::setText()
|
|
{
|
|
QFETCH(QString, text);
|
|
|
|
QAction action(nullptr);
|
|
action.setText(text);
|
|
|
|
QCOMPARE(action.text(), text);
|
|
|
|
QFETCH(QString, iconText);
|
|
QCOMPARE(action.iconText(), iconText);
|
|
}
|
|
|
|
void tst_QAction::setIconText()
|
|
{
|
|
QFETCH(QString, iconText);
|
|
|
|
QAction action(nullptr);
|
|
action.setIconText(iconText);
|
|
QCOMPARE(action.iconText(), iconText);
|
|
|
|
QFETCH(QString, textFromIconText);
|
|
QCOMPARE(action.text(), textFromIconText);
|
|
}
|
|
|
|
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);
|
|
}
|
|
|
|
//basic testing of standard keys
|
|
void tst_QAction::setStandardKeys()
|
|
{
|
|
QAction act(nullptr);
|
|
act.setShortcut(QKeySequence("CTRL+L"));
|
|
QList<QKeySequence> list;
|
|
act.setShortcuts(list);
|
|
act.setShortcuts(QKeySequence::Copy);
|
|
QCOMPARE(act.shortcut(), act.shortcuts().constFirst());
|
|
|
|
QList<QKeySequence> expected;
|
|
const QKeySequence ctrlC = QKeySequence(QStringLiteral("CTRL+C"));
|
|
const QKeySequence ctrlInsert = QKeySequence(QStringLiteral("CTRL+INSERT"));
|
|
switch (m_keyboardScheme) {
|
|
case QPlatformTheme::MacKeyboardScheme:
|
|
expected << ctrlC;
|
|
break;
|
|
case QPlatformTheme::WindowsKeyboardScheme:
|
|
expected << ctrlC << ctrlInsert;
|
|
break;
|
|
default: // X11
|
|
expected << ctrlC << ctrlInsert << QKeySequence(QStringLiteral("F16"));
|
|
break;
|
|
}
|
|
|
|
QCOMPARE(act.shortcuts(), expected);
|
|
}
|
|
|
|
|
|
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
|
|
}
|
|
|
|
void tst_QAction::task200823_tooltip()
|
|
{
|
|
const QScopedPointer<QAction> action(new QAction("foo", nullptr));
|
|
QString shortcut("ctrl+o");
|
|
action->setShortcut(shortcut);
|
|
|
|
// we want a non-standard tooltip that shows the shortcut
|
|
action->setToolTip(action->text() + QLatin1String(" (") + action->shortcut().toString() + QLatin1Char(')'));
|
|
|
|
QString ref = QLatin1String("foo (") + QKeySequence(shortcut).toString() + QLatin1Char(')');
|
|
QCOMPARE(action->toolTip(), ref);
|
|
}
|
|
|
|
void tst_QAction::task229128TriggeredSignalWithoutActiongroup()
|
|
{
|
|
// test without a group
|
|
const QScopedPointer<QAction> actionWithoutGroup(new QAction("Test", nullptr));
|
|
QSignalSpy spyWithoutGroup(actionWithoutGroup.data(), QOverload<bool>::of(&QAction::triggered));
|
|
QCOMPARE(spyWithoutGroup.count(), 0);
|
|
actionWithoutGroup->trigger();
|
|
// signal should be emitted
|
|
QCOMPARE(spyWithoutGroup.count(), 1);
|
|
|
|
// it is now a checkable checked action
|
|
actionWithoutGroup->setCheckable(true);
|
|
actionWithoutGroup->setChecked(true);
|
|
spyWithoutGroup.clear();
|
|
QCOMPARE(spyWithoutGroup.count(), 0);
|
|
actionWithoutGroup->trigger();
|
|
// signal should be emitted
|
|
QCOMPARE(spyWithoutGroup.count(), 1);
|
|
}
|
|
|
|
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);
|
|
}
|
|
|
|
void tst_QAction::repeat()
|
|
{
|
|
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::setData() // QTBUG-62006
|
|
{
|
|
QAction act(nullptr);
|
|
QSignalSpy spy(&act, &QAction::changed);
|
|
QCOMPARE(act.data(), QVariant());
|
|
QCOMPARE(spy.count(), 0);
|
|
act.setData(QVariant());
|
|
QCOMPARE(spy.count(), 0);
|
|
|
|
act.setData(-1);
|
|
QCOMPARE(spy.count(), 1);
|
|
act.setData(-1);
|
|
QCOMPARE(spy.count(), 1);
|
|
}
|
|
|
|
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()
|
|
{
|
|
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()
|
|
{
|
|
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);
|
|
}
|
|
|
|
QTEST_MAIN(tst_QAction)
|
|
#include "tst_qaction.moc"
|