diff options
| author | Craig Jennings <c@cjennings.net> | 2026-05-27 19:19:35 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-05-27 19:19:35 -0500 |
| commit | a74334bc9a321ddbb9b2ba81266f264d3522cc8c (patch) | |
| tree | 72214b77d2b27b205f9450763fdfd00ee4166524 /tests/test-pearl-sync-wrappers.el | |
| parent | 73f7656d02f916c361fc0da06d97d9bdaf9062a0 (diff) | |
| download | pearl-a74334bc9a321ddbb9b2ba81266f264d3522cc8c.tar.gz pearl-a74334bc9a321ddbb9b2ba81266f264d3522cc8c.zip | |
refactor: verb-align command names and prefixes, use issue not ticket
The keybinding prefixes now each name an action verb: f fetch, e edit, c create, k delete, o open, y copy. The url group is gone, its actions split into open and copy, and the direct lowercase c (new comment) is dropped so c can head the create group. Two commands carried non-verb names, so I renamed them: pearl-new-issue is now pearl-create-issue and pearl-add-comment is now pearl-create-comment. The old sync helper that already held the pearl-create-issue name moved to the private pearl--create-issue, where it belongs as internal plumbing.
I also swept "ticket" to "issue" across the labels, transient, docstrings, and README. Linear's own term is "issue" and the buffer mirrors Linear, so the command names already said "issue" while the labels said "ticket". Now they agree. The transient's per-entry keys still trail the keymap verbs, which stays its own task.
Diffstat (limited to 'tests/test-pearl-sync-wrappers.el')
| -rw-r--r-- | tests/test-pearl-sync-wrappers.el | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tests/test-pearl-sync-wrappers.el b/tests/test-pearl-sync-wrappers.el index a5c9c70..cb4ac96 100644 --- a/tests/test-pearl-sync-wrappers.el +++ b/tests/test-pearl-sync-wrappers.el @@ -43,11 +43,11 @@ '((data (team (states (nodes . (((id . "s1") (name . "Todo")))))) )) (should (= 1 (length (pearl-get-states "team-1")))))) -(ert-deftest test-pearl-create-issue-sync-returns-issue () +(ert-deftest test-pearl--create-issue-sync-returns-issue () "The sync create wrapper returns the created issue node." (testutil-linear-with-response '((data (issueCreate (success . t) (issue (id . "i1") (identifier . "ENG-1") (title . "T"))))) - (should (string-equal "ENG-1" (cdr (assoc 'identifier (pearl-create-issue "T" "" "team"))))))) + (should (string-equal "ENG-1" (cdr (assoc 'identifier (pearl--create-issue "T" "" "team"))))))) (ert-deftest test-pearl-sync-wrapper-times-out-without-callback () "A sync wrapper returns nil instead of hanging when no callback fires." |