UX Enhancements

[fdodino]

Issues a atacar

• Mover funciones a Wollok-TS #147 (falta mergear TS y releasear y posiblemente migrar references.ts)
• Refactor de los try - catches #146
• Renombrar un archivo con errores deja Problems zombies #149
• El spinner no desaparece si hay errores #92
• Agregar configuración: milisegundos para abrir el diagrama dinámico en el navegador #121
• Inactivar el diagrama en el ts-cli si no querés activarlo #122
• Visualizar íconos diferentes para programas y tests #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.

[ivojawer]

Hermoso PR!! Me gusta como quedaron los errores ❣️

[ivojawer]

Por si te complicaba la def de este archivo, por las dudas dejo link a la version en svg de este icono: https://drive.google.com/drive/folders/1h_GfnEt2FdGlOD7U_b-mNBr8eBhfiM_8?usp=drive_link

[codecov]

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 ⚠️

Additional details and impacted files

@@            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	Coverage Δ
lsp-ide-client	93.28% <75.00%> (-0.66%)	⬇️
lsp-ide-server	79.06% <100.00%> (+0.61%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 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]

GROSO Dodaiiiiinnnnnn!!! Siempre mejorando todo a su paso 🌱 🚶 🌹 🔆

[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.