angular / angular/angular-cli

Unit test: Add reporter watermark configuration to angular.json

オープン
#31,998 コメント 3 件 リアクション 0 件 担当者 0 名 GitHub で見る
angular/build:unit-test area: @angular/build feature feature: insufficient votes
主要言語
TypeScript
スター
27k
フォーク
11.8k
平均マージ
14時間 23分
マージ済み PR(30日)
162

説明

### Command

test

### Description

After taking advantage of all the new Vitest unit test features in 21.0.0, there is only one configuration item left in the vitest-base.config.ts:

```
export default defineConfig({
test: {
slowTestThreshold: 1000
}
});
```

This is required since the Vitest's [slowTestThreshold](https://vitest.dev/guide/cli.html#slowtestthreshold) default of 300 ms is not adequate for extended test cases.

Ideally this configuration should be available through the angular.json configuration unit-test options, since it would allow the complete removal of the vitest-base.config.ts file.

Maybe follow the nomenclature that you introduced with "coverageWatermarks" and call it "reporterWatermark".

### Describe the solution you'd like

_No response_

### Describe alternatives you've considered

_No response_

コントリビューションガイド

コントリビューションガイドを開く

調査の方向性

angular.json のユニットテストオプションと vitest-base.config.ts から始め、次にリポジトリのユニットテストコマンドを実行して、オプションがどのように Vitest に渡されるかを追跡します。遅いテストのしきい値として望ましい設定が angular.json を通じて受け入れられること、および設定を失わずに vitest-base.config.ts を削除できることを確認します。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
angular, typescript
領域
cli, testing, tooling
issue の種類
機能追加
難易度
4/5
見積もり時間
3〜5日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
45/100

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。