flutter / flutter/devtools

Remove use of `@sealed` in DevTools

オープン
#6,239 コメント 0 件 リアクション 0 件 担当者 0 名 GitHub で見る
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

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

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