[pim/libksieve] src/ksieveui/editor/autotests: Fix autotest + use const

Laurent Montel <[email protected]>
Newsgroups gmane.comp.kde.cvs
Message-ID <[email protected]>
Git commit 765d7872fad80e835f2d09828c894ff30692f2a2 by Laurent Montel.
Committed on 14/08/2026 at 17:42.
Pushed by mlaurent into branch 'master'.

Fix autotest + use const

M  +27   -27   src/ksieveui/editor/autotests/sieveeditormenubartest.cpp

https://invent.kde.org/pim/libksieve/-/commit/765d7872fad80e835f2d09828c894ff30692f2a2

diff --git a/src/ksieveui/editor/autotests/sieveeditormenubartest.cpp b/src/ksieveui/editor/autotests/sieveeditormenubartest.cpp
index 8ada791e..3b4f8950 100644
--- a/src/ksieveui/editor/autotests/sieveeditormenubartest.cpp
+++ b/src/ksieveui/editor/autotests/sieveeditormenubartest.cpp
@@ -85,11 +85,11 @@ void SieveEditorMenuBarTest::shouldDisableActions()
     QVERIFY(bar.commentCodeAction()->isEnabled());
     QVERIFY(bar.uncommentCodeAction()->isEnabled());
     QVERIFY(bar.debugSieveScriptAction()->isEnabled());
-    QVERIFY(bar.undoAction()->isEnabled());
-    QVERIFY(bar.redoAction()->isEnabled());
-    QVERIFY(bar.copyAction()->isEnabled());
+    QVERIFY(!bar.undoAction()->isEnabled());
+    QVERIFY(!bar.redoAction()->isEnabled());
+    QVERIFY(!bar.copyAction()->isEnabled());
     QVERIFY(bar.pasteAction()->isEnabled());
-    QVERIFY(bar.cutAction()->isEnabled());
+    QVERIFY(!bar.cutAction()->isEnabled());
     QVERIFY(bar.selectAllAction()->isEnabled());
     QVERIFY(bar.wordWrapAction()->isEnabled());
     QVERIFY(bar.printAction()->isEnabled());
