flutter / flutter/devtools

Remove use of `@sealed` in DevTools

Aperta
#6,239 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
P3 tech debt
Lingua principale
Dart
Stelle
1.7k
Fork
404
Merge medio
6g 17h
PR unite (30g)
18

Descrizione

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).

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Start by reading the DevTools uses of @sealed and the linked discussion in dart-lang/site-www#5143. Investigate whether final or base classes can be mocked, or whether the tests would need fakes instead. Done means removing @sealed without losing viable test coverage, but the issue provides no file or test entry point.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
dart
Ambito
testing-qa, tooling
Tipo di issue
Refactoring
Difficoltà
5/5
Tempo stimato
Più di una settimana
Stato di attività
Ferma
Chiarezza
Da chiarire
Idoneità per principianti
25/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.