Merge branch 'feature/ui-state-persistence'

Persistence cluster from the post-0.1.0 usability backlog: window,
splitter and column geometry survive restart, the message pane owns its
zoom and remembers it, and the startup query is chosen by name instead
of by alphabetical accident.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>

1 files changed, 64 insertions, 0 deletions

diff --git a/tests/test_mainwindow.cpp b/tests/test_mainwindow.cpp
index 6bfa925..b4d9e4a 100644
--- a/tests/test_mainwindow.cpp
+++ b/tests/test_mainwindow.cpp
@@ -20,12 +20,15 @@
 
 #include <QAction>
 #include <QDir>
+#include <QFile>
 #include <QSettings>
+#include <QStandardPaths>
 #include <QTemporaryDir>
 
 #include "config.h"
 #include "keymap.h"
 #include "mainwindow.h"
+#include "messageview.h"
 
 /// MainWindow is mostly wiring, and the parts that need a real database are
 /// still verified manually. What is checked here is the action registry: the
@@ -41,6 +44,9 @@ private slots:
     void configuredBindingReachesTheAction();
     void cidPrefixesAreBangFree();
     void cidPrefixesAreDistinctPerMessage();
+    void uiStateIsNotWrittenIntoTheUserConfig();
+    void uiStateSurvivesARestart();
+    void missingUiStateLeavesTheDefaults();
 };
 
 void TestMainWindow::everyKnownActionIsRegistered()
@@ -158,6 +164,64 @@ void TestMainWindow::cidPrefixesAreDistinctPerMessage()
     }
 }
 
+void TestMainWindow::uiStateIsNotWrittenIntoTheUserConfig()
+{
+    // The config file is hand-edited and must never gain a base64 geometry
+    // blob, nor be rewritten on exit: QSettings preserves neither comments nor
+    // key order, so writing it would quietly destroy the user's formatting.
+    QVERIFY(MainWindow::uiStatePath() != Config::defaultPath());
+
+    // One qtmaildir component, not two. QStandardPaths::StateLocation appends
+    // both the organization and the application name, and here both are
+    // "qtmaildir", so using it nests the directory inside itself.
+    QCOMPARE(MainWindow::uiStatePath().count(QStringLiteral("/qtmaildir/")), 1);
+    QVERIFY(MainWindow::uiStatePath().endsWith(
+        QStringLiteral("/qtmaildir/uistate.conf")));
+}
+
+void TestMainWindow::uiStateSurvivesARestart()
+{
+    // Test mode redirects QStandardPaths at the process level, so the state
+    // file lands in a scratch directory rather than the real ~/.local/state.
+    QStandardPaths::setTestModeEnabled(true);
+    QFile::remove(MainWindow::uiStatePath());
+
+    const QSize resized(940, 620);
+    {
+        const Config config;
+        MainWindow window(config);
+        window.resize(resized);
+        window.findChild<MessageView *>()->setZoomFactor(1.4);
+        window.close();  // closeEvent() is what persists the state
+    }
+
+    QVERIFY2(QFile::exists(MainWindow::uiStatePath()),
+             qPrintable(QStringLiteral("no state file at %1")
+                            .arg(MainWindow::uiStatePath())));
+
+    const Config config;
+    MainWindow reopened(config);
+    QCOMPARE(reopened.size(), resized);
+    QCOMPARE(reopened.findChild<MessageView *>()->zoomFactor(), 1.4);
+
+    QFile::remove(MainWindow::uiStatePath());
+    QStandardPaths::setTestModeEnabled(false);
+}
+
+void TestMainWindow::missingUiStateLeavesTheDefaults()
+{
+    // A restore that silently succeeded on an empty blob would give a
+    // zero-size window on first launch. Absent state must be a no-op.
+    QStandardPaths::setTestModeEnabled(true);
+    QFile::remove(MainWindow::uiStatePath());
+
+    const Config config;
+    MainWindow window(config);
+    QCOMPARE(window.size(), QSize(1200, 800));
+
+    QStandardPaths::setTestModeEnabled(false);
+}
+
 // Constructing a MainWindow needs a QApplication and a platform plugin. The
 // test has no display under ctest, so it runs offscreen unless the caller
 // asked for something else.