ManifestAssetBundle accepts dependencies via its constructor to allow for hermetic testing of its behavior:

However, we refer to the global Cache.flutterRoot in a couple places throughout the code:

This means that tools test using testWithoutContext have to remember to manually set this global. Example:

If nothing worse, this is noisy. Let's consider having ManifestAssetBundle accept the flutter root as a constructor parameter.