diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-12 00:45:36 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-12 00:45:36 -0500 |
| commit | 18ba99fda928769adb235bd85b485c8be94c3ddd (patch) | |
| tree | e27f938fd55afa09a7d320d750205e785cfe5b93 /tests/test-mu4e-attachments.el | |
| parent | 8cb30ef6efd06464ab1153b99261a3f626149f97 (diff) | |
| download | dotemacs-18ba99fda928769adb235bd85b485c8be94c3ddd.tar.gz dotemacs-18ba99fda928769adb235bd85b485c8be94c3ddd.zip | |
fix(mail): clear the marks after saving from the attachment selector
`cj/mu4e-attachment-selection-save-marked' left the `[x]' marks set after a successful save, so a second `s' silently re-saved the same files. It now unmarks every row and re-renders once the save returns, so the buffer stays open for another batch (and `q'/RET still exit). The save-marked test asserts the marks are cleared.
Diffstat (limited to 'tests/test-mu4e-attachments.el')
| -rw-r--r-- | tests/test-mu4e-attachments.el | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/tests/test-mu4e-attachments.el b/tests/test-mu4e-attachments.el index 3105d710..21336f75 100644 --- a/tests/test-mu4e-attachments.el +++ b/tests/test-mu4e-attachments.el @@ -192,7 +192,11 @@ so this fails the same way whether or not mu4e's MIME support is loadable." ((symbol-function 'message) (lambda (&rest _) nil))) (should (equal (cj/mu4e-attachment-selection-save-marked) '("/downloads/b.pdf"))) - (should (equal saved (list (list b) "/downloads/"))))) + (should (equal saved (list (list b) "/downloads/"))) + ;; marks clear after a successful save so `s' won't re-save them + (should-not (seq-some (lambda (entry) (plist-get entry :selected)) + cj/mu4e-attachment-selection-entries)) + (should (string-match-p "\\[ \\] b\\.pdf" (buffer-string))))) (kill-buffer buffer)))) (ert-deftest test-mu4e-attachments-selection-save-marked-errors-when-empty () |