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

Tests fail for full reindents with indent mode on non-nightly (0.10.0) Neovim #639

Closed
PriceHiller opened this issue Dec 27, 2023 · 0 comments · Fixed by #640
Closed

Tests fail for full reindents with indent mode on non-nightly (0.10.0) Neovim #639

PriceHiller opened this issue Dec 27, 2023 · 0 comments · Fixed by #640
Labels
bug Something isn't working

Comments

@PriceHiller
Copy link
Contributor

Describe the bug

On non-nightly versions of Neovim (<0.10.0) the tests for full reindent can intermittently fail.

These tests do not fail all the time, only occasionally -- this makes this a right headache to resolve. This problem was introduced by #629.

Steps to reproduce

Use a Neovim version in the 0.9.x range, ideally 0.9.0 as that exhibits the problem most consistently and then run (assuming a bash shell):

git clone https://github.com/nvim-orgmode/orgmode.git Orgmode \
 && cd Orgmode

TEST_RUN=1
while :; do
  printf "==== Current Test Run: %s ====\n" "${TEST_RUN}"
  if make test FILE=./tests/plenary/org/indent_spec.lua; then
    printf "\n\n++++ Finished Test Run: %s ++++\n\n" "${TEST_RUN}"
  else
    printf "\n\n==== Tests Failed on Run %s ====\n\n" "${TEST_RUN}"
    break
  fi
  ((TEST_RUN++))
done

Now wait for a failure (again use a Neovim version in range 0.9.x). It may take a while to happen.

Expected behavior

Tests consistently pass instead of only sometimes passing.

Emacs functionality

No response

Minimal init.lua

N/A. This is a testing issue.

Screenshots and recordings

orgmode-testing-bug

OS / Distro

Arch Linux -- Linux orion 6.6.8-arch1-1 #1 SMP PREEMPT_DYNAMIC Thu, 21 Dec 2023 19:01:01 +0000 x86_64 GNU/Linux

Neovim version/commit

0.9.1, 0.9.2, 0.9.4

Additional context

See here for where this problem was discovered.

This does not seem to affect nightly (0.10.0) versions of Neovim -- I can't cause any test failures on nightly at the time of writing.

This seemingly only impacts the tests, not any actual usage. I cannot replicate by opening a orgmode file in Neovim 0.9.0 several times. Seems fine from the user perspective. Purely a testing issue.

@kristijanhusak Opening this so its tracked, I've spent the last hour or so battling this and haven't made any progress. I'm going to finish up the Virtual indent stuff and try to solve this again when I'm not losing my sanity.

@PriceHiller PriceHiller added the bug Something isn't working label Dec 27, 2023
PriceHiller added a commit to PriceHiller/orgmode that referenced this issue Dec 28, 2023
PriceHiller added a commit to PriceHiller/orgmode that referenced this issue Dec 28, 2023
kristijanhusak pushed a commit that referenced this issue Jan 4, 2024
SlayerOfTheBad pushed a commit to SlayerOfTheBad/orgmode that referenced this issue Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant