diff options
| author | Craig Jennings <c@cjennings.net> | 2026-02-15 19:58:15 -0600 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-02-15 19:58:15 -0600 |
| commit | 61bfb79f2c55983697f87b9ba3961a9fb46de2fe (patch) | |
| tree | 6a9ebc025dcd71cba68f4612aef7b658781f4b8f /tests/test-integration-mousetrap-mode-profiles.el | |
| parent | 8eab9ccde339829c1866a40f51f942a6b36f2b02 (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 'tests/test-integration-mousetrap-mode-profiles.el')
| -rw-r--r-- | tests/test-integration-mousetrap-mode-profiles.el | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/tests/test-integration-mousetrap-mode-profiles.el b/tests/test-integration-mousetrap-mode-profiles.el index 6abd3ad2..741df442 100644 --- a/tests/test-integration-mousetrap-mode-profiles.el +++ b/tests/test-integration-mousetrap-mode-profiles.el @@ -55,26 +55,26 @@ Validates: (should (eq (lookup-key map (kbd "<wheel-up>")) nil)) (should (eq (lookup-key map (kbd "<drag-mouse-1>")) nil))))) -(ert-deftest test-integration-mousetrap-mode-profiles-dashboard-primary-click-only () - "Test dashboard-mode gets primary-click profile. +(ert-deftest test-integration-mousetrap-mode-profiles-dashboard-scroll-and-primary () + "Test dashboard-mode gets scroll+primary profile. Components integrated: - mouse-trap--get-profile-for-mode (lookup) - mouse-trap--build-keymap (selective event binding) Validates: -- Primary-click profile allows mouse-1 -- Blocks mouse-2/3 and scroll events" +- scroll+primary profile allows mouse-1 and scrolling +- Blocks mouse-2/3" (let ((major-mode 'dashboard-mode)) (let ((profile (mouse-trap--get-profile-for-mode)) (map (mouse-trap--build-keymap))) - (should (eq 'primary-click profile)) + (should (eq 'scroll+primary profile)) ;; mouse-1 allowed (should (eq (lookup-key map (kbd "<mouse-1>")) nil)) + ;; scroll allowed + (should (eq (lookup-key map (kbd "<wheel-up>")) nil)) ;; mouse-2/3 blocked - (should (eq (lookup-key map (kbd "<mouse-2>")) 'ignore)) - ;; scroll blocked - (should (eq (lookup-key map (kbd "<wheel-up>")) 'ignore))))) + (should (eq (lookup-key map (kbd "<mouse-2>")) 'ignore))))) (ert-deftest test-integration-mousetrap-mode-profiles-emacs-lisp-uses-default-disabled () "Test unmapped mode uses default disabled profile. |
