avajs/ava

Print warning when test files take a disproportionate amount of time to declare tests

Open

#2,565 opened on 2020年8月22日

GitHub で見る
 (2 comments) (4 reactions) (1 assignee)JavaScript (20,600 stars) (1,408 forks)batch import
help wantedscope:reportersscope:scheduling

説明

Based on feedback from @shusson in https://github.com/avajs/ava/issues/2268#issuecomment-676513972 it would be interesting if we could instrument how long it takes between AVA loading the test file to tests being declared.

This time is not taken by AVA itself, but instead by the test and project code. There are likely optimizations possible in the project that is using AVA.

If this takes a disproportionate amount of time of the overall test run (5%? 10%?) we should print a warning after the test run has completed.

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

技術スタック
javascriptnodejs
領域
testingperformance
Issue 種別
feature
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
3
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
half day
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
stale
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
mostly clear
前提条件
Basic understanding of AVA internalsNode.js timing APIs
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
55
調査方針
Examine the test loading workflow in AVA, specifically the `lib/runner.js` or similar files that handle test declaration. Look at how long it takes from loading a test file to the first test declaration. Use `performance.now()` or similar to measure durations and compare to total test run time. Check existing comments on issue #2268 for context. The warning should be printed after the test run completes if the loading time exceeds a threshold like 5% or 10% of total time.