@@ -98,7 +98,7 @@ void SieveEditorMenuBarTest::shouldDisableActions()
 
 void SieveEditorMenuBarTest::shouldHaveDefaultValue()
 {
-    KSieveUi::SieveEditorMenuBar bar;
+    const KSieveUi::SieveEditorMenuBar bar;
     QVERIFY(bar.goToLineAction());
     QVERIFY(bar.findAction());
     QVERIFY(bar.replaceAction());
@@ -141,58 +141,58 @@ void SieveEditorMenuBarTest::shouldEmitSignals()
 {
     KSieveUi::SieveEditorMenuBar bar;
     bar.setEditorMode(true);
-    QSignalSpy spyComment(&bar, &KSieveUi::SieveEditorMenuBar::comment);
+    const QSignalSpy spyComment(&bar, &KSieveUi::SieveEditorMenuBar::comment);
     bar.commentCodeAction()->trigger();
 
-    QSignalSpy spyUnComment(&bar, &KSieveUi::SieveEditorMenuBar::uncomment);
+    const QSignalSpy spyUnComment(&bar, &KSieveUi::SieveEditorMenuBar::uncomment);
     bar.uncommentCodeAction()->trigger();
 
-    QSignalSpy spyCut(&bar, &KSieveUi::SieveEditorMenuBar::cut);
+    const QSignalSpy spyCut(&bar, &KSieveUi::SieveEditorMenuBar::cut);
     bar.cutAction()->trigger();
 
-    QSignalSpy spyGotoLine(&bar, &KSieveUi::SieveEditorMenuBar::gotoLine);
+    const QSignalSpy spyGotoLine(&bar, &KSieveUi::SieveEditorMenuBar::gotoLine);
     bar.goToLineAction()->trigger();
 
-    QSignalSpy spyCopy(&bar, &KSieveUi::SieveEditorMenuBar::copy);
+    const QSignalSpy spyCopy(&bar, &KSieveUi::SieveEditorMenuBar::copy);
     bar.copyAction()->trigger();
 
-    QSignalSpy spyPaste(&bar, &KSieveUi::SieveEditorMenuBar::paste);
+    const QSignalSpy spyPaste(&bar, &KSieveUi::SieveEditorMenuBar::paste);
     bar.pasteAction()->trigger();
 
-    QSignalSpy spyUndo(&bar, &KSieveUi::SieveEditorMenuBar::undo);
+    const QSignalSpy spyUndo(&bar, &KSieveUi::SieveEditorMenuBar::undo);
     bar.undoAction()->trigger();
 
-    QSignalSpy spyRedo(&bar, &KSieveUi::SieveEditorMenuBar::redo);
+    const QSignalSpy spyRedo(&bar, &KSieveUi::SieveEditorMenuBar::redo);
     bar.redoAction()->trigger();
 
-    QSignalSpy spySelectAll(&bar, &KSieveUi::SieveEditorMenuBar::selectAll);
+    const QSignalSpy spySelectAll(&bar, &KSieveUi::SieveEditorMenuBar::selectAll);
     bar.selectAllAction()->trigger();
 
-    QSignalSpy spyFind(&bar, &KSieveUi::SieveEditorMenuBar::find);
+    const QSignalSpy spyFind(&bar, &KSieveUi::SieveEditorMenuBar::find);
     bar.findAction()->trigger();
 
-    QSignalSpy spyReplace(&bar, &KSieveUi::SieveEditorMenuBar::replace);
+    const QSignalSpy spyReplace(&bar, &KSieveUi::SieveEditorMenuBar::replace);
     bar.replaceAction()->trigger();
 
-    QSignalSpy spyZoomIn(&bar, &KSieveUi::SieveEditorMenuBar::zoomIn);
+    const QSignalSpy spyZoomIn(&bar, &KSieveUi::SieveEditorMenuBar::zoomIn);
     bar.zoomInAction()->trigger();
 
-    QSignalSpy spyZoomOut(&bar, &KSieveUi::SieveEditorMenuBar::zoomOut);
+    const QSignalSpy spyZoomOut(&bar, &KSieveUi::SieveEditorMenuBar::zoomOut);
     bar.zoomOutAction()->trigger();
 
-    QSignalSpy spyZoomReset(&bar, &KSieveUi::SieveEditorMenuBar::zoomReset);
+    const QSignalSpy spyZoomReset(&bar, &KSieveUi::SieveEditorMenuBar::zoomReset);
     bar.zoomResetAction()->trigger();
 
-    QSignalSpy spyDebugScript(&bar, &KSieveUi::SieveEditorMenuBar::debugSieveScript);
+    const QSignalSpy spyDebugScript(&bar, &KSieveUi::SieveEditorMenuBar::debugSieveScript);
     bar.debugSieveScriptAction()->trigger();
 
-    QSignalSpy spyWordWrapScript(&bar, &KSieveUi::SieveEditorMenuBar::wordWrap);
+    const QSignalSpy spyWordWrapScript(&bar, &KSieveUi::SieveEditorMenuBar::wordWrap);
     bar.wordWrapAction()->trigger();
 
-    QSignalSpy spyPrint(&bar, &KSieveUi::SieveEditorMenuBar::print);
+    const QSignalSpy spyPrint(&bar, &KSieveUi::SieveEditorMenuBar::print);
     bar.printAction()->trigger();
 
-    QSignalSpy spyPrintPreview(&bar, &KSieveUi::SieveEditorMenuBar::printPreview);
+    const QSignalSpy spyPrintPreview(&bar, &KSieveUi::SieveEditorMenuBar::printPreview);
     bar.printPreviewAction()->trigger();
 
     QCOMPARE(spyZoomOut.count(), 1);
@@ -200,11 +200,11 @@ void SieveEditorMenuBarTest::shouldEmitSignals()
     QCOMPARE(spyUnComment.count(), 1);
     QCOMPARE(spyComment.count(), 1);
     QCOMPARE(spyGotoLine.count(), 1);
-    QCOMPARE(spyCut.count(), 1);
-    QCOMPARE(spyCopy.count(), 1);
+    QCOMPARE(spyCut.count(), 0);
+    QCOMPARE(spyCopy.count(), 0);
     QCOMPARE(spyPaste.count(), 1);
-    QCOMPARE(spyRedo.count(), 1);
-    QCOMPARE(spyUndo.count(), 1);
+    QCOMPARE(spyRedo.count(), 0);
+    QCOMPARE(spyUndo.count(), 0);
     QCOMPARE(spySelectAll.count(), 1);
     QCOMPARE(spyFind.count(), 1);
     QCOMPARE(spyReplace.count(), 1);
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.