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?

Ouverte
#65 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
JavaScript
Étoiles
58
Forks
12
Merge moyen
6 h 27 min
PR mergées (30 j)
2

Description

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"
```

Guide de contribution

Ouvrir le guide de contribution

Évaluation

Cette issue n'a pas encore été évaluée.

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.