caolan / caolan/nodeunit

test.throws incorrectly implemented

Abierto
#156 2 comentarios 1 reacción 0 asignados Ver en GitHub
API Breaking Change
Lenguaje dominante
JavaScript
Estrellas
1.9k
Forks
359
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

According to http://nodejs.org/docs/v0.4.2/api/assert.html#assert.throws
... assert.throws can be passed three arguments:
`assert.throws(block, [error], [message])`

The `error` parameter can be constructor, regexp or validation function. Nodeunit assumes that `error` is a constructor and simply uses instanceof to validate. This behavior is incorrect. Regexp or validation function should be supported.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Línea de trabajo

Lee la implementación existente de assert.throws y sus pruebas, y luego compara el comportamiento con la documentación enlazada de Node.js sobre assert.throws. Se considera completado cuando se admiten argumentos de error de constructor, expresión regular y función de validación, en lugar de comprobar únicamente instanceof, con cobertura para cada forma.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
javascript, node.js
Área
testing-qa
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.