blob: c35bd6cd4990ab02a28035bb6f32bc738123d236 (
plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
|
;;; test-org-drill-config-commands.el --- Tests for org-drill edit / capture / refile -*- lexical-binding: t; -*-
;;; Commentary:
;; Sibling `test-org-drill-config.el' covers the file/dir picker and
;; `cj/drill-start' / `cj/drill-this-file'. This file covers the
;; three remaining wrapper commands:
;;
;; cj/drill-edit
;; cj/drill-capture
;; cj/drill-refile
;;; Code:
(require 'ert)
(require 'cl-lib)
(require 'package)
(package-initialize)
(require 'org)
(require 'org-capture)
(defvar cj/custom-keymap (make-sparse-keymap)
"Stub custom keymap for org-drill-config tests.")
(defvar drill-dir (file-name-as-directory
(expand-file-name "cj-drill-default" temporary-file-directory))
"Stub `drill-dir' for org-drill-config tests.")
;; `org-refile-targets' is defined by org-refile; declare at top level so
;; the module's `setq' on it reaches a dynamic binding under lexical scope.
(defvar org-refile-targets nil
"Dynamic stand-in for `org-refile-targets'.")
(add-to-list 'load-path (expand-file-name "modules" user-emacs-directory))
(require 'org-drill-config)
;;; cj/drill-edit
(ert-deftest test-org-drill-edit-opens-the-picked-file ()
"Normal: drill-edit picks a file and visits it (no `org-drill' call)."
(let (opened (drilled 0))
(cl-letf (((symbol-function 'cj/--drill-pick-file)
(lambda (_dir) "/decks/german.org"))
((symbol-function 'find-file) (lambda (f) (setq opened f)))
((symbol-function 'org-drill)
(lambda (&rest _) (cl-incf drilled))))
(cj/drill-edit))
(should (equal "/decks/german.org" opened))
(should (= 0 drilled))))
(ert-deftest test-org-drill-edit-prefix-uses-prompted-dir ()
"Normal: a prefix arg routes the file pick through the prompted directory."
(let ((tmp (file-name-as-directory (make-temp-file "cj-drill-edit-" t)))
opened)
(unwind-protect
(progn
(with-temp-file (expand-file-name "latin.org" tmp))
(cl-letf (((symbol-function 'read-directory-name) (lambda (&rest _) tmp))
((symbol-function 'completing-read) (lambda (&rest _) "latin.org"))
((symbol-function 'find-file) (lambda (f) (setq opened f))))
(cj/drill-edit t))
(should (equal (expand-file-name "latin.org" tmp) opened)))
(delete-directory tmp t))))
;;; cj/drill-capture
(ert-deftest test-org-drill-capture-routes-through-d-template ()
"Normal: drill-capture calls `org-capture' with the \"d\" template key."
(let (capture-key)
(cl-letf (((symbol-function 'org-capture)
(lambda (_arg key) (setq capture-key key))))
(cj/drill-capture))
(should (equal "d" capture-key))))
;;; cj/drill-refile
(ert-deftest test-org-drill-refile-targets-from-validated-helper ()
"Normal: drill-refile builds its drill targets from the shared
`cj/--drill-files-or-error' helper, expanded against `drill-dir' — not from
a raw `directory-files' call (so it inherits the helper's dot-file exclusion
and validation)."
(let ((drill-dir "/tmp/cj-drill/")
seen-targets called-fn)
(cl-letf (((symbol-function 'cj/--drill-files-or-error)
(lambda (_dir) '("a.org" "b.org")))
;; If the old raw path were still in use it would call
;; `directory-files'; a sentinel here keeps it from masquerading.
((symbol-function 'directory-files)
(lambda (&rest _) '("/WRONG/raw.org")))
((symbol-function 'call-interactively)
(lambda (fn)
(setq called-fn fn
seen-targets org-refile-targets))))
(cj/drill-refile))
(should (eq called-fn 'org-refile))
(should (= 2 (length seen-targets)))
(should (assoc nil seen-targets))
(should (equal (car (nth 1 seen-targets))
'("/tmp/cj-drill/a.org" "/tmp/cj-drill/b.org")))))
(ert-deftest test-org-drill-refile-does-not-clobber-global-targets ()
"Error: drill-refile let-binds `org-refile-targets'; the session-wide value
survives the call instead of being permanently replaced."
(let ((drill-dir "/tmp/cj-drill/")
(org-refile-targets '((sentinel :maxlevel . 9))))
(cl-letf (((symbol-function 'cj/--drill-files-or-error) (lambda (_dir) '("a.org")))
((symbol-function 'call-interactively) (lambda (_fn) nil)))
(cj/drill-refile))
(should (equal org-refile-targets '((sentinel :maxlevel . 9))))))
(ert-deftest test-org-drill-refile-errors-on-missing-drill-dir ()
"Error: a missing or unreadable drill dir signals a clear `user-error' via
the shared validated helper, instead of a low-level error, and never reaches
`org-refile'."
(let ((drill-dir (expand-file-name "cj-drill-nonexistent-XYZ/"
temporary-file-directory))
(called nil))
(cl-letf (((symbol-function 'call-interactively) (lambda (_fn) (setq called t))))
(should-error (cj/drill-refile) :type 'user-error))
(should-not called)))
(provide 'test-org-drill-config-commands)
;;; test-org-drill-config-commands.el ends here
|
