Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

diagnostics.test.vim fail when running with Vim v9.1.0949 or newer #4284

Open
6 of 12 tasks
jamessan opened this issue Jan 1, 2025 · 3 comments
Open
6 of 12 tasks

diagnostics.test.vim fail when running with Vim v9.1.0949 or newer #4284

jamessan opened this issue Jan 1, 2025 · 3 comments

Comments

@jamessan
Copy link

jamessan commented Jan 1, 2025

Issue Prelude

Please complete these steps and check these boxes (by putting an x inside
the brackets) before filing your issue:

  • I have read and understood YCM's CONTRIBUTING document.
  • I have read and understood YCM's CODE_OF_CONDUCT document.
  • I have read and understood YCM's README, especially the
    Frequently Asked Questions section.
  • I have searched YCM's issue tracker to find issues similar to the one I'm
    about to report and couldn't find an answer to my problem. (Example Google
    search.
    )
  • If filing a bug report, I have included the output of vim --version.
  • If filing a bug report, I have included the output of :YcmDebugInfo.
  • If filing a bug report, I have attached the contents of the logfiles using
    the :YcmToggleLogs command.
  • If filing a bug report, I have included which OS (including specific OS
    version) I am using.
  • If filing a bug report, I have included a minimal test case that reproduces
    my issue, using vim -Nu /path/to/YCM/vimrc_ycm_minimal, including what I
    expected to happen and what actually happened.
  • If filing a installation failure report, I have included the entire output
    of install.py (or cmake/make/ninja) including its invocation
  • I understand this is an open-source project staffed by volunteers and
    that any help I receive is a selfless, heartfelt gift of their free time. I
    know I am not entitled to anything and will be polite and courteous.
  • I understand my issue may be closed if it becomes obvious I didn't
    actually perform all of these steps.

Thank you for adhering to this process! It ensures your issue is resolved
quickly and that neither your nor our time is needlessly wasted.

Issue Details

  • What did you do?

After updating the Vim package in Debian, the vim-youcompleteme package's tests started failing. I bisected the issue to @bstaletic's Vim patch v9.1.0949.

This might be related to the discussion that spawned after the patch was merged.

  1. Install v9.1.0949 or newer Vim
  2. Run ./test/run_vim_tests diagnostics.test.vim
  • What did you expect to happen?

All tests pass

  • What actually happened?

Test_ShowDetailedDiagnostic_Popup_MultilineDiagFromStartOfLine and Test_ShowDetailedDiagnostic_Popup_MultilineDiagNotFromStartOfLine fail. Debug logs are available in this gist.

@puremourning
Copy link
Member

Thanks for the report.

Ironically this seems like a vim bug not a YCM one though right (Dramatis personae notwithstanding)?

@jamessan
Copy link
Author

jamessan commented Jan 1, 2025

I wasn't sure which side of the fence the issue would land on, but figured it was at least worth having it known here.

@puremourning
Copy link
Member

Yeah it makes sense. I think we can check it and transfer if needed. Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants