Remove use of `@sealed` in DevTools
オープン
P3
tech debt
- 主要言語
- Dart
- スター
- 1.7k
- フォーク
- 404
- 平均マージ
- 6日 17時間
- マージ済み PR(30日)
- 18
説明
we are currently using `@sealed` because there is not an easy (or possible?) way to mock a `final` or `base` class. Some discussion here: https://github.com/dart-lang/site-www/issues/5143. We will not be able to move away from `@sealed` until a testing solution is found for the class modifier language feature, or unless we completely restructure our tests to use all fakes for final classes and no mocks (this is more overhead than the benefit is worth).
コントリビューションガイド
調査の方向性
まず、DevTools での @sealed の使用箇所と、dart-lang/site-www#5143 にリンクされている議論を読んでください。final クラスまたは base クラスをモックできるのか、それともテストで代わりに fake が必要になるのかを調査してください。@sealed を削除しても実行可能なテストカバレッジを失わなければ完了ですが、この issue にはファイルやテストのエントリーポイントが示されていません。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- dart
- 領域
- testing-qa, tooling
- issue の種類
- リファクタリング
- 難易度
- 5/5
- 見積もり時間
- 1週間以上
- 活発さ
- 停滞
- 明瞭さ
- 説明が足りない
- 初心者へのやさしさ
- 25/100