bug#68213: 30.0.50; completion-preview-tests failure in --without-x build

[john muhl via "Bug reports for GNU Emacs, the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>]

Eshel Yaron <me@eshelyaron.com> writes:

> Hi John, Eli,
>
> Eli Zaretskii <eliz@gnu.org> writes:
>
>>> Date: Tue, 02 Jan 2024 10:42:59 -0600
>>> From:  john muhl via "Bug reports for GNU Emacs,
>>>  the Swiss army knife of text editors" <bug-gnu-emacs@gnu.org>
>>> 
>>> Running ‘make -k -C test check-lisp’ on master with a build
>>> configured without X produces the following error:
>>> 
>>>   cd emacs
>>>   ./autogen.sh
>>>   ./configure --without-x
>>>   make -j$(nproc)
>>>   make -k -C test check-lisp
>>>   …
>>>   In toplevel form:
>>>   lisp/completion-preview-tests.el:23:2: Error: Symbol's value
>>>   as variable is void: mouse-wheel-up-event
>>>   make[3]: *** [Makefile:159: lisp/completion-preview-tests.elc] Error 1
>>>   make[3]: Target 'lisp/completion-preview-tests.log' not remade because of errors.
>>
>> Thanks, please try again, I hope I fixed this.
>
> FWIW, I was going to suggest the attached patch, following how
> flymake.el handles this.  But if the current solution works, that seems
> just as good.
>
>
> Thanks,
>
> Eshel
>
> [2. text/x-patch; 0001-Explicitly-require-mwheel-in-completion-preview.el.patch]...

That was my first thought too but it led to another error:

  lisp/completion-preview-tests.el:23:2: Error: Duplicate
  definition for key: "<mouse-5>" (keymap (mouse-5 .
  completion-preview-prev-candidate) (down-mouse-2 .
  completion-at-point) (C-down-mouse-1 . completion-at-point)
  (down-mouse-1 . completion-preview-insert))