search

codenautas / qa-control

QA control of things that nobody controls
MIT License
1 stars 0 forks source link

Agregar un test de integridad que busque todos los ID de warnings #24

Closed emilioplatzer closed 9 years ago

emilioplatzer commented 9 years ago

El test debe abrir el archivo qa-control.js como un archivo de texto. Buscar todas las ocurrencias de warning: string (ya sea con o sin comillas dobles o simples) y luego verificar que figure en todos los idiomas (salvo inglés que se genera automático).

Por ejemplo si dentro del código figura: {warning:'missing_mandatory_cucarda_1',

hay que reivsar que esté (porque si no no lo muestra bien en qa-control-run) y siempre nos podemos olvidar de agregarlo.

diegoefe commented 9 years ago

Fijate si el test de integridad que agregué te satisface y cerrá el issue si es así. La ventaja que tiene es que te dice el número de línea de la falla. La desventaja es que no captura cosas como: warning: 'esto_es_un_warning' Si no te parece suficiente, avisame que lo extiendo para que soporte más casos

diegoefe commented 9 years ago

Con este método detecté que el incorrecto uso de 'missing_mandatory_cucarda_1' cuando iba 'lack_of_mandatory_cucarda_1'

emilioplatzer commented 9 years ago

Agregá que no permita warning: sin nada después, así obligamos a que esté en la misma línea

On Tue, Aug 18, 2015 at 9:49 AM, Diego Florio notifications@github.com wrote:

Con este método detecté que el incorrecto uso de 'missing_mandatory_cucarda_1' cuando iba 'lack_of_mandatory_cucarda_1'

— Reply to this email directly or view it on GitHub https://github.com/codenautas/qa-control/issues/24#issuecomment-132196321 .

diegoefe commented 9 years ago

ok