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?

Open
#65 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
58
Forks
12
Avg merge
6h 27m
Merged PRs (30d)
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"
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.