fix(flutter): drop redundant const in nested const contexts

Making the outer FabledSwordTheme(...) and HomeData(...) constructors
const made every inner const redundant — Dart's analyzer fires
unnecessary_const for each. Strip the inner consts so the outer
context owns the const inference.
This commit is contained in:
2026-05-02 19:17:05 -04:00
parent d86c694cde
commit 1e8b8aaa2d
2 changed files with 8 additions and 8 deletions
@@ -45,9 +45,9 @@ class FabledSwordTheme extends ThemeExtension<FabledSwordTheme> {
warning: FabledSwordTokens.warning,
error: FabledSwordTokens.error,
info: FabledSwordTokens.info,
display: const TextStyle(fontFamily: FabledSwordTokens.fontDisplay),
body: const TextStyle(fontFamily: FabledSwordTokens.fontBody),
mono: const TextStyle(fontFamily: FabledSwordTokens.fontMono),
display: TextStyle(fontFamily: FabledSwordTokens.fontDisplay),
body: TextStyle(fontFamily: FabledSwordTokens.fontBody),
mono: TextStyle(fontFamily: FabledSwordTokens.fontMono),
);
@override
@@ -12,15 +12,15 @@ import 'package:minstrel/theme/theme_data.dart';
void main() {
testWidgets('home renders sections with rows', (tester) async {
const fixture = HomeData(
recentlyAddedAlbums: const [
recentlyAddedAlbums: [
AlbumRef(id: 'a', title: 'Geogaddi', artistId: 'x', artistName: 'BoC'),
],
rediscoverAlbums: const [],
rediscoverArtists: const [
rediscoverAlbums: [],
rediscoverArtists: [
ArtistRef(id: 'r', name: 'Aphex Twin'),
],
mostPlayedTracks: const [],
lastPlayedArtists: const [],
mostPlayedTracks: [],
lastPlayedArtists: [],
);
await tester.pumpWidget(ProviderScope(
overrides: [