What does this PR do?

The previous behavior for hover replaces indentations with the   entity, because Markdown treats whitespace at the beginning of a line in a special way. However, when the indentation setting wasn't set, the regex that does that replace was set to empty ("") and so would include an   entity between every character.

This went under the radar because the previous PR #844 also changed the tests setup to include a .withIndentation call, which made sure all tests were passing.

In this PR I'm reverting that to the default behavior for most tests, but overriding the default only on the test introduced in #844.

What issues does this PR fix or reference?

The issue is referenced on a comment of PR #844. here

Is it tested? How?

I have tested this fix with a neovim setup as described in the original problem report in the PR comment.

At the risk of simply repeating myself, below is the full set of reproduction steps:

local settings = {
  yaml = {
    schemas = {
      ["https://gitlab.com/gitlab-org/gitlab/-/raw/master/app/assets/javascripts/editor/schema/ci.json"] = ".gitlab-ci.y*l",
    },
    hover = true,
  }
}

vim.api.nvim_create_autocmd('FileType', {
  pattern = "yaml",
  callback = function(args)
    vim.lsp.start({
      name = 'yamlls',
      cmd = { 'yaml-language-server', '--stdio' }, -- replace with local installation as necessary
      root_dir = vim.fn.fnamemodify('./.gitlab-ci.yaml', ':p:h'),
      settings = settings
    })
  end
})

vim.keymap.set('n', '<space>', vim.lsp.buf.hover, {})

stages:
  - hello