wollok
wollok copied to clipboard
uqbar-project
Ejecutar tests de carpetas que tienen espacios rompe el launcher
Si intentamos generar un directorio con espacios y ejecutamos wollok test desde el wollok-cli se rompe, principalmente porque el launcher de Wollok Xtext está tirando un NPE. Habría que investigarlo.
Bueno, el problema es que el CommandLine de apache-cli es muy básico, no se banca bien los espacios como picocli: https://github.com/remkop/picocli/. El tema es que no quiero poner una biblioteca adicional que se sumaría al target platform, así que una jugarreta que podemos hacer es convertir los espacios en otro caracter en el build y reconvertirlo de nuevo en espacio en el parse.
Ver WollokLauncherParameters.xtend
Bueno, tengo el branch fix-#1851-allow-folder-in-spaces-for-tests que resuelve ese problema, incluso tengo el fix en wollok-cli, corren
- tests del proyecto, de una carpeta con espacios, dos o tres tests seleccionados a mano, y un test solo con o sin diagrama dinámico
- programas
El tema con el REPL es que hace un import a manopla. Si la carpeta donde está el proyecto tiene espacio, no habría drama, pero si elegís un package con espacio, eso rompe el modelo de imports que tenemos. Y acá medio que me agarró la duda y los arrobo @PalumboN @nscarcella @lspigariol @asanzo : hoy estamos impidiendo desde el IDE que creen una carpeta con espacio, entonces me parece que no vale la pena seguir metiéndole esfuerzo para que desde el wollok-cli o desde otra herramienta metamos una carpeta con espacio y se rompa todo. Qué opinan?
Perdón, no entendí: ¿El problema es sólo con los tests? ¿Es sólo con los packages? ¿Qué tienen que ver los packages con las carpetas? ¿Esto sucede con el IDE puesto en una carpeta con espacios? ¿O sólo con nombres de archivo/de carpeta del proyecto que tengan espacios? No sabía que se podía tener un nombre de package con espacios.
Perdón por mi falta de conocimiento :face_with_head_bandage:
Hoy el IDE no te deja crear una carpeta con espacios ni caracteres raros, esto es dentro del proyecto. Si usás un workspace (espacio de trabajo) donde adentro del proyecto respetás todo, entiendo que se puede. Tendría que rechequearlo. Lo que no se puede es dentro del proyecto tener subcarpetas con espacios. La verdad es que es complicado hacerlo andar tanto para Linux como para Windows porque encima trabajan distinto. No se, por eso me embalé a hacerlo andar pero después me pareció innecesario el feature, a lo sumo trataría de mejorar el mensaje de error.
- Poder poner la carpeta del eclipse dentro de un path que tenga espacios me parece hermoso feature (uno no espera que falle, copia medio a ciegas). Pero si no se puede, no me caliento, lo dejamos bien clarito.
- Poder tener carpetas con espacios dentro del proyecto me friega. Me gusta la de tener un error descriptivo.
:heart: