UX Enhancements

[fdodino]

Issues a atacar

• [X] #147 (falta mergear TS y releasear y posiblemente migrar references.ts)
• [x] #146
• [x] #149
• [x] #92
• [x] #121
• [x] #122
• [x] #111

[fdodino]

@PalumboN @ivojawer ésto ya estaría, falta mergear los PRs que están dando vuelta en wollok-ts y releasear para que el CI pase.

[codecov[bot]]

Codecov Report

Attention: Patch coverage is 85.71429% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 83.44%. Comparing base (df35b09) to head (11e985c).

Files	Patch %	Lines
client/src/extension.ts	33.33%	2 Missing :warning:

Additional details and impacted files

```diff @@ Coverage Diff @@ ## master #160 +/- ## ========================================== + Coverage 83.21% 83.44% +0.23% ========================================== Files 17 17 Lines 429 435 +6 Branches 100 97 -3 ========================================== + Hits 357 363 +6 Misses 72 72 ``` | [Flag](https://app.codecov.io/gh/uqbar-project/wollok-lsp-ide/pull/160/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uqbar-project) | Coverage Δ | | |---|---|---| | [lsp-ide-client](https://app.codecov.io/gh/uqbar-project/wollok-lsp-ide/pull/160/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uqbar-project) | `93.28% <75.00%> (-0.66%)` | :arrow_down: | | [lsp-ide-server](https://app.codecov.io/gh/uqbar-project/wollok-lsp-ide/pull/160/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uqbar-project) | `79.06% <100.00%> (+0.61%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=uqbar-project#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[fdodino]

Uf, @PalumboN @ivojawer hoy terminé de cerrar el PR, pero me encontré con un error en los tests de hover, hay dos atributos undefined nuevos. Le metí un hack para eliminar esos atributos vacíos:

async function testHover(uri: Uri, position: Position, expected: any): Promise<void> {
  await activate(uri)
  const actual = await commands.executeCommand('vscode.executeHoverProvider', uri, position)
//
  delete actual[0]['canDecreaseHover']
  delete actual[0]['canIncreaseHover']
//
  assert.deepEqual(actual, [expected])

El error que tira el CI también me lo daba en mi máquina.

[PalumboN]

Uf, @PalumboN @ivojawer hoy terminé de cerrar el PR, pero me encontré con un error en los tests de hover, hay dos atributos undefined nuevos.

Esto fue porque actualizamos alguna versión de algo o qué onda? El hover igual sigue andando bien, no?

[fdodino]

Uf, @PalumboN @ivojawer hoy terminé de cerrar el PR, pero me encontré con un error en los tests de hover, hay dos atributos undefined nuevos.

Esto fue porque actualizamos alguna versión de algo o qué onda? El hover igual sigue andando bien, no?

woww!! nunca había visto funcionando el hover, es el debugger del AST en runtime! Sí , funciona ok.