[okorohelijah]

pendingFonts waits for fonts to be loaded fails on roughly 3 out of 10 runs of Windows_x64 dart_unit_tests_shard_1, and only for PRs that make the tool run every package (.ci/, script/, Flutter rolls, ...), so it never shows up in postsubmit and blocks exactly the PRs that can't avoid it. The history and the analysis are in flutter/flutter#194407; the short version:

• The three expects in the test pass on every platform. The failure is a late error that flips an already-passed test to failed (the CI log shows the pass counter dropping from +92 to +91 -1 with no [E] block).
• After the mock HTTP fetch, _httpFetchFontAndSaveToDevice writes the font to the application support directory without awaiting the result. The test does not fake PathProviderPlatform, so under flutter test that write fails with MissingPluginException inside the dropped future, after the test body has returned. This file has a single test, so whether the runner is still attached when the error lands is a timing question; Linux consistently loses the error, Windows regularly does not.
• Keeping the test alive for two seconds after its last expect reproduces the error on Linux every time.

This change installs the same FakePathProviderPlatform pointing at a temp directory that load_font_if_necessary_test.dart already uses, so the background write succeeds and there is no straggler error. With the change, the "wait two seconds" variant passes.

Test-only change, so no version or CHANGELOG update (exempt category: PRs that only affect tests).

Fixes flutter/flutter#194407
Unblocks #13223

Pre-Review Checklist

• I read the Contributor Guide and followed the process outlined there for submitting PRs.
• I read the AI contribution guidelines and understand my responsibilities, or I am not using AI tools.
• I read the Tree Hygiene page, which explains my responsibilities.
• I read and followed the relevant style guides and ran the auto-formatter.
• I signed the CLA.
• The title of the PR starts with the name of the package surrounded by square brackets, e.g. [shared_preferences]
• I linked to at least one issue that this PR fixes in the description above.
• I followed the version and CHANGELOG instructions, using semantic versioning and the repository CHANGELOG style, or I have commented below to indicate which documented exception this PR falls under¹.
• I updated/added any relevant documentation (doc comments with ///).
• I added new tests to check the change I am making, or I have commented below to indicate which test exemption this PR falls under¹.
• All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Footnotes

1. Regular contributors who have demonstrated familiarity with the repository guidelines only need to comment if the PR is not auto-exempted by repo tooling. ↩ ↩²

[fluttergithubbot]

An existing Git SHA, 375702ed6074ac1d5d88b78438845bb0ea3c97c2, was detected, and no actions were taken.

To re-trigger presubmits after closing or re-opeing a PR, or pushing a HEAD commit (i.e. with --force) that already was pushed before, push a blank commit (git commit --allow-empty -m "Trigger Build") or rebase to continue.

[gemini-code-assist]

Code Review

This pull request updates google_fonts_text_style_test.dart to run on the VM and configures a fake PathProviderPlatform using a temporary directory to prevent late-write errors. Feedback suggests registering a tear-down callback to delete the temporary directory and avoid leaking resources.

[Piinks]

LGTM! Thank you for this!