diff options
| author | Philipp Stephani <phst@google.com> | 2021-04-23 12:52:51 +0200 |
|---|---|---|
| committer | Philipp Stephani <phst@google.com> | 2021-04-23 12:52:51 +0200 |
| commit | 3806d2168bdc36796d2dfac5f743582f0ce71c72 (patch) | |
| tree | 9f6e92bfeb0774addb2572cc6ca4a1dde2751fdd /test/lisp/minibuffer-tests.el | |
| parent | 804f5acda4bd947f058c6843e373797202314467 (diff) | |
| download | emacs-3806d2168bdc36796d2dfac5f743582f0ce71c72.tar.gz emacs-3806d2168bdc36796d2dfac5f743582f0ce71c72.tar.bz2 emacs-3806d2168bdc36796d2dfac5f743582f0ce71c72.zip | |
Fix small bug in 'completion-table-subvert'.
Even for a trivial underlying completion table (where the 'boundaries'
action returns nil), we need to provide nontrivial boundaries so that
they match the behavior of 'all-completions'.
* lisp/minibuffer.el (completion-table-subvert): Return boundaries
even for trivial underlying completion table.
* test/lisp/minibuffer-tests.el (completion-table-subvert-test):
Amend unit test to also test boundaries. While there, also test
the other completion functions.
Diffstat (limited to 'test/lisp/minibuffer-tests.el')
| -rw-r--r-- | test/lisp/minibuffer-tests.el | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/test/lisp/minibuffer-tests.el b/test/lisp/minibuffer-tests.el index 791e51cdcd5..027711c21e6 100644 --- a/test/lisp/minibuffer-tests.el +++ b/test/lisp/minibuffer-tests.el @@ -83,7 +83,12 @@ (let* ((origtable '("A-hello" "A-there")) (subvtable (completion-table-subvert origtable "B" "A"))) (should (equal (try-completion "B-hel" subvtable) - "B-hello")))) + "B-hello")) + (should (equal (all-completions "B-hel" subvtable) '("-hello"))) + (should (test-completion "B-hello" subvtable)) + (should (equal (completion-boundaries "B-hel" subvtable + nil "suffix") + '(1 . 6))))) (ert-deftest completion-table-test-quoting () (let ((process-environment |