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 件 担当者 0 名 GitHub で見る
主要言語
JavaScript
スター
58
フォーク
12
平均マージ
6時間 27分
マージ済み PR(30日)
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 を短くまとめたダイジェスト。