avajs/ava

Make t.fail() accept an object

Open

#2,615 opened on 2020年11月25日

GitHub で見る
 (5 comments) (1 reaction) (0 assignees)JavaScript (20,600 stars) (1,408 forks)batch import
enhancementhelp wantedscope:assertions

説明

It seems like the only problem with #1094 is providing detailed error output when a custom assertion fails. Everything else is straightforward. What if t.fail() could accept an object to use for creating an AssertionError? The fields could be message, actual, expected, and format that will specifying which formatting function to use for setting the values field on the error object. Could also add an optional assertion field for a custom assertion name.

Would this be compatible with #2435?

If there's interest, I can submit a PR for this.

コントリビューターガイド

技術スタック
javascriptnodejs
領域
testing
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-3 hours
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
clear
前提条件
Knowledge of AVAJavaScriptNode.js
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
50
調査方針
Investigate the current implementation of t.fail() in AVA's source code. Review issues #1094 and #2435 for context on desired behavior. Look at how AssertionError is created in the codebase. Consider adding a new function or modifying t.fail to accept an object with fields like message, actual, expected, format, and assertion. Ensure compatibility with existing error formatting.
Make t.fail() accept an object · avajs/ava#2615 | Good First Issue