adopted-ember-addons / adopted-ember-addons/ember-qunit-nice-errors

Is there a way to remove the stack trace and improve the actual/expected messages?

Abierto
#65 1 comentario 0 reacciones 0 asignados Ver en GitHub
Lenguaje dominante
JavaScript
Estrellas
58
Forks
12
Merge medio
6 h 27 min
PR fusionados (30 d)
2

Descripción

Today errors look like this:

```
Integration | Component | x-form/x-radio-set: it should render a label if provided
✘ assert.equal(this.$('[for$="-input"]').text().trim(), 'Subject') at dummy/tests/integration/components/x-form/x-radio-set-test.js:60:4
at Object. (http://localhost:7357/assets/tests.js:802:10)
at runTest (http://localhost:7357/assets/test-support.js:4108:30)
at Test.run (http://localhost:7357/assets/test-support.js:4094:6)
at http://localhost:7357/assets/test-support.js:4300:12
at Object.advance (http://localhost:7357/assets/test-support.js:3746:26)
at begin (http://localhost:7357/assets/test-support.js:5567:20)
expected Subject
```

Would be nicer and cleaner to show something like this:

```
Integration | Component | x-form/x-radio-set: it should render a label if provided
✘ assert.equal(this.$('[for$="-input"]').text().trim(), 'Subject') at dummy/tests/integration/components/x-form/x-radio-set-test.js:60:4
expected Subject
```

Also would be even nicer to force it to always show the "Actual" value, even if it's `null`, `undefined` or `''`, wrapping it in `"` so we can tell there is actually a string, or even to tell the difference between `123` and `"123"`.

```
Integration | Component | x-form/x-radio-set: it should render a label if provided
✘ assert.equal(this.$('[for$="-input"]').text().trim(), 'Subject') at dummy/tests/integration/components/x-form/x-radio-set-test.js:60:4
actual ""
expected "Subject"
```

Guía de contribución

Abrir la guía de contribución

Evaluación

Este issue todavía no se ha evaluado.

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.