Skip to content

Commit 9366538

Browse files
joborJoerg Bornemann
andauthored
Emacs 29 (#459)
* Fix dumb-jump-goto-file-line-test for Emacs 29 (#442) In Emacs 29, the ring-insert function is not called anymore when dumb-jump-goto-file-line is called. The corresponding test relied on an implementation detail of xref-push-marker-stack. See the longer explanation in #442. Fix the test by guarding the check for the ring-insert call with a Emacs version check. * Add Emacs 29.4 to CI --------- Co-authored-by: Joerg Bornemann <[email protected]>
1 parent 9f3967b commit 9366538

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

.github/workflows/test.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@ jobs:
2121
- '26.3'
2222
- '27.1'
2323
- '28.1'
24+
- '29.4'
2425
cask_version:
2526
- 'snapshot'
2627
steps:

test/dumb-jump-test.el

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -348,7 +348,8 @@
348348
(ert-deftest dumb-jump-goto-file-line-test ()
349349
(let ((js-file (f-join test-data-dir-proj1 "src" "js" "fake.js")))
350350
(with-mock
351-
(mock (ring-insert * *))
351+
(when (version< emacs-version "29")
352+
(mock (ring-insert * *)))
352353
(dumb-jump-goto-file-line js-file 3 0)
353354
(should (string= (buffer-file-name) js-file))
354355
(should (= (line-number-at-pos) 3)))))

0 commit comments

Comments
 (0)