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?

未关闭
#65 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看
主要语言
JavaScript
星标
58
派生
12
平均合并
6 小时 27 分钟
30 天内合并 PR
2

描述

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

贡献指南

打开贡献指南

评估

这个 Issue 还没有评估数据。

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。