Bug#1142936: vim-youcompleteme: Test_Select_Next_Previous_InsertModeMapping in upstream-vim-tests/completion*.vim fails with recent Vim

--ml77553yym5jknv7
Content-Type: multipart/mixed; protected-headers=v1;
	boundary="ikzghdj6kgowqv6l"
Content-Disposition: inline
Subject: Re: Bug#1142936: vim-youcompleteme:
 Test_Select_Next_Previous_InsertModeMapping in
 upstream-vim-tests/completion*.vim fails with recent Vim
MIME-Version: 1.0


--ikzghdj6kgowqv6l
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

Control: severity -1 normal

Am Wed, Jul 29, 2026 at 09:37:14AM +0200, schrieb David Kalnischkies:
> The later does not happen anymore with the remap of <C-n>,
> but only for <C-n>. <Down> directly works as before.
> That seems like a very odd behaviour change=E2=80=A6

I still think its an odd change and wonder what caused it, but it seems
in line with what happens in the 'normal' OmniComplete (in all 3 tested
versions of vim) that the line isn't changed on a remapped <C-n>
(my feeble attempt at reproducing this in bare vim attached).

So in some sense this change in behavior is aligning it closer to how
vim itself behaves=E2=80=A6 except that it is explicitly tested to not beha=
ve
this way in their tests=E2=80=A6


> It seems to not effect anything big at least, so if it helps I can
> disable the test for now so the CVE fixes and/or perl can pass on
> to testing and worry about this at a later point in time=E2=80=A6 ?

I still believe that this is an odd behaviour change triggered by
vim, so investigating it eventually would be nice, but to remove
the pressure on what seems rather minor of an issue, I will disable
the failing test for now as suggested previously
(just uploaded, should be available in a short while).


Best regards

David Kalnischkies

--ikzghdj6kgowqv6l
Content-Type: text/plain; charset=us-ascii
Content-Disposition: attachment; filename="reproducer.vim"

set encoding=utf-8

function GetLine(id)
        if getline('.') != 'baz'
                throw "Line is: '" . getline('.') . "' should be: 'baz'"
        endif
endfunction

function PickCompletion(id)
        call feedkeys("\<C-n>")
        call timer_start(1000, funcref('GetLine'))
endfunction

function Remap(id)
        inoremap <C-n> <Down>
        call timer_start(1000, funcref('PickCompletion'))
endfunction

function TriggerCompletion(id)
        call feedkeys("\<C-n>")
        call timer_start(1000, funcref('Remap'))
endfunction

function Setup()
        call feedkeys("ifoo\<Enter>bar\<Enter>baz\<Enter>ball\<Enter>ba")
        call timer_start(1000, funcref('TriggerCompletion'))
endfunction

call Setup()

--ikzghdj6kgowqv6l--

--ml77553yym5jknv7
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCgAdFiEE5sn+Q4uCja/tn0GrMRvlz3HQeIMFAmpwlEEACgkQMRvlz3HQ
eIPi7g//XaGHEEvLz4V0D8qQ/Dv7OWW07IEcUd6dvb1wc00BUNEitC12Cj+U1iMH
etxNjSt03NWJpOF6Q/LATdoKldxwbnFgXwRHI0ciME5ksqo36/sObm2D5/B2syZN
lnExDDKsbhVg3XIG8UmS1X8BRgD4UcgLooSnld0olNmNsvrBus9VZ+sG5IverTmG
CgVFPs53V287Bam/f3QkTdhDEjTsIkph1DWaRi0iwdUV5Pwfx9RP0lEK79xPZHSN
S05rYROFjLkeaDd98ydGxsdQgxsX+YbhHxcc+ZODRGUGhHzpWWjyyAcXb01lWF1x
jMjarnahZ5E/ADlhjhyzlVq8XoixMhSlR2PpU93xHa9XrDTOLA7iAxZJuT6tXbdR
D0/lIVtfeGQ0YWFj0+nXMgaxBDftonPqVj4/c1ROAmgHIZo7DWwbwlK1gQIe6sFb
EuUf5FYGMbb57aGI9X0K58qkRQ7kjddpNkfEsnEey57VyCb0oah3AM7NoeT5usZm
vqrM3y5BOWrJapZTuOjfRM67KX5KBzTYdrs1s5ycyY5119hRdeCDt+9ZoOHm6IAY
IwzFPEN98iIXhrze8WN33yDyczyc6YM32DKbs9KR/botS0VK15ToyB4ofLBuy3LJ
ciatxbWV0bx2sqKXWzxgugtYUONg9nwW7P9vgut8Mj3PO6U29mI=
=WEOV
-----END PGP SIGNATURE-----

--ml77553yym5jknv7--