diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-26 15:06:42 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-26 15:06:42 -0500 |
| commit | 02d1c7b009f109a83a6c62ae56f4a3a9ba2d21e3 (patch) | |
| tree | 40e2d12dfede9e93423a3fc163229a8d453fe265 /tests/test-pearl-org-write.el | |
| parent | 41a33964c6b97858a17fd12288592b03d5df68d9 (diff) | |
| download | pearl-02d1c7b009f109a83a6c62ae56f4a3a9ba2d21e3.tar.gz pearl-02d1c7b009f109a83a6c62ae56f4a3a9ba2d21e3.zip | |
refactor: remove the dead state-to-todo mapping subsystem
`pearl-state-to-todo-mapping` and its derived regex stopped doing real work once keyword rendering moved to slugifying the Linear state name (`pearl--state-name-to-keyword`) and the keyword-to-state direction moved to resolving through the team's workflow states. The defcustom fed only `pearl--get-todo-states-pattern`, which had no caller left, and the two cache vars existed only to serve that dead function. State extraction reads `org-get-todo-state` off the buffer's `#+TODO` line, so nothing live touched the mapping.
I removed the defcustom, both cache vars, and the dead function, and dropped the three pattern tests that exercised it. Removing the defcustom orphaned the mapping bindings that ~12 test setup macros still carried. They were passing through slugification rather than the map, so I stripped them too and kept the `org-todo-keywords` bindings the temp buffers actually need. Also dropped the stale README config row.
Diffstat (limited to 'tests/test-pearl-org-write.el')
| -rw-r--r-- | tests/test-pearl-org-write.el | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/tests/test-pearl-org-write.el b/tests/test-pearl-org-write.el index 6491b54..6c30380 100644 --- a/tests/test-pearl-org-write.el +++ b/tests/test-pearl-org-write.el @@ -33,12 +33,10 @@ "One normalized issue, enough to render a recognizable org entry.") (defmacro test-pearl--with-org-file (var &rest body) - "Bind VAR to a fresh temp .org path and run BODY, cleaning up after. -The state mapping is bound so rendering is deterministic." + "Bind VAR to a fresh temp .org path and run BODY, cleaning up after." (declare (indent 1)) `(let* ((,var (make-temp-file "linear-test-" nil ".org")) - (pearl-org-file-path ,var) - (pearl-state-to-todo-mapping '(("Todo" . "TODO")))) + (pearl-org-file-path ,var)) (unwind-protect (progn ,@body) (let ((b (find-buffer-visiting ,var))) |
