Skip to content

Conversation

@biggs0125
Copy link
Contributor

Description

Flutter no longer provides access to the AssetManifest.json file that google_fonts was depending on:
https://docs.flutter.dev/release/breaking-changes/asset-manifest-dot-json

Instead Flutter provides an API to access this same information.

This was causing "AssetManifest.json not found" errors and test failures in other repos like dart-pad which depend on this package.

Tests

I've updated the tests to mock out the AssetManager API. We're only using the listAssets API here so this is just a matter of returning the correct List of strings for each test.

Issues

Fixes # #661

Checklist

@biggs0125 biggs0125 requested a review from a team as a code owner July 30, 2025 18:00
@biggs0125 biggs0125 requested review from guidezpl and removed request for a team July 30, 2025 18:00
@parlough
Copy link

parlough commented Jul 31, 2025

Thanks for updating this Nate!

I'm not sure if @guidezpl will be able to review this? @jwill perhaps you can help?

@biggs0125 biggs0125 force-pushed the remove-asset-manifest branch from 519ebd4 to 06c188a Compare August 1, 2025 07:04
@guidezpl guidezpl merged commit 3e9aae3 into material-foundation:main Aug 1, 2025
3 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants