summaryrefslogtreecommitdiff
path: root/modules/mousetrap-mode.el
diff options
context:
space:
mode:
authorCraig Jennings <c@cjennings.net>2026-02-15 19:58:15 -0600
committerCraig Jennings <c@cjennings.net>2026-02-15 19:58:15 -0600
commit61bfb79f2c55983697f87b9ba3961a9fb46de2fe (patch)
tree6a9ebc025dcd71cba68f4612aef7b658781f4b8f /modules/mousetrap-mode.el
parent8eab9ccde339829c1866a40f51f942a6b36f2b02 (diff)
test: add 86 new tests, fix 3 production bugs, fix 8 stale tests
New test coverage (86 tests across 7 files): - custom-case: 43 tests (title-case-region, upcase-dwim, downcase-dwim) - custom-datetime: 10 tests (all insert methods with mocked time) - config-utilities: 11 tests (format-build-time type branching) - org-capture-config: 22 tests (date-prefix + event-content) Production bugs found and fixed: - custom-case: title-case-region crashes on whitespace/punctuation-only input (char-after returns nil when no word chars found) - org-capture-config: browser branch missing empty-string guard on :initial plist value, producing stray newline in capture output - mousetrap-mode: keymap never registered in minor-mode-map-alist, so mode was silently not blocking any mouse events. Now pushes/removes on toggle. Additional fixes: - local-repository: fix cons cell syntax bug in localrepo-initialize (was calling vars as functions instead of using cons) - dupre-theme tests: update rainbow-delimiter color expectations - mousetrap tests: update dashboard profile (primary-click → scroll+primary) - music completion test: bind completion-ignore-case to prevent pollution - Delete redundant interactive recording test (duplicate of ERT version) Refactoring: - org-capture-config: extract cj/org-capture--date-prefix pure function from cj/org-capture-format-event-headline for testability Test checklist: todo.org updated to [11/25], removed untestable modules
Diffstat (limited to 'modules/mousetrap-mode.el')
-rw-r--r--modules/mousetrap-mode.el8
1 files changed, 7 insertions, 1 deletions
diff --git a/modules/mousetrap-mode.el b/modules/mousetrap-mode.el
index 7ee91d3b..0df08d7c 100644
--- a/modules/mousetrap-mode.el
+++ b/modules/mousetrap-mode.el
@@ -178,10 +178,16 @@ See `mouse-trap-profiles' for available profiles and
(if mouse-trap-mode
(progn
(setq mouse-trap-mode-map (mouse-trap--build-keymap))
+ ;; Register keymap so Emacs actually uses it for key dispatch
+ (let ((entry (assq 'mouse-trap-mode minor-mode-map-alist)))
+ (if entry
+ (setcdr entry mouse-trap-mode-map)
+ (push (cons 'mouse-trap-mode mouse-trap-mode-map) minor-mode-map-alist)))
;; Add dynamic lighter to mode-line-misc-info (always visible)
(unless (member '(:eval (mouse-trap--lighter-string)) mode-line-misc-info)
(push '(:eval (mouse-trap--lighter-string)) mode-line-misc-info)))
- ;; When disabling, clear the keymap
+ ;; When disabling, remove keymap from minor-mode-map-alist
+ (setq minor-mode-map-alist (assq-delete-all 'mouse-trap-mode minor-mode-map-alist))
(setq mouse-trap-mode-map nil)
;; Note: We keep the lighter in mode-line-misc-info so it shows 🐭 when disabled
))