lsp-completion: Fix triggerKind on manual invokation

[gexplorer]

Related to issue #2857.

If space is among a servers triggerCharacters for completion, it does not return any results after a space.

This happens because lsp-completion--get-context returns CompletionTriggerKind = 2 when completion is invoked manually but the previous character is one of triggerCharacters.

Steps to Reproduce

See example project

• Inside a vue file

Failed completion

• Try completion with cursor separated one space from the curly bracket:

import { | } from 'vue';

• The request is:

Params: {
  "textDocument": {
    "uri": "file:///Users/eder/Projects/veturpack/src/Test.vue"
  },
  "position": {
    "line": 7,
    "character": 9
  },
  "context": {
    "triggerKind": 2,
    "triggerCharacter": " "
  }
}

• It won't return results

Correct completion

• Try completion with cursor right after the curly bracket

import {|  } from 'vue';

• The request is:

Params: {
  "textDocument": {
    "uri": "file:///Users/eder/Projects/veturpack/src/Test.vue"
  },
  "position": {
    "line": 7,
    "character": 8
  },
  "context": {
    "triggerKind": 1
  }
}

[gexplorer]

Hey @kiennq do you know about the errors in the build? I'm not sure they are related to what I changed 😖

[yyoncho]

Hey @kiennq do you know about the errors in the build? I'm not sure they are related to what I changed

Errors seem to be unrelated.

[kiennq]

Hey @kiennq do you know about the errors in the build? I'm not sure they are related to what I changed 😖

We're not using Cask on Windows so I think you will need to update windows-boostrap.el to include el-mock

[kiennq]

Thank you